agent: Attaching to a NULL context should attach to the default one
authorOlivier Crête <olivier.crete@collabora.com>
Thu, 30 Jan 2014 23:37:25 +0000 (18:37 -0500)
committerOlivier Crête <olivier.crete@collabora.com>
Fri, 31 Jan 2014 06:49:07 +0000 (01:49 -0500)
Otherwise it would have attached to a newly created context

agent/agent.c

index 4f7842b..986dfb8 100644 (file)
@@ -3660,9 +3660,6 @@ nice_agent_attach_recv (
   Stream *stream = NULL;
   gboolean ret = FALSE;
 
-  /* ctx must be non-NULL if func is non-NULL. */
-  g_return_val_if_fail (func == NULL || ctx != NULL, FALSE);
-
   agent_lock();
 
   /* attach candidates */
@@ -3674,6 +3671,9 @@ nice_agent_attach_recv (
     goto done;
   }
 
+  if (ctx == NULL)
+    ctx = g_main_context_default ();
+
   /* Set the component’s I/O context. */
   component_set_io_context (component, ctx);
   component_set_io_callback (component, func, data, NULL, 0, NULL);