Changeset 6834

Timestamp:

Oct 2, 2008, 4:42:35 PM (13 years ago)

Author:

charles

Message:

(gtk) #1290: removing torrent in web/remote interface crashes GTK+ client

File:

• trunk/gtk/main.c (modified) (3 diffs)

trunk/gtk/main.c

@@ -310 +310 @@
 }
 
-struct rpc_data
-{
-    int              type;
-    int              torrentId;
-    tr_torrent *     tor;
-    struct cbdata *  cbdata;
-};
-
-static int
-onRPCIdle( void * vdata )
-{
-    struct rpc_data * data = vdata;
-
-    switch( data->type )
+static tr_rpc_callback_status
+onRPCChanged( tr_handle             * handle UNUSED,
+              tr_rpc_callback_type    type,
+              struct tr_torrent     * tor,
+              void                  * gdata )
+{
+    struct cbdata * cbdata = gdata;
+    gdk_threads_enter( );
+
+    switch( type )
     {
         case TR_RPC_TORRENT_ADDED:
-            tr_core_add_torrent( data->cbdata->core,
-                                tr_torrent_new_preexisting(
-                                    data->tor ) );
+            tr_core_add_torrent( cbdata->core,
+                                 tr_torrent_new_preexisting( tor ) );
             break;
 
@@ -340 +335 @@
 
         case TR_RPC_TORRENT_REMOVING:
-            tr_core_torrent_destroyed( data->cbdata->core, data->torrentId );
+            tr_core_torrent_destroyed( cbdata->core, tr_torrentId( tor ) );
             break;
 
@@ -348 +343 @@
             break;
     }
-    g_free( data );
-    return FALSE;
-}
-
-static tr_rpc_callback_status
-onRPCChanged( tr_handle            * handle UNUSED,
-              tr_rpc_callback_type          type,
-              struct tr_torrent *           tor,
-              void *                        cbdata )
-{
-    /* this callback is being invoked from the libtransmission thread,
-     * so let's push the information over to the gtk+ thread where
-     * it's safe to update the gui */
-    struct rpc_data * data = g_new0( struct rpc_data, 1 );
-
-    data->type = type;
-    data->torrentId = tor ? tr_torrentId( tor ) : -1;
-    data->tor = type == TR_RPC_TORRENT_REMOVING ? NULL : tor;
-    data->cbdata = cbdata;
-    g_idle_add( onRPCIdle, data );
+
+    gdk_threads_leave( );
     return TR_RPC_OK;
 }