]> git.sur5r.net Git - openldap/blobdiff - libraries/libldap/tls.c
More cleanup in ldap_pvt_tls_destroy()
[openldap] / libraries / libldap / tls.c
index 62c04e3d6a1c3e117e3c533359548aaa87ec3847..3c409404db004c0f0317faee1b5032c967bf7bca 100644 (file)
@@ -1,6 +1,6 @@
 /* $OpenLDAP$ */
 /*
- * Copyright 1998-2000 The OpenLDAP Foundation, All Rights Reserved.
+ * Copyright 1998-2002 The OpenLDAP Foundation, All Rights Reserved.
  * COPYING RESTRICTIONS APPLY, see COPYRIGHT file
  *
  * tls.c - Handle tls/ssl using SSLeay or OpenSSL.
@@ -31,6 +31,7 @@
 #include <openssl/x509v3.h>
 #include <openssl/err.h>
 #include <openssl/rand.h>
+#include <openssl/safestack.h>
 #elif defined( HAVE_SSL_H )
 #include <ssl.h>
 #endif
@@ -45,7 +46,7 @@ static char *tls_opt_ciphersuite = NULL;
 static char *tls_opt_randfile = NULL;
 
 #define HAS_TLS( sb )  ber_sockbuf_ctrl( sb, LBER_SB_OPT_HAS_IO, \
-                               (void *)&ldap_pvt_sockbuf_io_tls )
+                               (void *)&sb_tls_sbio )
 
 static void tls_report_error( void );
 
@@ -97,6 +98,44 @@ static void tls_init_threads( void )
 }
 #endif /* LDAP_R_COMPILE */
 
+/*
+ * Tear down the TLS subsystem. Should only be called once.
+ */
+void
+ldap_pvt_tls_destroy( void )
+{
+       SSL_CTX_free(tls_def_ctx);
+       tls_def_ctx = NULL;
+
+       EVP_cleanup();
+       ERR_free_strings();
+
+       if ( tls_opt_certfile ) {
+               LDAP_FREE( tls_opt_certfile );
+               tls_opt_certfile = NULL;
+       }
+       if ( tls_opt_keyfile ) {
+               LDAP_FREE( tls_opt_keyfile );
+               tls_opt_keyfile = NULL;
+       }
+       if ( tls_opt_cacertfile ) {
+               LDAP_FREE( tls_opt_cacertfile );
+               tls_opt_cacertfile = NULL;
+       }
+       if ( tls_opt_cacertdir ) {
+               LDAP_FREE( tls_opt_cacertdir );
+               tls_opt_cacertdir = NULL;
+       }
+       if ( tls_opt_ciphersuite ) {
+               LDAP_FREE( tls_opt_ciphersuite );
+               tls_opt_ciphersuite = NULL;
+       }
+       if ( tls_opt_randfile ) {
+               LDAP_FREE( tls_opt_randfile );
+               tls_opt_randfile = NULL;
+       }
+}
+
 /*
  * Initialize TLS subsystem. Should be called only once.
  */
@@ -137,12 +176,14 @@ ldap_pvt_tls_init_def_ctx( void )
                tls_def_ctx = SSL_CTX_new( SSLv23_method() );
                if ( tls_def_ctx == NULL ) {
                        Debug( LDAP_DEBUG_ANY,
-                              "TLS: could not allocate default ctx.\n",0,0,0);
+                      "TLS: could not allocate default ctx (%d).\n",
+                               ERR_peek_error(),0,0);
                        goto error_exit;
                }
                if ( tls_opt_ciphersuite &&
                     !SSL_CTX_set_cipher_list( tls_def_ctx,
-                       tls_opt_ciphersuite ) ) {
+                       tls_opt_ciphersuite ) )
+               {
                        Debug( LDAP_DEBUG_ANY,
                               "TLS: could not set cipher list %s.\n",
                               tls_opt_ciphersuite, 0, 0 );
@@ -155,17 +196,21 @@ ldap_pvt_tls_init_def_ctx( void )
                                                             tls_opt_cacertdir )
                             || !SSL_CTX_set_default_verify_paths( tls_def_ctx ) )
                        {
-                               Debug( LDAP_DEBUG_ANY,
-                       "TLS: could not load verify locations (file:`%s',dir:`%s').\n",
-                                      tls_opt_cacertfile,tls_opt_cacertdir,0);
+                               Debug( LDAP_DEBUG_ANY, "TLS: "
+                                       "could not load verify locations (file:`%s',dir:`%s').\n",
+                                       tls_opt_cacertfile ? tls_opt_cacertfile : "",
+                                       tls_opt_cacertdir ? tls_opt_cacertdir : "",
+                                       0 );
                                tls_report_error();
                                goto error_exit;
                        }
                        calist = get_ca_list( tls_opt_cacertfile, tls_opt_cacertdir );
                        if ( !calist ) {
-                               Debug( LDAP_DEBUG_ANY,
-                       "TLS: could not load client CA list (file:`%s',dir:`%s').\n",
-                                      tls_opt_cacertfile,tls_opt_cacertdir,0);
+                               Debug( LDAP_DEBUG_ANY, "TLS: "
+                                       "could not load client CA list (file:`%s',dir:`%s').\n",
+                                       tls_opt_cacertfile ? tls_opt_cacertfile : "",
+                                       tls_opt_cacertdir ? tls_opt_cacertdir : "",
+                                       0 );
                                tls_report_error();
                                goto error_exit;
                        }
@@ -174,7 +219,8 @@ ldap_pvt_tls_init_def_ctx( void )
                if ( tls_opt_keyfile &&
                     !SSL_CTX_use_PrivateKey_file( tls_def_ctx,
                                                   tls_opt_keyfile,
-                                                  SSL_FILETYPE_PEM ) ) {
+                                                  SSL_FILETYPE_PEM ) )
+               {
                        Debug( LDAP_DEBUG_ANY,
                               "TLS: could not use key file `%s'.\n",
                               tls_opt_keyfile,0,0);
@@ -184,7 +230,8 @@ ldap_pvt_tls_init_def_ctx( void )
                if ( tls_opt_certfile &&
                     !SSL_CTX_use_certificate_file( tls_def_ctx,
                                                    tls_opt_certfile,
-                                                   SSL_FILETYPE_PEM ) ) {
+                                                   SSL_FILETYPE_PEM ) )
+               {
                        Debug( LDAP_DEBUG_ANY,
                               "TLS: could not use certificate `%s'.\n",
                               tls_opt_certfile,0,0);
@@ -192,7 +239,8 @@ ldap_pvt_tls_init_def_ctx( void )
                        goto error_exit;
                }
                if ( ( tls_opt_certfile || tls_opt_keyfile ) &&
-                    !SSL_CTX_check_private_key( tls_def_ctx ) ) {
+                    !SSL_CTX_check_private_key( tls_def_ctx ) )
+               {
                        Debug( LDAP_DEBUG_ANY,
                               "TLS: private key mismatch.\n",
                               0,0,0);
@@ -249,8 +297,7 @@ alloc_handle( void *ctx_arg )
        if ( ctx_arg ) {
                ctx = (SSL_CTX *) ctx_arg;
        } else {
-               if ( ldap_pvt_tls_init_def_ctx() < 0 )
-                       return NULL;
+               if ( ldap_pvt_tls_init_def_ctx() < 0 ) return NULL;
                ctx = tls_def_ctx;
        }
 
@@ -259,10 +306,6 @@ alloc_handle( void *ctx_arg )
                Debug( LDAP_DEBUG_ANY,"TLS: can't create ssl handle.\n",0,0,0);
                return NULL;
        }
-
-       if ( tls_opt_trace ) {
-               SSL_set_info_callback( ssl, tls_info_cb );
-       }
        return ssl;
 }
 
@@ -297,7 +340,7 @@ struct tls_data {
        Sockbuf_IO_Desc         *sbiod;
 };
 
-extern BIO_METHOD ldap_pvt_sb_bio_method;
+static BIO_METHOD sb_tls_bio_method;
 
 static int
 sb_tls_setup( Sockbuf_IO_Desc *sbiod, void *arg )
@@ -313,7 +356,7 @@ sb_tls_setup( Sockbuf_IO_Desc *sbiod, void *arg )
        
        p->ssl = (SSL *)arg;
        p->sbiod = sbiod;
-       bio = BIO_new( &ldap_pvt_sb_bio_method );
+       bio = BIO_new( &sb_tls_bio_method );
        bio->ptr = (void *)p;
        SSL_set_bio( p->ssl, bio, bio );
        sbiod->sbiod_pvt = p;
@@ -427,7 +470,7 @@ sb_tls_write( Sockbuf_IO_Desc *sbiod, void *buf, ber_len_t len)
        return ret;
 }
 
-Sockbuf_IO ldap_pvt_sockbuf_io_tls =
+static Sockbuf_IO sb_tls_sbio =
 {
        sb_tls_setup,           /* sbi_setup */
        sb_tls_remove,          /* sbi_remove */
@@ -482,7 +525,7 @@ sb_tls_bio_read( BIO *b, char *buf, int len )
 }
 
 static int
-sb_tls_bio_write( BIO *b, char *buf, int len )
+sb_tls_bio_write( BIO *b, const char *buf, int len )
 {
        struct tls_data         *p;
        int                     ret;
@@ -495,7 +538,7 @@ sb_tls_bio_write( BIO *b, char *buf, int len )
        if ( p == NULL || p->sbiod == NULL )
                return 0;
 
-       ret = LBER_SBIOD_WRITE_NEXT( p->sbiod, buf, len );
+       ret = LBER_SBIOD_WRITE_NEXT( p->sbiod, (char *)buf, len );
 
        BIO_clear_retry_flags( b );
        if ( ret < 0 && errno == EWOULDBLOCK )
@@ -505,7 +548,7 @@ sb_tls_bio_write( BIO *b, char *buf, int len )
 }
 
 static long
-sb_tls_bio_ctrl( BIO *b, int cmd, long num, char *ptr )
+sb_tls_bio_ctrl( BIO *b, int cmd, long num, void *ptr )
 {
        if ( cmd == BIO_CTRL_FLUSH ) {
                /* The OpenSSL library needs this */
@@ -521,12 +564,12 @@ sb_tls_bio_gets( BIO *b, char *buf, int len )
 }
 
 static int
-sb_tls_bio_puts( BIO *b, char *str )
+sb_tls_bio_puts( BIO *b, const char *str )
 {
        return sb_tls_bio_write( b, str, strlen( str ) );
 }
        
-BIO_METHOD ldap_pvt_sb_bio_method =
+static BIO_METHOD sb_tls_bio_method =
 {
        ( 100 | 0x400 ),                /* it's a source/sink BIO */
        "sockbuf glue",
@@ -552,24 +595,34 @@ BIO_METHOD ldap_pvt_sb_bio_method =
  *             and call again.
  */
 
-int
-ldap_pvt_tls_connect( LDAP *ld, Sockbuf *sb, void *ctx_arg )
+static int
+ldap_int_tls_connect( LDAP *ld, LDAPConn *conn )
 {
+       Sockbuf *sb = conn->lconn_sb;
        int     err;
        SSL     *ssl;
 
        if ( HAS_TLS( sb ) ) {
                ber_sockbuf_ctrl( sb, LBER_SB_OPT_GET_SSL, (void *)&ssl );
+
        } else {
-               ssl = alloc_handle( ctx_arg );
-               if ( ssl == NULL )
-                       return -1;
+               void *ctx = ld->ld_defconn
+                       ? ld->ld_defconn->lconn_tls_ctx : NULL;
+
+               ssl = alloc_handle( ctx );
+
+               if ( ssl == NULL ) return -1;
+
 #ifdef LDAP_DEBUG
                ber_sockbuf_add_io( sb, &ber_sockbuf_io_debug,
                        LBER_SBIOD_LEVEL_TRANSPORT, (void *)"tls_" );
 #endif
-               ber_sockbuf_add_io( sb, &ldap_pvt_sockbuf_io_tls,
+               ber_sockbuf_add_io( sb, &sb_tls_sbio,
                        LBER_SBIOD_LEVEL_TRANSPORT, (void *)ssl );
+
+               if( ctx == NULL ) {
+                       conn->lconn_tls_ctx = tls_def_ctx;
+               }
        }
 
        err = SSL_connect( ssl );
@@ -578,14 +631,15 @@ ldap_pvt_tls_connect( LDAP *ld, Sockbuf *sb, void *ctx_arg )
        errno = WSAGetLastError();
 #endif
        if ( err <= 0 ) {
-               if ( update_flags( sb, ssl, err ))
+               if ( update_flags( sb, ssl, err )) {
                        return 1;
+               }
                if ((err = ERR_peek_error())) {
                        char buf[256];
                        ld->ld_error = LDAP_STRDUP(ERR_error_string(err, buf));
                }
                Debug( LDAP_DEBUG_ANY,"TLS: can't connect.\n",0,0,0);
-               ber_sockbuf_remove_io( sb, &ldap_pvt_sockbuf_io_tls,
+               ber_sockbuf_remove_io( sb, &sb_tls_sbio,
                        LBER_SBIOD_LEVEL_TRANSPORT );
 #ifdef LDAP_DEBUG
                ber_sockbuf_remove_io( sb, &ber_sockbuf_io_debug,
@@ -593,6 +647,7 @@ ldap_pvt_tls_connect( LDAP *ld, Sockbuf *sb, void *ctx_arg )
 #endif
                return -1;
        }
+
        return 0;
 }
 
@@ -616,7 +671,7 @@ ldap_pvt_tls_accept( Sockbuf *sb, void *ctx_arg )
                ber_sockbuf_add_io( sb, &ber_sockbuf_io_debug,
                        LBER_SBIOD_LEVEL_TRANSPORT, (void *)"tls_" );
 #endif
-               ber_sockbuf_add_io( sb, &ldap_pvt_sockbuf_io_tls,
+               ber_sockbuf_add_io( sb, &sb_tls_sbio,
                        LBER_SBIOD_LEVEL_TRANSPORT, (void *)ssl );
        }
 
@@ -630,7 +685,7 @@ ldap_pvt_tls_accept( Sockbuf *sb, void *ctx_arg )
                        return 1;
                Debug( LDAP_DEBUG_ANY,"TLS: can't accept.\n",0,0,0 );
                tls_report_error();
-               ber_sockbuf_remove_io( sb, &ldap_pvt_sockbuf_io_tls,
+               ber_sockbuf_remove_io( sb, &sb_tls_sbio,
                        LBER_SBIOD_LEVEL_TRANSPORT );
 #ifdef LDAP_DEBUG
                ber_sockbuf_remove_io( sb, &ber_sockbuf_io_debug,
@@ -651,7 +706,7 @@ ldap_pvt_tls_inplace ( Sockbuf *sb )
 }
 
 void *
-ldap_pvt_tls_sb_handle( Sockbuf *sb )
+ldap_pvt_tls_sb_ctx( Sockbuf *sb )
 {
        void                    *p;
        
@@ -663,12 +718,6 @@ ldap_pvt_tls_sb_handle( Sockbuf *sb )
        return NULL;
 }
 
-void *
-ldap_pvt_tls_get_handle( LDAP *ld )
-{
-       return ldap_pvt_tls_sb_handle( ld->ld_sb );
-}
-
 int
 ldap_pvt_tls_get_strength( void *s )
 {
@@ -679,7 +728,7 @@ ldap_pvt_tls_get_strength( void *s )
 }
 
 
-const char *
+char *
 ldap_pvt_tls_get_peer( void *s )
 {
     X509 *x;
@@ -697,12 +746,33 @@ ldap_pvt_tls_get_peer( void *s )
     return p;
 }
 
+char *
+ldap_pvt_tls_get_peer_dn( void *s )
+{
+       X509 *x;
+       X509_NAME *xn;
+       char buf[2048], *p, *dn;
+
+       x = SSL_get_peer_certificate((SSL *)s);
+
+       if (!x) return NULL;
+    
+       xn = X509_get_subject_name(x);
+       p = X509_NAME_oneline(xn, buf, sizeof(buf));
+
+       dn = ldap_dcedn2dn( p );
+
+       X509_free(x);
+       return dn;
+}
+
 char *
 ldap_pvt_tls_get_peer_hostname( void *s )
 {
        X509 *x;
        X509_NAME *xn;
        char buf[2048], *p;
+       int ret;
 
        x = SSL_get_peer_certificate((SSL *)s);
 
@@ -711,7 +781,8 @@ ldap_pvt_tls_get_peer_hostname( void *s )
        
        xn = X509_get_subject_name(x);
 
-       if ( X509_NAME_get_text_by_NID(xn, NID_commonName, buf, sizeof(buf)) == -1 ) {
+       ret = X509_NAME_get_text_by_NID(xn, NID_commonName, buf, sizeof(buf));
+       if( ret == -1 ) {
                X509_free(x);
                return NULL;
        }
@@ -721,6 +792,103 @@ ldap_pvt_tls_get_peer_hostname( void *s )
        return p;
 }
 
+int
+ldap_pvt_tls_check_hostname( void *s, const char *name_in )
+{
+    int i, ret = LDAP_LOCAL_ERROR;
+    X509 *x;
+       const char *name;
+
+       if( ldap_int_hostname &&
+               ( !name_in || !strcasecmp( name_in, "localhost" ) ) )
+       {
+               name = ldap_int_hostname;
+       } else {
+               name = name_in;
+       }
+
+    x = SSL_get_peer_certificate((SSL *)s);
+    if (!x)
+    {
+       Debug( LDAP_DEBUG_ANY,
+               "TLS: unable to get peer certificate.\n",
+               0, 0, 0 );
+       return ret;
+    }
+
+    i = X509_get_ext_by_NID(x, NID_subject_alt_name, -1);
+    if (i >= 0)
+    {
+       X509_EXTENSION *ex;
+       STACK_OF(GENERAL_NAME) *alt;
+
+       ex = X509_get_ext(x, i);
+       alt = X509V3_EXT_d2i(ex);
+       if (alt)
+       {
+           int n, len1, len2;
+           char *domain;
+           GENERAL_NAME *gn;
+           X509V3_EXT_METHOD *method;
+
+           len1 = strlen(name);
+           n = sk_GENERAL_NAME_num(alt);
+           domain = strchr(name, '.');
+           if (domain)
+               len2 = len1 - (domain-name);
+           for (i=0; i<n; i++)
+           {
+               gn = sk_GENERAL_NAME_value(alt, i);
+               if (gn->type == GEN_DNS)
+               {
+                   char *sn = ASN1_STRING_data(gn->d.ia5);
+                   int sl = ASN1_STRING_length(gn->d.ia5);
+
+                   /* Is this an exact match? */
+                   if ((len1 == sl) && !strncasecmp(name, sn, len1))
+                       break;
+
+                   /* Is this a wildcard match? */
+                   if ((*sn == '*') && domain && (len2 == sl-1) &&
+                       !strncasecmp(domain, sn+1, len2))
+                       break;
+               }
+           }
+           method = X509V3_EXT_get(ex);
+           method->ext_free(alt);
+           if (i < n)  /* Found a match */
+               ret = LDAP_SUCCESS;
+       }
+    }
+
+    if (ret != LDAP_SUCCESS)
+    {
+       X509_NAME *xn;
+       char buf[2048];
+
+       xn = X509_get_subject_name(x);
+
+       if (X509_NAME_get_text_by_NID(xn, NID_commonName, buf, sizeof(buf))
+           == -1)
+       {
+           Debug( LDAP_DEBUG_ANY,
+                   "TLS: unable to get common name from peer certificate.\n",
+                   0, 0, 0 );
+       } else if (strcasecmp(name, buf))
+       {
+           Debug( LDAP_DEBUG_ANY, "TLS: hostname (%s) does not match "
+                   "common name in certificate (%s).\n", 
+                   name, buf, 0 );
+           ret =  LDAP_CONNECT_ERROR;
+       } else
+       {
+           ret = LDAP_SUCCESS;
+       }
+    }
+    X509_free(x);
+    return ret;
+}
+
 const char *
 ldap_pvt_tls_get_peer_issuer( void *s )
 {
@@ -744,7 +912,7 @@ ldap_pvt_tls_get_peer_issuer( void *s )
 }
 
 int
-ldap_int_tls_config( struct ldapoptions *lo, int option, const char *arg )
+ldap_int_tls_config( LDAP *ld, int option, const char *arg )
 {
        int i;
 
@@ -754,12 +922,14 @@ ldap_int_tls_config( struct ldapoptions *lo, int option, const char *arg )
        case LDAP_OPT_X_TLS_CERTFILE:
        case LDAP_OPT_X_TLS_KEYFILE:
        case LDAP_OPT_X_TLS_RANDOM_FILE:
-               return ldap_pvt_tls_set_option( NULL, option, (void *) arg );
+               return ldap_pvt_tls_set_option( ld, option, (void *) arg );
+
        case LDAP_OPT_X_TLS_REQUIRE_CERT:
                i = ( ( strcasecmp( arg, "on" ) == 0 ) ||
                      ( strcasecmp( arg, "yes" ) == 0) ||
                      ( strcasecmp( arg, "true" ) == 0 ) );
-               return ldap_pvt_tls_set_option( NULL, option, (void *) &i );
+               return ldap_pvt_tls_set_option( ld, option, (void *) &i );
+
        case LDAP_OPT_X_TLS:
                i = -1;
                if ( strcasecmp( arg, "never" ) == 0 )
@@ -772,8 +942,10 @@ ldap_int_tls_config( struct ldapoptions *lo, int option, const char *arg )
                        i = LDAP_OPT_X_TLS_TRY ;
                if ( strcasecmp( arg, "hard" ) == 0 )
                        i = LDAP_OPT_X_TLS_HARD ;
-               if (i >= 0)
-                       return ldap_pvt_tls_set_option( lo, option, &i );
+
+               if (i >= 0) {
+                       return ldap_pvt_tls_set_option( ld, option, &i );
+               }
                return -1;
        }
 
@@ -781,17 +953,36 @@ ldap_int_tls_config( struct ldapoptions *lo, int option, const char *arg )
 }
 
 int
-ldap_pvt_tls_get_option( struct ldapoptions *lo, int option, void *arg )
+ldap_pvt_tls_get_option( LDAP *ld, int option, void *arg )
 {
+       struct ldapoptions *lo;
+
+       if( ld != NULL ) {
+               assert( LDAP_VALID( ld ) );
+
+               if( !LDAP_VALID( ld ) ) {
+                       return LDAP_OPT_ERROR;
+               }
+
+               lo = &ld->ld_options;
+
+       } else {
+               /* Get pointer to global option structure */
+               lo = LDAP_INT_GLOBAL_OPT();   
+               if ( lo == NULL ) {
+                       return LDAP_NO_MEMORY;
+               }
+       }
+
        switch( option ) {
        case LDAP_OPT_X_TLS:
                *(int *)arg = lo->ldo_tls_mode;
                break;
-       case LDAP_OPT_X_TLS_CERT:
-               if ( lo == NULL )
+       case LDAP_OPT_X_TLS_CTX:
+               if ( ld == NULL )
                        *(void **)arg = (void *) tls_def_ctx;
                else
-                       *(void **)arg = lo->ldo_tls_ctx;
+                       *(void **)arg = ld->ld_defconn->lconn_tls_ctx;
                break;
        case LDAP_OPT_X_TLS_CACERTFILE:
                *(char **)arg = tls_opt_cacertfile ?
@@ -813,7 +1004,8 @@ ldap_pvt_tls_get_option( struct ldapoptions *lo, int option, void *arg )
                *(int *)arg = tls_opt_require_cert;
                break;
        case LDAP_OPT_X_TLS_RANDOM_FILE:
-               *(char **)arg = tls_opt_randfile;
+               *(char **)arg = tls_opt_randfile ?
+                       LDAP_STRDUP( tls_opt_randfile ) : NULL;
                break;
        default:
                return -1;
@@ -822,8 +1014,27 @@ ldap_pvt_tls_get_option( struct ldapoptions *lo, int option, void *arg )
 }
 
 int
-ldap_pvt_tls_set_option( struct ldapoptions *lo, int option, void *arg )
+ldap_pvt_tls_set_option( LDAP *ld, int option, void *arg )
 {
+       struct ldapoptions *lo;
+
+       if( ld != NULL ) {
+               assert( LDAP_VALID( ld ) );
+
+               if( !LDAP_VALID( ld ) ) {
+                       return LDAP_OPT_ERROR;
+               }
+
+               lo = &ld->ld_options;
+
+       } else {
+               /* Get pointer to global option structure */
+               lo = LDAP_INT_GLOBAL_OPT();   
+               if ( lo == NULL ) {
+                       return LDAP_NO_MEMORY;
+               }
+       }
+
        switch( option ) {
        case LDAP_OPT_X_TLS:
                switch( *(int *) arg ) {
@@ -840,17 +1051,17 @@ ldap_pvt_tls_set_option( struct ldapoptions *lo, int option, void *arg )
                }
                return -1;
 
-       case LDAP_OPT_X_TLS_CERT:
-               if ( lo == NULL ) {
+       case LDAP_OPT_X_TLS_CTX:
+               if ( ld == NULL ) {
                        tls_def_ctx = (SSL_CTX *) arg;
 
                } else {
-                       lo->ldo_tls_ctx = arg;
+                       ld->ld_defconn->lconn_tls_ctx = arg;
                }
                return 0;
        }
 
-       if ( lo != NULL ) {
+       if ( ld != NULL ) {
                return -1;
        }
 
@@ -889,54 +1100,42 @@ ldap_pvt_tls_set_option( struct ldapoptions *lo, int option, void *arg )
 }
 
 int
-ldap_pvt_tls_start ( LDAP *ld, Sockbuf *sb, void *ctx_arg )
+ldap_int_tls_start ( LDAP *ld, LDAPConn *conn, LDAPURLDesc *srv )
 {
-       char *peer_cert_cn, *peer_hostname;
+       Sockbuf *sb = conn->lconn_sb;
+       char *host;
        void *ssl;
 
+       if( srv ) {
+               host = srv->lud_host;
+       } else {
+               host = conn->lconn_server->lud_host;
+       }
+
+       /* avoid NULL host */
+       if( host == NULL ) {
+               host = "localhost";
+       }
+
        (void) ldap_pvt_tls_init();
 
        /*
         * Fortunately, the lib uses blocking io...
         */
-       if ( ldap_pvt_tls_connect( ld, sb, ctx_arg ) < 0 ) {
-               return LDAP_CONNECT_ERROR;
+       if ( ldap_int_tls_connect( ld, conn ) < 0 ) {
+               ld->ld_errno = LDAP_CONNECT_ERROR;
+               return (ld->ld_errno);
        }
 
-       ssl = (void *) ldap_pvt_tls_sb_handle( sb );
+       ssl = (void *) ldap_pvt_tls_sb_ctx( sb );
+       assert( ssl != NULL );
+
        /* 
-        * compare hostname of server with name in certificate 
+        * compare host with name(s) in certificate
         */
-       peer_cert_cn = ldap_pvt_tls_get_peer_hostname( ssl );
-       if ( !peer_cert_cn ) {
-               /* could not get hostname from peer certificate */
-               Debug( LDAP_DEBUG_ANY,
-                       "TLS: unable to get common name from peer certificate.\n",
-                       0, 0, 0 );
-               return LDAP_LOCAL_ERROR;
-       }
-       
-       peer_hostname = ldap_host_connected_to( sb );
-       if ( !peer_hostname ) {
-               /* could not lookup hostname */
-               Debug( LDAP_DEBUG_ANY,
-                       "TLS: unable to reverse lookup peer hostname.\n",
-                       0, 0, 0 );
-               LDAP_FREE( peer_cert_cn );
-               return LDAP_LOCAL_ERROR;
-       }
-
-       if ( strcasecmp(peer_hostname, peer_cert_cn) != 0 ) {
-               Debug( LDAP_DEBUG_ANY, "TLS: hostname (%s) does not match "
-                       "common name in certificate (%s).", 
-                       peer_hostname, peer_cert_cn, 0 );
-               LDAP_FREE( peer_cert_cn );
-               LDAP_FREE( peer_hostname );
-               return LDAP_CONNECT_ERROR;
-
-       } else {
-               LDAP_FREE( peer_cert_cn );
-               LDAP_FREE( peer_hostname );
+       ld->ld_errno = ldap_pvt_tls_check_hostname( ssl, host );
+       if (ld->ld_errno != LDAP_SUCCESS) {
+               return ld->ld_errno;
        }
 
        /*
@@ -950,7 +1149,7 @@ ldap_pvt_tls_start ( LDAP *ld, Sockbuf *sb, void *ctx_arg )
                ssf = ldap_pvt_tls_get_strength( ssl );
                authid = ldap_pvt_tls_get_peer( ssl );
 
-               (void) ldap_int_sasl_external( ld, authid, ssf );
+               (void) ldap_int_sasl_external( ld, conn, authid, ssf );
        }
 
        return LDAP_SUCCESS;
@@ -1071,6 +1270,7 @@ tls_seed_PRNG( const char *randfile )
 {
 #ifndef URANDOM_DEVICE
        /* no /dev/urandom (or equiv) */
+       long total=0;
        char buffer[MAXPATHLEN];
 
        if (randfile == NULL) {
@@ -1093,7 +1293,7 @@ tls_seed_PRNG( const char *randfile )
                return -1;
        }
 
-       RAND_load_file(randfile, -1);
+       total = RAND_load_file(randfile, -1);
 
        if (RAND_status() == 0) {
                Debug( LDAP_DEBUG_ANY,
@@ -1101,6 +1301,12 @@ tls_seed_PRNG( const char *randfile )
                        0, 0, 0);
                return -1;
        }
+
+       /* assume if there was enough bits to seed that it's okay
+        * to write derived bits to the file
+        */
+       RAND_write_file(randfile);
+
 #endif
 
        return 0;
@@ -1120,14 +1326,15 @@ ldap_start_tls_s ( LDAP *ld,
        LDAPControl **serverctrls,
        LDAPControl **clientctrls )
 {
-#ifdef HAVE_TLS
        int rc;
+
+#ifdef HAVE_TLS
        char *rspoid = NULL;
        struct berval *rspdata = NULL;
 
-       /* XXYYZ: this initiates operaton only on default connection! */
+       /* XXYYZ: this initiates operation only on default connection! */
 
-       if ( ldap_pvt_tls_inplace( ld->ld_sb ) != 0 ) {
+       if ( ld->ld_sb != NULL && ldap_pvt_tls_inplace( ld->ld_sb ) != 0 ) {
                return LDAP_LOCAL_ERROR;
        }
 
@@ -1145,10 +1352,10 @@ ldap_start_tls_s ( LDAP *ld,
                ber_bvfree( rspdata );
        }
 
-       rc = ldap_pvt_tls_start( ld, ld->ld_sb, ld->ld_options.ldo_tls_ctx );
-       return rc;
+       rc = ldap_int_tls_start( ld, ld->ld_defconn, NULL );
 #else
-       return LDAP_NOT_SUPPORTED;
+       rc = LDAP_NOT_SUPPORTED;
 #endif
+       return rc;
 }