diff options
| author | Gerald Carter <jerry@samba.org> | 2003-10-03 21:43:56 +0000 |
|---|---|---|
| committer | Gerald Carter <jerry@samba.org> | 2003-10-03 21:43:56 +0000 |
| commit | 9cfd26141cf9530f27e673bfae119d9879894dd4 (patch) | |
| tree | a41a3ff126ac6bfcd824db4e147fadbb6a31a1f2 /source3/libads/ldap_utils.c | |
| parent | ec7716734ef8860ec8071ef7872672c4cb02a4e8 (diff) | |
| download | samba-9cfd26141cf9530f27e673bfae119d9879894dd4.tar.gz samba-9cfd26141cf9530f27e673bfae119d9879894dd4.tar.bz2 samba-9cfd26141cf9530f27e673bfae119d9879894dd4.zip | |
don't call ads_destroy() twice; fixes segfault in winbindd when DC goes down; bug 437
(This used to be commit 9da4d1f7dbb289dd1db5e57a4fd78004bbfbd26b)
Diffstat (limited to 'source3/libads/ldap_utils.c')
| -rw-r--r-- | source3/libads/ldap_utils.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/source3/libads/ldap_utils.c b/source3/libads/ldap_utils.c index 1fa9ebfc97..4142bceabc 100644 --- a/source3/libads/ldap_utils.c +++ b/source3/libads/ldap_utils.c @@ -54,15 +54,20 @@ ADS_STATUS ads_do_search_retry(ADS_STRUCT *ads, const char *bind_path, int scope return status; } - if (*res) ads_msgfree(ads, *res); + if (*res) + ads_msgfree(ads, *res); *res = NULL; + DEBUG(3,("Reopening ads connection to realm '%s' after error %s\n", ads->config.realm, ads_errstr(status))); + if (ads->ld) { ldap_unbind(ads->ld); } + ads->ld = NULL; status = ads_connect(ads); + if (!ADS_ERR_OK(status)) { DEBUG(1,("ads_search_retry: failed to reconnect (%s)\n", ads_errstr(status))); |