Changeset 9798

Timestamp:

Dec 18, 2009, 5:32:16 PM (12 years ago)

Author:

charles

Message:

(trunk libT) it certainly smells like we're using freed memory in the libcurl + libevent code in web.c... let's trash the structures right before free()ing them

File:

• trunk/libtransmission/web.c (modified) (4 diffs)

trunk/libtransmission/web.c

@@ -24 +24 @@
 enum
 {
+    TR_MEMORY_TRASH = 0xCC,
+
     DEFAULT_TIMER_MSEC = 1500 /* arbitrary */
 };
@@ -58 +60 @@
     curl_multi_cleanup( g->multi );
     evtimer_del( &g->timer_event );
+    memset( g, TR_MEMORY_TRASH, sizeof( struct tr_web ) );
     tr_free( g );
 }
@@ -82 +85 @@
     tr_free( task->range );
     tr_free( task->url );
+    memset( task, TR_MEMORY_TRASH, sizeof( struct tr_web_task ) );
     tr_free( task );
 }
@@ -283 +287 @@
         {
             event_del( io_event );
+            memset( io_event, TR_MEMORY_TRASH, sizeof( struct event ) );
             tr_free( io_event );
             curl_multi_assign( web->multi, fd, NULL );