]> git.sesse.net Git - vlc/blobdiff - src/misc/update.c
Remove p_playlist from p_libvlc
[vlc] / src / misc / update.c
index e595d24807d794789d8d5529d857ce8485397e07..cd1cd408bfb5ea1ab4f6bb4b4887ebc0a93190c9 100644 (file)
@@ -22,7 +22,9 @@
  * along with this program; if not, write to the Free Software
  * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA  02111, USA.
  *****************************************************************************/
-
+/*
+ * XXX: should use v4 signatures for binary files (already used for public key)
+ */
 /**
  *   \file
  *   This file contains functions related to VLC update management
@@ -46,6 +48,7 @@
 #include <vlc_pgpkey.h>
 #include <vlc_stream.h>
 #include <vlc_interface.h>
+#include <vlc_gcrypt.h>
 
 
 /*****************************************************************************
@@ -85,9 +88,7 @@
  * Local Prototypes
  *****************************************************************************/
 static void EmptyRelease( update_t *p_update );
-static vlc_bool_t GetUpdateFile( update_t *p_update );
-static int CompareReleases( const struct update_release_t *p1,
-                            const struct update_release_t *p2 );
+static bool GetUpdateFile( update_t *p_update );
 static char * size_str( long int l_size );
 
 
@@ -414,7 +415,12 @@ static int download_signature(  vlc_object_t *p_this,
     int i_bytes = pgp_unarmor( p_buf, i_size, (uint8_t*)p_sig, 65 );
     free( p_buf );
 
-    if( i_bytes > 65 )
+    if( i_bytes == 0 )
+    {
+        msg_Dbg( p_this, "Unarmoring failed" );
+        return VLC_EGENERIC;
+    }
+    else if( i_bytes > 65 )
     {
         msg_Dbg( p_this, "Signature is too big: %d bytes", i_bytes );
         return VLC_EGENERIC;
@@ -424,7 +430,8 @@ static int download_signature(  vlc_object_t *p_this,
         int i_r_len = mpi_len( p_sig->r );
         if( i_r_len > 20 )
         {
-            msg_Dbg( p_this, "Signature invalid" );
+            msg_Dbg( p_this, "Invalid signature, r number too big: %d bytes",
+                                i_r_len );
             return VLC_EGENERIC;
         }
         else if( i_r_len < 20 )
@@ -667,8 +674,10 @@ static uint8_t *hash_sha1_from_file( const char *psz_file,
     fclose( f );
     gcry_md_final( hd );
 
-    uint8_t *p_hash = (uint8_t*) gcry_md_read( hd, GCRY_MD_SHA1);
-    p_hash = strdup( p_hash );
+    uint8_t *p_tmp = (uint8_t*) gcry_md_read( hd, GCRY_MD_SHA1);
+    uint8_t *p_hash = malloc( 20 );
+    if( p_hash )
+        memcpy( p_hash, p_tmp, 20 );
     gcry_md_close( hd );
     return p_hash;
 }
@@ -801,16 +810,19 @@ static uint8_t *key_sign_hash( public_key_t *p_pkey )
 
     gcry_md_final( hd );
 
-    uint8_t *p_hash = gcry_md_read( hd, GCRY_MD_SHA1);
+    uint8_t *p_tmp = gcry_md_read( hd, GCRY_MD_SHA1);
 
-    if( p_hash[0] != p_pkey->sig.hash_verification[0] ||
-        p_hash[1] != p_pkey->sig.hash_verification[1] )
+    if( !p_tmp ||
+        p_tmp[0] != p_pkey->sig.hash_verification[0] ||
+        p_tmp[1] != p_pkey->sig.hash_verification[1] )
     {
         gcry_md_close( hd );
         return NULL;
     }
 
-    p_hash = strdup( p_hash );
+    uint8_t *p_hash = malloc( 20 );
+    if( p_hash )
+        memcpy( p_hash, p_tmp, 20 );
     gcry_md_close( hd );
     return p_hash;
 }
@@ -834,7 +846,7 @@ update_t *__update_New( vlc_object_t *p_this )
     p_update = (update_t *)malloc( sizeof( update_t ) );
     if( !p_update ) return NULL;
 
-    vlc_mutex_init( p_this, &p_update->lock );
+    vlc_mutex_init( &p_update->lock );
 
     p_update->p_libvlc = p_this->p_libvlc;
 
@@ -842,6 +854,7 @@ update_t *__update_New( vlc_object_t *p_this )
     p_update->release.psz_desc = NULL;
 
     p_update->p_pkey = NULL;
+    vlc_gcrypt_init();
 
     return p_update;
 }
@@ -883,12 +896,12 @@ static void EmptyRelease( update_t *p_update )
 
 /**
  * Get the update file and parse it
- * *p_update has to be locked when calling this function
+ * p_update has to be locked when calling this function
  *
  * \param p_update pointer to update struct
- * \return VLC_TRUE if the update is valid and authenticated
+ * \return true if the update is valid and authenticated
  */
-static vlc_bool_t GetUpdateFile( update_t *p_update )
+static bool GetUpdateFile( update_t *p_update )
 {
     stream_t *p_stream = NULL;
     int i_major = 0;
@@ -1021,7 +1034,7 @@ static vlc_bool_t GetUpdateFile( update_t *p_update )
 
     gcry_md_hd_t hd;
     if( gcry_md_open( &hd, GCRY_MD_SHA1, 0 ) )
-        goto error;
+        goto error_hd;
 
     gcry_md_write( hd, psz_version_line, strlen( psz_version_line ) );
     FREENULL( psz_version_line );
@@ -1050,28 +1063,29 @@ static vlc_bool_t GetUpdateFile( update_t *p_update )
         p_hash[1] != sign.hash_verification[1] )
     {
         msg_Warn( p_update->p_libvlc, "Bad SHA1 hash for status file" );
-        goto error;
+        goto error_hd;
     }
 
     if( verify_signature( sign.r, sign.s, &p_update->p_pkey->key, p_hash )
             != VLC_SUCCESS )
     {
         msg_Err( p_update->p_libvlc, "BAD SIGNATURE for status file" );
-        goto error;
+        goto error_hd;
     }
     else
     {
         msg_Info( p_update->p_libvlc, "Status file authenticated" );
         gcry_md_close( hd );
-        return VLC_TRUE;
+        return true;
     }
 
-error:
+error_hd:
     gcry_md_close( hd );
+error:
     if( p_stream )
         stream_Delete( p_stream );
     free( psz_version_line );
-    return VLC_FALSE;
+    return false;
 }
 
 
@@ -1082,7 +1096,7 @@ typedef struct
 {
     VLC_COMMON_MEMBERS
     update_t *p_update;
-    void (*pf_callback)( void *, vlc_bool_t );
+    void (*pf_callback)( void *, bool );
     void *p_data;
 } update_check_thread_t;
 
@@ -1096,7 +1110,7 @@ void update_CheckReal( update_check_thread_t *p_uct );
  * \param p_data pointer to some datas to give to the callback
  * \returns nothing
  */
-void update_Check( update_t *p_update, void (*pf_callback)( void*, vlc_bool_t ), void *p_data )
+void update_Check( update_t *p_update, void (*pf_callback)( void*, bool ), void *p_data )
 {
     assert( p_update );
 
@@ -1113,12 +1127,12 @@ void update_Check( update_t *p_update, void (*pf_callback)( void*, vlc_bool_t ),
     p_uct->p_data = p_data;
 
     vlc_thread_create( p_uct, "check for update", update_CheckReal,
-                       VLC_THREAD_PRIORITY_LOW, VLC_FALSE );
+                       VLC_THREAD_PRIORITY_LOW, false );
 }
 
 void update_CheckReal( update_check_thread_t *p_uct )
 {
-    vlc_bool_t b_ret;
+    bool b_ret;
     vlc_mutex_lock( &p_uct->p_update->lock );
 
     EmptyRelease( p_uct->p_update );
@@ -1128,51 +1142,23 @@ void update_CheckReal( update_check_thread_t *p_uct )
     if( p_uct->pf_callback )
         (p_uct->pf_callback)( p_uct->p_data, b_ret );
 
-    vlc_object_destroy( p_uct );
-}
-
-/**
- * Compare two release numbers
- *
- * \param p1 first release
- * \param p2 second release
- * \return UpdateReleaseStatus(Older|Equal|Newer)
- */
-static int CompareReleases( const struct update_release_t *p1,
-                            const struct update_release_t *p2 )
-{
-    int32_t d;
-    d = ( p1->i_major << 24 ) + ( p1->i_minor << 16 ) + ( p1->i_revision << 8 )
-      - ( p2->i_major << 24 ) - ( p2->i_minor << 16 ) - ( p2->i_revision << 8 )
-      + ( p1->extra ) - ( p2->extra );
-
-    if( d < 0 )
-        return UpdateReleaseStatusOlder;
-    else if( d == 0 )
-        return UpdateReleaseStatusEqual;
-    else
-        return UpdateReleaseStatusNewer;
+    vlc_object_release( p_uct );
 }
 
 /**
  * Compare a given release's version number to the current VLC's one
  *
  * \param p_update structure
- * \return UpdateReleaseStatus(Older|Equal|Newer)
+ * \return true if we have to upgrade to the given version to be up to date
  */
-int update_CompareReleaseToCurrent( update_t *p_update )
+bool update_NeedUpgrade( update_t *p_update )
 {
     assert( p_update );
 
-    struct update_release_t c;
-
-    /* get the current version number */
-    c.i_major = *PACKAGE_VERSION_MAJOR - '0';
-    c.i_minor = *PACKAGE_VERSION_MINOR - '0';
-    c.i_revision = *PACKAGE_VERSION_REVISION - '0';
-    c.extra = *PACKAGE_VERSION_EXTRA;
-
-    return CompareReleases( &p_update->release, &c );
+    return  p_update->release.i_major < *PACKAGE_VERSION_MAJOR - '0' ||
+            p_update->release.i_minor < *PACKAGE_VERSION_MINOR - '0' ||
+            p_update->release.i_revision < *PACKAGE_VERSION_REVISION ||
+            p_update->release.extra < *PACKAGE_VERSION_EXTRA;
 }
 
 /**
@@ -1198,7 +1184,7 @@ static char *size_str( long int l_size )
 }
 
 
-/*
+/**
  * Struct to launch the download in a thread
  */
 typedef struct
@@ -1222,7 +1208,7 @@ void update_Download( update_t *p_update, char *psz_destdir )
     assert( p_update );
 
     update_download_thread_t *p_udt = vlc_object_create( p_update->p_libvlc,
-                                                      sizeof( update_download_thread_t ) );
+                                        sizeof( update_download_thread_t ) );
     if( !p_udt )
     {
         msg_Err( p_update->p_libvlc, "out of memory" );
@@ -1233,7 +1219,7 @@ void update_Download( update_t *p_update, char *psz_destdir )
     p_udt->psz_destdir = psz_destdir ? strdup( psz_destdir ) : NULL;
 
     vlc_thread_create( p_udt, "download update", update_DownloadReal,
-                       VLC_THREAD_PRIORITY_LOW, VLC_FALSE );
+                       VLC_THREAD_PRIORITY_LOW, false );
 }
 
 void update_DownloadReal( update_download_thread_t *p_udt )
@@ -1291,7 +1277,8 @@ void update_DownloadReal( update_download_thread_t *p_udt )
         goto end;
 
     psz_size = size_str( l_size );
-    if( asprintf( &psz_status, "%s\nDownloading... O.O/%s %.1f%% done",  p_update->release.psz_url, psz_size, 0.0 ) != -1 )
+    if( asprintf( &psz_status, "%s\nDownloading... O.O/%s %.1f%% done",
+        p_update->release.psz_url, psz_size, 0.0 ) != -1 )
     {
         i_progress = intf_UserProgress( p_udt, "Downloading ...", psz_status, 0.0, 0 );
         free( psz_status );
@@ -1310,8 +1297,9 @@ void update_DownloadReal( update_download_thread_t *p_udt )
         psz_downloaded = size_str( l_downloaded );
         f_progress = 100.0*(float)l_downloaded/(float)l_size;
 
-        if( asprintf( &psz_status, "%s\nDonwloading... %s/%s %.1f%% done", p_update->release.psz_url,
-                      psz_downloaded, psz_size, f_progress ) != -1 )
+        if( asprintf( &psz_status, "%s\nDonwloading... %s/%s %.1f%% done",
+                      p_update->release.psz_url, psz_downloaded, psz_size,
+                      f_progress ) != -1 )
         {
             intf_ProgressUpdate( p_udt, i_progress, psz_status, f_progress, 0 );
             free( psz_status );
@@ -1325,7 +1313,8 @@ void update_DownloadReal( update_download_thread_t *p_udt )
 
     if( !intf_ProgressIsCancelled( p_udt, i_progress ) )
     {
-        if( asprintf( &psz_status, "%s\nDone %s (100.0%%)", p_update->release.psz_url, psz_size ) != -1 )
+        if( asprintf( &psz_status, "%s\nDone %s (100.0%%)",
+            p_update->release.psz_url, psz_size ) != -1 )
         {
             intf_ProgressUpdate( p_udt, i_progress, psz_status, 100.0, 0 );
             free( psz_status );
@@ -1343,19 +1332,31 @@ void update_DownloadReal( update_download_thread_t *p_udt )
     {
         utf8_unlink( psz_destfile );
 
-        intf_UserFatal( p_udt, VLC_TRUE, _("File can not be verified"),
-            _("It was not possible to downlaod a cryptographic signature for "
+        intf_UserFatal( p_udt, true, _("File can not be verified"),
+            _("It was not possible to download a cryptographic signature for "
               "downloaded file \"%s\", and so VLC deleted it."),
             psz_destfile );
         msg_Err( p_udt, "Couldn't download signature of downloaded file" );
         goto end;
     }
 
+    if( memcmp( sign.issuer_longid, p_update->p_pkey->longid, 8 ) )
+    {
+        utf8_unlink( psz_destfile );
+        msg_Err( p_udt, "Invalid signature issuer" );
+        intf_UserFatal( p_udt, true, _("Invalid signature"),
+            _("The cryptographic signature for downloaded file \"%s\" was "
+              "invalid and couldn't be used to securely verify it, and so "
+              "VLC deleted it."),
+            psz_destfile );
+        goto end;
+    }
+
     if( sign.type != BINARY_SIGNATURE )
     {
         utf8_unlink( psz_destfile );
         msg_Err( p_udt, "Invalid signature type" );
-        intf_UserFatal( p_udt, VLC_TRUE, _("Invalid signature"),
+        intf_UserFatal( p_udt, true, _("Invalid signature"),
             _("The cryptographic signature for downloaded file \"%s\" was "
               "invalid and couldn't be used to securely verify it, and so "
               "VLC deleted it."),
@@ -1368,7 +1369,7 @@ void update_DownloadReal( update_download_thread_t *p_udt )
     {
         msg_Err( p_udt, "Unable to hash %s", psz_destfile );
         utf8_unlink( psz_destfile );
-        intf_UserFatal( p_udt, VLC_TRUE, _("File not verifiable"),
+        intf_UserFatal( p_udt, true, _("File not verifiable"),
             _("It was not possible to securely verify downloaded file \"%s\", "
               "and so VLC deleted it."),
             psz_destfile );
@@ -1380,7 +1381,7 @@ void update_DownloadReal( update_download_thread_t *p_udt )
         p_hash[1] != sign.hash_verification[1] )
     {
         utf8_unlink( psz_destfile );
-        intf_UserFatal( p_udt, VLC_TRUE, _("File corrupted"),
+        intf_UserFatal( p_udt, true, _("File corrupted"),
             _("Downloaded file \"%s\" was corrupted, and so VLC deleted it."),
              psz_destfile );
         msg_Err( p_udt, "Bad SHA1 hash for %s", psz_destfile );
@@ -1392,7 +1393,7 @@ void update_DownloadReal( update_download_thread_t *p_udt )
             != VLC_SUCCESS )
     {
         utf8_unlink( psz_destfile );
-        intf_UserFatal( p_udt, VLC_TRUE, _("File corrupted"),
+        intf_UserFatal( p_udt, true, _("File corrupted"),
             _("Downloaded file \"%s\" was corrupted, and so VLC deleted it."),
              psz_destfile );
         msg_Err( p_udt, "BAD SIGNATURE for %s", psz_destfile );
@@ -1413,7 +1414,7 @@ end:
     free( p_buffer );
     free( psz_size );
 
-    vlc_object_destroy( p_udt );
+    vlc_object_release( p_udt );
 }
 
 #endif