[squid-dev] [PATCH] Fix ssl use for kerberos_ldap_group

Amos Jeffries squid3 at treenet.co.nz
Thu Jul 14 11:58:25 UTC 2016


On 3/07/2016 4:57 a.m., Markus Moeller wrote:
> Hi Amos,
> 
>   Here is a small patch for cases SSL must be used to connect to an ldap
> server.
> 

Looks okay, except for two minor things.

* some debug messages are missing a space between end of one sentence
and the '(' starting the next. Search for ".("

* what about xstrdup()'ing the output of getenv() so you can drop the
free_path variable and if-statements wrapping xfree() ?

Cheers
Amos



More information about the squid-dev mailing list