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