Attachment 52238 Details for Bug 41697 – _tp_contacts_from_values: skip NULL contacts

[patch] _tp_contacts_from_values: skip NULL contacts

tpcontactsfromvalues-skip-NULL-contacts.patch (text/plain), 974 bytes, created by Guillaume Desmottes on 2011-10-11 14:39:04 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Guillaume Desmottes

Created: 2011-10-11 14:39:04 UTC

Size: 974 bytes

patch

obsolete

>From 357648407e52ee8f1991bac9dd863874571a1871 Mon Sep 17 00:00:00 2001
>From: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
>Date: Tue, 11 Oct 2011 17:38:11 -0400
>Subject: [PATCH] _tp_contacts_from_values: skip NULL contacts
>
>dup_owners_table() can insert NULL contacts into the hash (if the owner is
>unknown) so we should just ignore those.
>
>https://bugs.freedesktop.org/show_bug.cgi?id=41697
>---
> telepathy-glib/util.c |    4 +++-
> 1 files changed, 3 insertions(+), 1 deletions(-)
>
>diff --git a/telepathy-glib/util.c b/telepathy-glib/util.c
>index 9f26c3c..f8ffbbc 100644
>--- a/telepathy-glib/util.c
>+++ b/telepathy-glib/util.c
>@@ -2026,7 +2026,9 @@ _tp_contacts_from_values (GHashTable *table)
>   g_hash_table_iter_init (&iter, table);
>   while (g_hash_table_iter_next (&iter, NULL, &value))
>     {
>-      g_assert (G_IS_OBJECT (value));
>+      if (value == NULL)
>+        continue;
>+
>       g_ptr_array_add (contacts, g_object_ref (value));
>     }
> 
>-- 
>1.7.4.1

Actions: View | Splinter Review

Attachments on bug 41697: 52238 | 52239

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.