]> git.sesse.net Git - vlc/blobdiff - modules/misc/gnutls.c
Remove redumdant parameter to vlc_global
[vlc] / modules / misc / gnutls.c
index f0887490f235bc4a0f2b8f68a2fadd904ce14589..fd368b3acc4587f925b6dee7def4a453b59e8843 100644 (file)
 /*****************************************************************************
  * Preamble
  *****************************************************************************/
-#define _GNU_SOURCE
+
+#include <vlc/vlc.h>
 #include <stdlib.h>
 #include <errno.h>
 #include <time.h>
-#include <vlc/vlc.h>
 
 #include <sys/types.h>
 #include <errno.h>
@@ -44,7 +44,7 @@
 
 
 #include "vlc_tls.h"
-#include "charset.h"
+#include <vlc_charset.h>
 
 #include <gcrypt.h>
 #include <gnutls/gnutls.h>
@@ -150,37 +150,40 @@ typedef struct tls_client_sys_t
 } tls_client_sys_t;
 
 
-static int
-_get_Int( vlc_object_t *p_this, const char *var )
+static int gnutls_Error (vlc_object_t *obj, int val)
 {
-    vlc_value_t value;
-
-    if( var_Get( p_this, var, &value ) != VLC_SUCCESS )
+    switch (val)
     {
-        var_Create( p_this, var, VLC_VAR_INTEGER | VLC_VAR_DOINHERIT );
-        var_Get( p_this, var, &value );
-    }
-
-    return value.i_int;
-}
+        case GNUTLS_E_AGAIN:
+#if ! defined(WIN32)
+            errno = EAGAIN;
+            break;
+#endif
+            /* WinSock does not return EAGAIN, return EINTR instead */
 
-static int
-_get_Bool( vlc_object_t *p_this, const char *var )
-{
-    vlc_value_t value;
+        case GNUTLS_E_INTERRUPTED:
+#if defined(WIN32)
+            WSASetLastError(WSAEINTR);
+#else
+            errno = EINTR;
+#endif
+            break;
 
-    if( var_Get( p_this, var, &value ) != VLC_SUCCESS )
-    {
-        var_Create( p_this, var, VLC_VAR_BOOL | VLC_VAR_DOINHERIT );
-        var_Get( p_this, var, &value );
+        default:
+            msg_Err (obj, "%s", gnutls_strerror (val));
+#ifdef DEBUG
+            if (!gnutls_error_is_fatal (val))
+                msg_Err (obj, "Error above should be handled");
+#endif
+#if defined(WIN32)
+            WSASetLastError(WSAECONNRESET);
+#else
+            errno = ECONNRESET;
+#endif
     }
-
-    return value.b_bool;
+    return -1;
 }
 
-#define get_Int( a, b ) _get_Int( (vlc_object_t *)(a), (b) )
-#define get_Bool( a, b ) _get_Bool( (vlc_object_t *)(a), (b) )
-
 
 /**
  * Sends data through a TLS session.
@@ -194,8 +197,7 @@ gnutls_Send( void *p_session, const void *buf, int i_length )
     p_sys = (tls_session_sys_t *)(((tls_session_t *)p_session)->p_sys);
 
     val = gnutls_record_send( p_sys->session, buf, i_length );
-    /* TODO: handle fatal error */
-    return val < 0 ? -1 : val;
+    return (val < 0) ? gnutls_Error ((vlc_object_t *)p_session, val) : val;
 }
 
 
@@ -211,8 +213,7 @@ gnutls_Recv( void *p_session, void *buf, int i_length )
     p_sys = (tls_session_sys_t *)(((tls_session_t *)p_session)->p_sys);
 
     val = gnutls_record_recv( p_sys->session, buf, i_length );
-    /* TODO: handle fatal error */
-    return val < 0 ? -1 : val;
+    return (val < 0) ? gnutls_Error ((vlc_object_t *)p_session, val) : val;
 }
 
 
@@ -232,7 +233,6 @@ gnutls_ContinueHandshake( tls_session_t *p_session)
 
     p_sys = (tls_session_sys_t *)(p_session->p_sys);
 
-     /* TODO: handle fatal error */
 #ifdef WIN32
     WSASetLastError( 0 );
 #endif
@@ -245,7 +245,7 @@ gnutls_ContinueHandshake( tls_session_t *p_session)
 #ifdef WIN32
         msg_Dbg( p_session, "Winsock error %d", WSAGetLastError( ) );
 #endif
-        msg_Err( p_session, "TLS handshake failed: %s",
+        msg_Err( p_session, "TLS handshake error: %s",
                  gnutls_strerror( val ) );
         p_session->pf_close( p_session );
         return -1;
@@ -255,52 +255,6 @@ gnutls_ContinueHandshake( tls_session_t *p_session)
     return 0;
 }
 
-static int
-gnutls_VerifyHostname( vlc_object_t *p_this, gnutls_session session,
-                       const char *psz_hostname )
-{
-    const gnutls_datum *p_data;
-    gnutls_x509_crt cert;
-    unsigned status;
-    int val;
-
-    /* certificate (host)name verification */
-    p_data = gnutls_certificate_get_peers( session, &status );
-    if( p_data == NULL )
-    {
-        msg_Err( p_this, "TLS peer certificate not available" );
-        return -1;
-    }
-
-    val = gnutls_x509_crt_init( &cert );
-    if( val )
-    {
-        msg_Err( p_this, "x509 fatal error: %s", gnutls_strerror( val ) );
-        return -1;
-    }
-
-    val = gnutls_x509_crt_import( cert, p_data, GNUTLS_X509_FMT_DER );
-    if( val )
-    {
-        msg_Err( p_this, "x509 certificate import error: %s",
-                gnutls_strerror( val ) );
-        gnutls_x509_crt_deinit( cert );
-        return -1;
-    }
-
-    if( gnutls_x509_crt_check_hostname( cert, psz_hostname ) == 0 )
-    {
-        msg_Err( p_this, "x509 certificate does not match \"%s\"",
-                psz_hostname );
-        gnutls_x509_crt_deinit( cert );
-        return -1;
-    }
-
-    gnutls_x509_crt_deinit( cert );
-    msg_Dbg( p_this, "x509 hostname matches %s", psz_hostname );
-    return 0;
-}
-
 
 typedef struct
 {
@@ -351,7 +305,7 @@ gnutls_HandshakeAndValidate( tls_session_t *session )
         {
             if( status & e->flag )
             {
-                msg_Err( session, e->msg );
+                msg_Err( session, "%s", e->msg );
                 status &= ~e->flag;
             }
         }
@@ -385,7 +339,6 @@ gnutls_HandshakeAndValidate( tls_session_t *session )
     {
         msg_Err( session, "Certificate import error: %s",
                  gnutls_strerror( val ) );
-        gnutls_x509_crt_deinit( cert );
         goto crt_error;
     }
 
@@ -448,7 +401,7 @@ gnutls_BeginHandshake( tls_session_t *p_session, int fd,
     {
         gnutls_server_name_set( p_sys->session, GNUTLS_NAME_DNS, psz_hostname,
                                 strlen( psz_hostname ) );
-        if( get_Bool( p_session, "tls-check-hostname" ) )
+        if (var_CreateGetBool (p_session, "tls-check-hostname"))
         {
             p_sys->psz_hostname = strdup( psz_hostname );
             if( p_sys->psz_hostname == NULL )
@@ -486,6 +439,111 @@ gnutls_SessionClose( tls_session_t *p_session )
     free( p_sys );
 }
 
+
+typedef int (*tls_prio_func) (gnutls_session_t, const int *);
+
+static int
+gnutls_SetPriority (vlc_object_t *restrict obj, const char *restrict name,
+                    tls_prio_func func, gnutls_session_t session,
+                    const int *restrict values)
+{
+    int val = func (session, values);
+    if (val < 0)
+    {
+        msg_Err (obj, "cannot set %s priorities: %s", name,
+                 gnutls_strerror (val));
+        return VLC_EGENERIC;
+    }
+    return VLC_SUCCESS;
+}
+
+
+static int
+gnutls_SessionPrioritize (vlc_object_t *obj, gnutls_session_t session)
+{
+    /* Note that ordering matters (on the client side) */
+    static const int protos[] =
+    {
+        GNUTLS_TLS1_1,
+        GNUTLS_TLS1_0,
+        GNUTLS_SSL3,
+        0
+    };
+    static const int comps[] =
+    {
+        GNUTLS_COMP_DEFLATE,
+        GNUTLS_COMP_NULL,
+        0
+    };
+    static const int macs[] =
+    {
+        GNUTLS_MAC_SHA1,
+        GNUTLS_MAC_RMD160, // RIPEMD
+        GNUTLS_MAC_MD5,
+        //GNUTLS_MAC_MD2,
+        //GNUTLS_MAC_NULL,
+        0
+    };
+    static const int ciphers[] =
+    {
+        GNUTLS_CIPHER_AES_256_CBC,
+        GNUTLS_CIPHER_AES_128_CBC,
+        GNUTLS_CIPHER_3DES_CBC,
+        GNUTLS_CIPHER_ARCFOUR_128,
+        //GNUTLS_CIPHER_DES_CBC,
+        //GNUTLS_CIPHER_ARCFOUR_40,
+        //GNUTLS_CIPHER_RC2_40_CBC,
+        //GNUTLS_CIPHER_NULL,
+        0
+    };
+    static const int kx[] =
+    {
+        GNUTLS_KX_DHE_RSA,
+        GNUTLS_KX_DHE_DSS,
+        GNUTLS_KX_RSA,
+        //GNUTLS_KX_RSA_EXPORT,
+        //GNUTLS_KX_DHE_PSK, TODO
+        //GNUTLS_KX_PSK,     TODO
+        //GNUTLS_KX_SRP_RSA, TODO
+        //GNUTLS_KX_SRP_DSS, TODO
+        //GNUTLS_KX_SRP,     TODO
+        //GNUTLS_KX_ANON_DH,
+        0
+    };
+    static const int cert_types[] =
+    {
+        GNUTLS_CRT_X509,
+        //GNUTLS_CRT_OPENPGP, TODO
+        0
+    };
+
+    int val = gnutls_set_default_priority (session);
+    if (val < 0)
+    {
+        msg_Err (obj, "cannot set default TLS priorities: %s",
+                 gnutls_strerror (val));
+        return VLC_EGENERIC;
+    }
+
+    if (gnutls_SetPriority (obj, "protocols",
+                            gnutls_protocol_set_priority, session, protos)
+     || gnutls_SetPriority (obj, "compression algorithms",
+                            gnutls_compression_set_priority, session, comps)
+     || gnutls_SetPriority (obj, "MAC algorithms",
+                            gnutls_mac_set_priority, session, macs)
+     || gnutls_SetPriority (obj, "ciphers",
+                            gnutls_cipher_set_priority, session, ciphers)
+     || gnutls_SetPriority (obj, "key exchange algorithms",
+                            gnutls_kx_set_priority, session, kx)
+     || gnutls_SetPriority (obj, "certificate types",
+                            gnutls_certificate_type_set_priority, session,
+                            cert_types))
+        return VLC_EGENERIC;
+
+    return VLC_SUCCESS;
+}
+
+
 static void
 gnutls_ClientDelete( tls_session_t *p_session )
 {
@@ -512,7 +570,6 @@ gnutls_Addx509Directory( vlc_object_t *p_this,
                          vlc_bool_t b_priv )
 {
     DIR* dir;
-    const char *psz_dirent;
 
     if( *psz_dirname == '\0' )
         psz_dirname = ".";
@@ -545,23 +602,20 @@ gnutls_Addx509Directory( vlc_object_t *p_this,
     }
 #endif
 
-    while( ( psz_dirent = utf8_readdir( dir ) ) != NULL )
+    for (;;)
     {
-        char *psz_filename;
-        int check;
+        char *ent = utf8_readdir (dir);
+        if (ent == NULL)
+            break;
 
-        if( ( strcmp( ".", psz_dirent ) == 0 )
-         || ( strcmp( "..", psz_dirent ) == 0 ) )
+        if ((strcmp (ent, ".") == 0) || (strcmp (ent, "..") == 0))
             continue;
 
-        check = asprintf( &psz_filename, "%s/%s", psz_dirname,
-                              psz_dirent );
-        LocaleFree( psz_dirent );
-        if( check == -1 )
-            continue;
+        char path[strlen (psz_dirname) + strlen (ent) + 2];
+        sprintf (path, "%s"DIR_SEP"%s", psz_dirname, ent);
+        free (ent);
 
-        gnutls_Addx509File( p_this, cred, psz_filename, b_priv );
-        free( psz_filename );
+        gnutls_Addx509File( p_this, cred, path, b_priv );
     }
 
     closedir( dir );
@@ -625,11 +679,6 @@ gnutls_ClientCreate( tls_t *p_tls )
     tls_session_t *p_session = NULL;
     tls_client_sys_t *p_sys = NULL;
     int i_val;
-    const int cert_type_priority[3] =
-    {
-        GNUTLS_CRT_X509,
-        0
-    };
 
     p_sys = (tls_client_sys_t *)malloc( sizeof(struct tls_client_sys_t) );
     if( p_sys == NULL )
@@ -654,6 +703,13 @@ gnutls_ClientCreate( tls_t *p_tls )
 
     vlc_object_attach( p_session, p_tls );
 
+    const char *homedir = p_tls->p_libvlc->psz_homedir,
+               *datadir = config_GetDataDir ();
+    size_t l1 = strlen (homedir), l2 = strlen (datadir);
+    char path[((l1 > l2) ? l1 : l2) + sizeof ("/"CONFIG_DIR"/ssl/private")];
+    //                              > sizeof ("/"CONFIG_DIR"/ssl/certs")
+    //                              > sizeof ("/ca-certificates.crt")
+
     i_val = gnutls_certificate_allocate_credentials( &p_sys->x509_cred );
     if( i_val != 0 )
     {
@@ -662,45 +718,23 @@ gnutls_ClientCreate( tls_t *p_tls )
         goto error;
     }
 
-    if( get_Bool( p_tls, "tls-check-cert" ) )
+    if (var_CreateGetBool (p_tls, "tls-check-cert"))
     {
-        char *psz_path;
+        sprintf (path, "%s/"CONFIG_DIR"/ssl/certs", homedir);
+        gnutls_Addx509Directory ((vlc_object_t *)p_session,
+                                  p_sys->x509_cred, path, VLC_FALSE);
 
-        if( asprintf( &psz_path, "%s/"CONFIG_DIR"/ssl/certs",
-                      p_tls->p_libvlc->psz_homedir ) != -1 )
-        {
-            gnutls_Addx509Directory( (vlc_object_t *)p_session,
-                                     p_sys->x509_cred, psz_path, VLC_FALSE );
-            free( psz_path );
-        }
-
-        if( asprintf( &psz_path, "%s/ca-certificates.crt",
-            config_GetDataDir ( (vlc_object_t *)p_session) ) != -1 )
-        {
-            gnutls_Addx509File( (vlc_object_t *)p_session,
-                                p_sys->x509_cred, psz_path, VLC_FALSE );
-            free( psz_path );
-        }
+        sprintf (path, "%s/ca-certificates.crt", datadir);
+        gnutls_Addx509File ((vlc_object_t *)p_session,
+                            p_sys->x509_cred, path, VLC_FALSE);
         p_session->pf_handshake2 = gnutls_HandshakeAndValidate;
     }
     else
         p_session->pf_handshake2 = gnutls_ContinueHandshake;
 
-    {
-        char *psz_path;
-
-        if( asprintf( &psz_path, "%s/"CONFIG_DIR"/ssl/private",
-                      p_tls->p_libvlc->psz_homedir ) == -1 )
-        {
-            gnutls_certificate_free_credentials( p_sys->x509_cred );
-            goto error;
-        }
-
-        gnutls_Addx509Directory( (vlc_object_t *)p_session, p_sys->x509_cred,
-                                 psz_path, VLC_TRUE );
-
-        free( psz_path );
-    }
+    sprintf (path, "%s/"CONFIG_DIR"/ssl/private", homedir);
+    gnutls_Addx509Directory ((vlc_object_t *)p_session, p_sys->x509_cred,
+                             path, VLC_TRUE);
 
     i_val = gnutls_init( &p_sys->session.session, GNUTLS_CLIENT );
     if( i_val != 0 )
@@ -711,26 +745,9 @@ gnutls_ClientCreate( tls_t *p_tls )
         goto error;
     }
 
-    i_val = gnutls_set_default_priority( p_sys->session.session );
-    if( i_val < 0 )
-    {
-        msg_Err( p_tls, "cannot set ciphers priorities: %s",
-                 gnutls_strerror( i_val ) );
-        gnutls_deinit( p_sys->session.session );
-        gnutls_certificate_free_credentials( p_sys->x509_cred );
-        goto error;
-    }
-
-    i_val = gnutls_certificate_type_set_priority( p_sys->session.session,
-                                                  cert_type_priority );
-    if( i_val < 0 )
-    {
-        msg_Err( p_tls, "cannot set certificate type priorities: %s",
-                 gnutls_strerror( i_val ) );
-        gnutls_deinit( p_sys->session.session );
-        gnutls_certificate_free_credentials( p_sys->x509_cred );
-        goto error;
-    }
+    if (gnutls_SessionPrioritize (VLC_OBJECT (p_session),
+                                  p_sys->session.session))
+        goto s_error;
 
     i_val = gnutls_credentials_set( p_sys->session.session,
                                     GNUTLS_CRD_CERTIFICATE,
@@ -739,13 +756,15 @@ gnutls_ClientCreate( tls_t *p_tls )
     {
         msg_Err( p_tls, "cannot set TLS session credentials: %s",
                  gnutls_strerror( i_val ) );
-        gnutls_deinit( p_sys->session.session );
-        gnutls_certificate_free_credentials( p_sys->x509_cred );
-        goto error;
+        goto s_error;
     }
 
     return p_session;
 
+s_error:
+    gnutls_deinit( p_sys->session.session );
+    gnutls_certificate_free_credentials( p_sys->x509_cred );
+
 error:
     vlc_object_detach( p_session );
     vlc_object_destroy( p_session );
@@ -898,11 +917,8 @@ gnutls_ServerSessionPrepare( tls_server_t *p_server )
 
     ((tls_session_sys_t *)p_session->p_sys)->session = session;
 
-    i_val = gnutls_set_default_priority( session );
-    if( i_val < 0 )
+    if (gnutls_SessionPrioritize (VLC_OBJECT (p_session), session))
     {
-        msg_Err( p_server, "cannot set ciphers priorities: %s",
-                 gnutls_strerror( i_val ) );
         gnutls_deinit( session );
         goto error;
     }
@@ -920,11 +936,12 @@ gnutls_ServerSessionPrepare( tls_server_t *p_server )
     if( p_session->pf_handshake2 == gnutls_HandshakeAndValidate )
         gnutls_certificate_server_set_request( session, GNUTLS_CERT_REQUIRE );
 
-    gnutls_dh_set_prime_bits( session, get_Int( p_server, "gnutls-dh-bits" ) );
+    i_val = config_GetInt (p_server, "gnutls-dh-bits");
+    gnutls_dh_set_prime_bits (session, i_val);
 
     /* Session resumption support */
-    gnutls_db_set_cache_expiration( session, get_Int( p_server,
-                                    "gnutls-cache-expiration" ) );
+    i_val = config_GetInt (p_server, "gnutls-cache-expiration");
+    gnutls_db_set_cache_expiration (session, i_val);
     gnutls_db_set_retrieve_function( session, cb_fetch );
     gnutls_db_set_remove_function( session, cb_delete );
     gnutls_db_set_store_function( session, cb_store );
@@ -1047,7 +1064,7 @@ gnutls_ServerCreate( tls_t *p_tls, const char *psz_cert_path,
     if( p_sys == NULL )
         return NULL;
 
-    p_sys->i_cache_size = get_Int( p_tls, "gnutls-cache-size" );
+    p_sys->i_cache_size = config_GetInt (p_tls, "gnutls-cache-size");
     p_sys->p_cache = (struct saved_session_t *)calloc( p_sys->i_cache_size,
                                            sizeof( struct saved_session_t ) );
     if( p_sys->p_cache == NULL )
@@ -1111,7 +1128,7 @@ gnutls_ServerCreate( tls_t *p_tls, const char *psz_cert_path,
     {
         msg_Dbg( p_server, "computing Diffie Hellman ciphers parameters" );
         val = gnutls_dh_params_generate2( p_sys->dh_params,
-                                          get_Int( p_tls, "gnutls-dh-bits" ) );
+                                          config_GetInt( p_tls, "gnutls-dh-bits" ) );
     }
     if( val < 0 )
     {
@@ -1197,22 +1214,19 @@ static struct gcry_thread_cbs gcry_threads_vlc =
 /*****************************************************************************
  * Module initialization
  *****************************************************************************/
+static unsigned refs = 0;
+
 static int
 Open( vlc_object_t *p_this )
 {
     tls_t *p_tls = (tls_t *)p_this;
+    vlc_mutex_t *lock;
 
-    vlc_value_t lock, count;
-
-    var_Create( p_this->p_libvlc_global, "gnutls_mutex", VLC_VAR_MUTEX );
-    var_Get( p_this->p_libvlc_global, "gnutls_mutex", &lock );
-    vlc_mutex_lock( lock.p_address );
+    lock = var_GetGlobalMutex( "gnutls_mutex" );
+    vlc_mutex_lock( lock );
 
     /* Initialize GnuTLS only once */
-    var_Create( p_this->p_libvlc_global, "gnutls_count", VLC_VAR_INTEGER );
-    var_Get( p_this->p_libvlc_global, "gnutls_count", &count);
-
-    if( count.i_int == 0)
+    if( refs == 0 )
     {
 #ifdef NEED_THREAD_CONTEXT
         __p_gcry_data = VLC_OBJECT( p_this->p_libvlc );
@@ -1222,7 +1236,7 @@ Open( vlc_object_t *p_this )
         if( gnutls_global_init( ) )
         {
             msg_Warn( p_this, "cannot initialize GnuTLS" );
-            vlc_mutex_unlock( lock.p_address );
+            vlc_mutex_unlock( lock );
             return VLC_EGENERIC;
         }
 
@@ -1230,16 +1244,15 @@ Open( vlc_object_t *p_this )
         if( psz_version == NULL )
         {
             gnutls_global_deinit( );
-            vlc_mutex_unlock( lock.p_address );
+            vlc_mutex_unlock( lock );
             msg_Err( p_this, "unsupported GnuTLS version" );
             return VLC_EGENERIC;
         }
         msg_Dbg( p_this, "GnuTLS v%s initialized", psz_version );
     }
 
-    count.i_int++;
-    var_Set( p_this->p_libvlc_global, "gnutls_count", count);
-    vlc_mutex_unlock( lock.p_address );
+    refs++;
+    vlc_mutex_unlock( lock );
 
     p_tls->pf_server_create = gnutls_ServerCreate;
     p_tls->pf_client_create = gnutls_ClientCreate;
@@ -1256,22 +1269,16 @@ Close( vlc_object_t *p_this )
     /*tls_t *p_tls = (tls_t *)p_this;
     tls_sys_t *p_sys = (tls_sys_t *)(p_this->p_sys);*/
 
-    vlc_value_t lock, count;
-
-    var_Create( p_this->p_libvlc_global, "gnutls_mutex", VLC_VAR_MUTEX );
-    var_Get( p_this->p_libvlc_global, "gnutls_mutex", &lock );
-    vlc_mutex_lock( lock.p_address );
+    vlc_mutex_t *lock;
 
-    var_Create( p_this->p_libvlc_global, "gnutls_count", VLC_VAR_INTEGER );
-    var_Get( p_this->p_libvlc_global, "gnutls_count", &count);
-    count.i_int--;
-    var_Set( p_this->p_libvlc_global, "gnutls_count", count);
+    lock = var_GetGlobalMutex( "gnutls_mutex" );
+    vlc_mutex_lock( lock );
 
-    if( count.i_int == 0 )
+    if( --refs == 0 )
     {
         gnutls_global_deinit( );
         msg_Dbg( p_this, "GnuTLS deinitialized" );
     }
 
-    vlc_mutex_unlock( lock.p_address );
+    vlc_mutex_unlock( lock );
 }