Apologies. I must have overlooked it. Here is the updated patch
Markus "Amos Jeffries" wrote in message news:53e5c2df.3080...@treenet.co.nz...
On 9/08/2014 8:32 a.m., Markus Moeller wrote:
It should be in there or did I miss some ?
The original bits are still there in the patch copy mailed to the list.
Specifically in helpers/external_acl/kerberos_ldap_group/support_ldap.cc
get_bin_attributes() bits I can see :
* redux function setup:
+ LDAPMessage *msg;
+ char **attr_value = NULL;
+ int *attr_len=NULL;
+ size_t max_attr = 0;
+
+ attr_value = *ret_value;
+ attr_len = *ret_len;
should be:
+ char **attr_value = *ret_value;
+ int *attr_len = *ret_len;
+ size_t max_attr = 0;
* main for loop:
- for (msg = ldap_first_entry
+ for (LDAPMessage *msg = ldap_first_entry
* drop these:
BerElement *b;
char *attr;
* switch case should be:
case LDAP_RES_SEARCH_ENTRY:
{
BerElement *b = NULL;
...
ber_free(b, 0);
} break;
* for loops inside that switch case should be:
- for (attr = ldap_first_attribute...
+ for (char *attr = ldap_first_attribute...
- int il; for (il = 0; ...
+ for (int il = 0; ...
Otherwise it looks okay.
Amos
Markus -----Original Message----- From: Amos Jeffries Sent: Friday, August 08, 2014 1:28 PM To: squid-dev@squid-cache.org ; Markus Moeller Subject: Re: [PATCH] Kerberos configure patch + some cleanup On 8/08/2014 8:02 a.m., Markus Moeller wrote:Are there any objections to this patch ?The audit results from me I accidentally sent in private. Do you have an updated patch with those fixes? Amos
trunk_kerberos_cleanup_8.patch
Description: Binary data
