[PATCH] Bugs LDAPSAM : 2 fixes

Jeremy Allison jra at samba.org
Fri Oct 31 19:17:16 GMT 2003

On Fri, Oct 31, 2003 at 12:12:43PM +0100, Aurélien Degrémont wrote:
> Inside ldapsam_update_group_mapping_entry():
> The ldap_mods_free() call must really be moved after the 
> smbldap_get_dn() call, otherwise the ldapsam_update_group_mapping will 
> crash and keep crashing.
> In fact, 'result' is a chained-list and 'entry' refers to the first 
> element of this list. They point on the same address, so, when 'result' 
> is freed, 'entry' is freed too, and as a result, the smbldap_get_dn() 
> call crashes as it uses a memory-free 'entry'. Ask me if it's not clear 
> :). But understand that the patch must be applied completely :). 

Applied, thanks !


More information about the samba-technical mailing list