Changeset 7687

Timestamp:

Jan 12, 2009, 7:58:16 PM (12 years ago)

Author:

charles

Message:

(trunk libT) #1427: Files are kept open in read/write mode after a torrent finishes

File:

• trunk/libtransmission/torrent.c (modified) (7 diffs)

trunk/libtransmission/torrent.c

@@ -20 +20 @@
 #include <string.h> /* memcmp */
 #include <stdlib.h> /* qsort */
+
+#include <event.h> /* evbuffer */
 
 #include "transmission.h"
@@ -1130 +1132 @@
 
 static void
+tr_torrentCloseLocalFiles( const tr_torrent * tor )
+{
+    tr_file_index_t i;
+    struct evbuffer * buf = evbuffer_new( );
+
+    for( i=0; i<tor->info.fileCount; ++i )
+    {
+        const tr_file * file = &tor->info.files[i];
+        evbuffer_drain( buf, EVBUFFER_LENGTH( buf ) );
+        evbuffer_add_printf( buf, "%s%s%s", tor->downloadDir, TR_PATH_DELIMITER_STR, file->name );
+        tr_fdFileClose( (const char*) EVBUFFER_DATA( buf ) );
+    }
+
+    evbuffer_free( buf );
+}
+
+
+static void
 stopTorrent( void * vtor )
 {
-    tr_file_index_t i;
-
-    tr_torrent *    tor = vtor;
+    tr_torrent * tor = vtor;
 
     tr_verifyRemove( tor );
@@ -1140 +1158 @@
     tr_trackerStop( tor->tracker );
 
-    for( i = 0; i < tor->info.fileCount; ++i )
-    {
-        const tr_file * file = &tor->info.files[i];
-        char * path = tr_buildPath( tor->downloadDir, file->name, NULL );
-        tr_fdFileClose( path );
-        tr_free( path );
-    }
+    tr_torrentCloseLocalFiles( tor );
 }
 
@@ -1278 +1290 @@
 
         tor->completeness = completeness;
+        tr_torrentCloseLocalFiles( tor );
         fireCompletenessChange( tor, completeness );
 
@@ -1855 +1868 @@
     walkLocalData( tor, root, root, NULL, &torrentFiles, &folders, &dirtyFolders );
 
-    /* close all the files because we're about to delete them */
-    for( f=0; f<tor->info.fileCount; ++f ) {
-        char * path = tr_buildPath( tor->downloadDir, tor->info.files[f].name, NULL );
-        tr_fdFileClose( path );
-        tr_free( path );
-    }
-
     /* try to remove entire folders first, so that the recycle bin will be tidy */
     s = (char**) tr_ptrArrayPeek( &folders, &n );
@@ -1904 +1910 @@
         fileFunc = unlink;
 
+    /* close all the files because we're about to delete them */
+    tr_torrentCloseLocalFiles( tor );
+
     if( tor->info.fileCount > 1 )
         deleteLocalData( tor, fileFunc );
@@ -1909 +1918 @@
         /* torrent only has one file */
         char * path = tr_buildPath( tor->downloadDir, tor->info.files[0].name, NULL );
-        tr_fdFileClose( path );
         fileFunc( path );
         tr_free( path );