diff options
author | Günther Deschner <gd@samba.org> | 2005-11-23 11:29:55 +0000 |
---|---|---|
committer | Gerald (Jerry) Carter <jerry@samba.org> | 2007-10-10 11:05:33 -0500 |
commit | 3ccea032cb681426a1b29907a44b87b8a94e0840 (patch) | |
tree | 47eff2811d782cdfdc44b5a9cda6fb0d677f0edc /source/lib | |
parent | 95543fab0f6aa1c483b40247c16cde79cbc9c012 (diff) | |
download | samba-3ccea032cb681426a1b29907a44b87b8a94e0840.tar.gz samba-3ccea032cb681426a1b29907a44b87b8a94e0840.tar.xz samba-3ccea032cb681426a1b29907a44b87b8a94e0840.zip |
r11876: When we are using START_TLS to secure the LDAP connection, we *have* to
call START_TLS again after rebinding to another LDAP server. (ldaps://
uri's are handled at by recent versions of OpenLDAP).
Guenther
Diffstat (limited to 'source/lib')
-rw-r--r-- | source/lib/smbldap.c | 15 |
1 files changed, 12 insertions, 3 deletions
diff --git a/source/lib/smbldap.c b/source/lib/smbldap.c index 75842ec193e..a3ebe72df21 100644 --- a/source/lib/smbldap.c +++ b/source/lib/smbldap.c @@ -777,9 +777,18 @@ static int rebindproc_connect_with_state (LDAP *ldap_struct, { struct smbldap_state *ldap_state = arg; int rc; - DEBUG(5,("rebindproc_connect_with_state: Rebinding as \"%s\"\n", - ldap_state->bind_dn)); - + int version; + + DEBUG(5,("rebindproc_connect_with_state: Rebinding to %s as \"%s\"\n", + url, ldap_state->bind_dn)); + + /* call START_TLS again (ldaps:// is handled by the OpenLDAP library + * itself) before rebinding to another LDAP server to avoid to expose + * our credentials. At least *try* to secure the connection - Guenther */ + + smb_ldap_upgrade_conn(ldap_struct, &version); + smb_ldap_start_tls(ldap_struct, version); + /** @TODO Should we be doing something to check what servers we rebind to? Could we get a referral to a machine that we don't want to give our username and password to? */ |