Changeset 5356

Timestamp:

Mar 24, 2008, 3:58:06 PM (14 years ago)

Author:

charles

Message:

(#814) require a valid announce URL when creating torrents

Location:

trunk

Files:

• gtk/makemeta-ui.c (modified) (1 diff)
• libtransmission/makemeta.c (modified) (6 diffs)
• libtransmission/makemeta.h (modified) (1 diff)
• libtransmission/metainfo.c (modified) (2 diffs)
• libtransmission/utils.c (modified) (2 diffs)
• libtransmission/utils.h (modified) (1 diff)

trunk/gtk/makemeta-ui.c

@@ -93 +93 @@
         if( ui->builder->failed )
         {
-            const char * reason = ui->builder->abortFlag
-                ? _("Torrent creation cancelled")
-                : _("Torrent creation failed");
-
+            char * reason = ui->builder->abortFlag
+                ? g_strdup( _( "Torrent creation cancelled" ) )
+                : g_strdup_printf( _( "Torrent creation failed: %s" ), ui->builder->error );
             gtk_progress_bar_set_text( p, reason );
             gtk_progress_bar_set_fraction( p, 0 );
+            g_free( reason );
         }
         else

trunk/libtransmission/makemeta.c

@@ -206 +206 @@
     fp = fopen( b->files[fileIndex].filename, "rb" );
     if( !fp ) {
-        tr_err( _( "Couldn't open \"%s\": %s" ), b->files[fileIndex].filename, tr_strerror( errno ) );
+        snprintf( b->error, sizeof( b->error ), _( "Couldn't open \"%s\": %s" ), b->files[fileIndex].filename, tr_strerror( errno ) );
+        tr_err( "%s", b->error );
         tr_free( ret );
         b->failed = 1;
@@ -235 +236 @@
                     fp = fopen( b->files[fileIndex].filename, "rb" );
                     if( !fp ) {
-                        tr_err( _( "Couldn't open \"%s\": %s" ), b->files[fileIndex].filename, tr_strerror( errno ) );
+                        snprintf( b->error, sizeof( b->error ),
+                                  _( "Couldn't open \"%s\": %s" ),
+                                  b->files[fileIndex].filename, tr_strerror( errno ) ); 
+                        tr_err( "%s", b->error );
                         tr_free( ret );
                         b->failed = 1;
@@ -250 +254 @@
 
         if( b->abortFlag ) {
+            snprintf( b->error, sizeof( b->error ), _( "Torrent creation cancelled" ) );
             b->failed = 1;
             break;
@@ -367 +372 @@
 }
 
-static void tr_realMakeMetaInfo ( tr_metainfo_builder * builder )
+static void
+tr_realMakeMetaInfo ( tr_metainfo_builder * builder )
 {
     int n = 5;
@@ -378 +384 @@
     val = tr_bencDictAdd( &top, "announce" );
     tr_bencInitStrDup( val, builder->announce );
+    if( tr_httpParseUrl( builder->announce, -1, NULL, NULL, NULL ) )
+    {
+        snprintf( builder->error, sizeof( builder->error ), _( "Invalid announce URL" ) );
+        tr_err( "%s", builder->error );
+        builder->failed = 1;
+    }
     
-    if( builder->comment && *builder->comment ) {
-        val = tr_bencDictAdd( &top, "comment" );
-        tr_bencInitStrDup( val, builder->comment );
-    }
-
-    val = tr_bencDictAdd( &top, "created by" );
-    tr_bencInitStrDup( val, TR_NAME "/" LONG_VERSION_STRING );
-
-    val = tr_bencDictAdd( &top, "creation date" );
-    tr_bencInitInt( val, time(0) );
-
-    val = tr_bencDictAdd( &top, "encoding" );
-    tr_bencInitStrDup( val, "UTF-8" );
-
-    val = tr_bencDictAdd( &top, "info" );
-    tr_bencInit( val, TYPE_DICT );
-    tr_bencDictReserve( val, 666 );
-    makeInfoDict( val, builder );
+    if( !builder->failed && !builder->abortFlag )
+    {
+        if( builder->comment && *builder->comment ) {
+            val = tr_bencDictAdd( &top, "comment" );
+            tr_bencInitStrDup( val, builder->comment );
+        }
+
+        val = tr_bencDictAdd( &top, "created by" );
+        tr_bencInitStrDup( val, TR_NAME "/" LONG_VERSION_STRING );
+
+        val = tr_bencDictAdd( &top, "creation date" );
+        tr_bencInitInt( val, time(0) );
+
+        val = tr_bencDictAdd( &top, "encoding" );
+        tr_bencInitStrDup( val, "UTF-8" );
+
+        val = tr_bencDictAdd( &top, "info" );
+        tr_bencInit( val, TYPE_DICT );
+        tr_bencDictReserve( val, 666 );
+        makeInfoDict( val, builder );
+    }
 
     /* save the file */
@@ -404 +419 @@
         FILE * fp = fopen( builder->outputFile, "wb+" );
         nmemb = n;
-        if( fp == NULL )
+        if( fp == NULL ) {
+            snprintf( builder->error, sizeof( builder->error ), _( "Couldn't open \"%s\": %s" ), builder->outputFile, tr_strerror( errno ) );
+            tr_err( "%s", builder->error );
             builder->failed = 1;
-        else if( fwrite( pch, 1, nmemb, fp ) != nmemb )
+        } else if( fwrite( pch, 1, nmemb, fp ) != nmemb ) {
+            snprintf( builder->error, sizeof( builder->error ), _( "Couldn't save file \"%s\": %s" ), builder->outputFile, tr_strerror( errno ) );
+            tr_err( "%s", builder->error ); 
             builder->failed = 1;
+            fclose( fp );
+        }
         tr_free( pch );
-        fclose( fp );
     }
 

trunk/libtransmission/makemeta.h

@@ -59 +59 @@
     int isDone;
     int failed; /* only meaningful if isDone is set */
+    char error[1024]; /* only meaningful if failed is set */
 
     /**

trunk/libtransmission/metainfo.c

@@ -33 +33 @@
 #include <unistd.h> /* unlink, stat */
 
-#include <miniupnp/miniwget.h> /* parseURL */
-
 #include "transmission.h"
 #include "bencode.h"
@@ -42 +40 @@
 #include "trcompat.h" /* strlcpy */
 #include "utils.h"
-
-
-static int
-tr_httpParseUrl( const char * url_in, int len,
-                 char ** setme_host, int * setme_port, char ** setme_path )
-{
-    char * url = tr_strndup( url_in, len );
-    char * path;
-    char host[4096+1];
-    unsigned short port;
-    int success;
-
-    success = parseURL( url, host, &port, &path );
-
-    if( success ) {
-        *setme_host = tr_strdup( host );
-        *setme_port = port;
-        *setme_path = tr_strdup( path );
-    }
-
-    tr_free( url );
-
-    return !success;
-}
 
 /***********************************************************************

trunk/libtransmission/utils.c

@@ -44 +44 @@
 #endif
 
+#include <miniupnp/miniwget.h> /* parseURL */
+
 #include "transmission.h"
 #include "trcompat.h"
@@ -974 +976 @@
     *out = '\0';
 }
+
+/***
+****
+***/
+
+int
+tr_httpParseUrl( const char * url_in, int len,
+                 char ** setme_host,
+                 int * setme_port,
+                 char ** setme_path )
+{
+    char * url = tr_strndup( url_in, len );
+    char * path;
+    char host[4096+1];
+    unsigned short port;
+    int success;
+
+    success = parseURL( url, host, &port, &path );
+
+    if( success ) {
+        if( setme_host ) *setme_host = tr_strdup( host );
+        if( setme_port ) *setme_port = port;
+        if( setme_path ) *setme_path = tr_strdup( path );
+    }
+
+    tr_free( url );
+
+    return !success;
+}
+

trunk/libtransmission/utils.h

@@ -163 +163 @@
 void tr_sha1_to_hex( char * out, const uint8_t * sha1 );
 
+
+int tr_httpParseUrl( const char * url,
+                     int          url_len,
+                     char     ** setme_host,
+                     int       * setme_port,
+                     char     ** setme_path );
+
+
 /***
 ****