[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 !
Jeremy.
More information about the samba-technical
mailing list