ITS#10326 mbedtls: always call mbedtls_ssl_set_hostname()

Even if hostname is NULL, the library requires this be called once.
If non-NULL, mbedtls may do a hostname check which is redundant
since libldap does its own check.

diff --git a/libraries/libldap/tls_mt.c b/libraries/libldap/tls_mt.c
index 908d84058cf423d81799774398dbfcc550b37a1e..a92c300aded12a27ceabf5590ebbd1852a1a20db 100644 (file)
--- a/libraries/libldap/tls_mt.c
+++ b/libraries/libldap/tls_mt.c
@@ -380,11 +380,9 @@ static int
 tlsmt_session_connect( LDAP *ld, tls_session *sess, const char *name_in )
 {
        tlsmt_session *s = (tlsmt_session *)sess;
-       if (name_in) {
-               int ret = mbedtls_ssl_set_hostname( &(s->ssl_ctx), name_in );
-               if ( ret != 0 ) {
-                       return ret;
-               }
+       int ret = mbedtls_ssl_set_hostname( &(s->ssl_ctx), name_in );
+       if ( ret != 0 ) {
+               return ret;
        }
 
        return tlsmt_session_accept(sess);