* 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
#include <vlc_pgpkey.h>
#include <vlc_stream.h>
#include <vlc_interface.h>
+#include <vlc_gcrypt.h>
/*****************************************************************************
* 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 );
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;
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 )
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;
}
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;
}
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;
p_update->release.psz_desc = NULL;
p_update->p_pkey = NULL;
+ vlc_gcrypt_init();
return 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;
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 );
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;
}
{
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;
* \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 );
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 );
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;
}
/**
}
-/*
+/**
* Struct to launch the download in a thread
*/
typedef struct
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" );
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 )
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 );
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 );
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 );
{
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."),
{
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 );
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 );
!= 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 );
free( p_buffer );
free( psz_size );
- vlc_object_destroy( p_udt );
+ vlc_object_release( p_udt );
}
#endif