getaddrinfo-is-threadsafe

OpenLDAP upstream conservatively assumes that certain resolver functions
(getaddrinfo, getnameinfo, res_query, dn_expand) are not re-entrant; but we
know that the glibc implementations of these functions are thread-safe, so
we should bypass the use of this mutex.  This fixes a locking problem when
an application uses libldap and libnss-ldap is also used for hosts
resolution.

Closes Debian bug #340601.

Not suitable for forwarding upstream; might be made suitable by adding a
configure-time check for glibc and disabling the mutex only on known
thread-safe implementations.

Gbp-Pq: Name getaddrinfo-is-threadsafe

diff --git a/libraries/libldap/os-ip.c b/libraries/libldap/os-ip.c
index a84735e7afead4525b1800feb90fedd7619e009b..ba0354bc8488f147ba57c0588fae8848de610236 100644 (file)
--- a/libraries/libldap/os-ip.c
+++ b/libraries/libldap/os-ip.c
@@ -645,13 +645,7 @@ ldap_connect_to_host(LDAP *ld, Sockbuf *sb,
        hints.ai_socktype = socktype;
        snprintf(serv, sizeof serv, "%d", port );
 
-       /* most getaddrinfo(3) use non-threadsafe resolver libraries */
-       LDAP_MUTEX_LOCK(&ldap_int_resolv_mutex);
-
        err = getaddrinfo( host, serv, &hints, &res );
-
-       LDAP_MUTEX_UNLOCK(&ldap_int_resolv_mutex);
-
        if ( err != 0 ) {
                Debug1(LDAP_DEBUG_TRACE,
                        "ldap_connect_to_host: getaddrinfo failed: %s\n",

diff --git a/libraries/libldap/util-int.c b/libraries/libldap/util-int.c
index 57c65233818e466c3cf5a69bbd2cd17320df6bf2..8cefcc089cd705e9f7015b6ccb6cd76aaf8e46e2 100644 (file)
--- a/libraries/libldap/util-int.c
+++ b/libraries/libldap/util-int.c
@@ -559,9 +559,7 @@ int ldap_pvt_get_hname(
        int rc;
 #if defined( HAVE_GETNAMEINFO )
 
-       LDAP_MUTEX_LOCK( &ldap_int_resolv_mutex );
        rc = getnameinfo( sa, len, name, namelen, NULL, 0, 0 );
-       LDAP_MUTEX_UNLOCK( &ldap_int_resolv_mutex );
        if ( rc ) *err = (char *)AC_GAI_STRERROR( rc );
        return rc;