diff options
| author | Günther Deschner <gd@samba.org> | 2007-03-16 15:48:07 +0000 |
|---|---|---|
| committer | Gerald (Jerry) Carter <jerry@samba.org> | 2007-10-10 12:18:38 -0500 |
| commit | b067d986b4f45996a3920c2412210a397871af06 (patch) | |
| tree | a71d2d23802df3f3b03e14dcfe6beec840b4dd2e | |
| parent | 761f60f83a81103c4c6e39b6460b9d7062a85b5f (diff) | |
| download | samba-b067d986b4f45996a3920c2412210a397871af06.tar.gz samba-b067d986b4f45996a3920c2412210a397871af06.tar.bz2 samba-b067d986b4f45996a3920c2412210a397871af06.zip | |
r21855: Fix a memleak in the krb5 locator and comment out gfree_all() which doesn't
make sense as long as it doesn't work as an lp_unload().
Guenther
(This used to be commit 128ea9bebbb215e41d2f0576e1a73c6a362b7467)
| -rw-r--r-- | source3/libads/smb_krb5_locator.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/source3/libads/smb_krb5_locator.c b/source3/libads/smb_krb5_locator.c index 9861511714..be14a12697 100644 --- a/source3/libads/smb_krb5_locator.c +++ b/source3/libads/smb_krb5_locator.c @@ -239,7 +239,7 @@ void smb_krb5_locator_close(void *private_data) { DEBUG(10,("smb_krb5_locator_close: called\n")); - gfree_all(); + /* gfree_all(); */ } /** @@ -324,7 +324,7 @@ krb5_error_code smb_krb5_locator_lookup(void *private_data, /* if we didn't found any KDCs on our site go to the main list */ if (NT_STATUS_IS_OK(status) && sitename && (count == 0)) { - ip_list = NULL; + SAFE_FREE(ip_list); SAFE_FREE(sitename); status = get_kdc_list(realm, NULL, &ip_list, &count); } @@ -365,6 +365,8 @@ krb5_error_code smb_krb5_locator_lookup(void *private_data, } } + SAFE_FREE(ip_list); + return ret; } |