ring-conference-channel: keep a self ref while removing channels

ring_conference_channel_emit_channel_removed() can lead to tearing
down the whole conference if no participants are left. To properly
do channel clean up, and send a signal for conference channel close,
keep a self reference until the function is completed.

1 files changed, 8 insertions, 1 deletions

diff --git a/src/ring-conference-channel.c b/src/ring-conference-channel.c
index f4789db..8636f13 100644
--- a/src/ring-conference-channel.c
+++ b/src/ring-conference-channel.c
@@ -1438,6 +1438,10 @@ ring_conference_channel_emit_channel_removed(
   if (i >= MODEM_MAX_CALLS)
     return;
 
+  /* note: as this function may lead to tearing down the whole
+   *       conference, keep a local self-reference */
+  g_object_ref (self);
+
   while (member) {
     TpIntSet *remove = tp_intset_new();
 
@@ -1495,7 +1499,7 @@ ring_conference_channel_emit_channel_removed(
   }
 
   if (n > 2)
-    return;
+    goto out;
 
   DEBUG("Too few members, close channel %p", self);
 
@@ -1504,6 +1508,9 @@ ring_conference_channel_emit_channel_removed(
    * not hangup.
    */
   ring_conference_channel_close_impl (self, FALSE, FALSE);
+
+out:
+  g_object_unref (self);
 }