Bug 19101

Created attachment 21180 [details]
main.c

Here is a little example that triggers the crash.

This is still a problem for me with telepathy-glib from git.

Here is the backtrace:

Program received signal SIGSEGV, Segmentation fault.
0xb7c68ab1 in IA__g_str_hash (v=0x0) at gstring.c:95
95	  guint32 h = *p;
(gdb) bt
#0  0xb7c68ab1 in IA__g_str_hash (v=0x0) at gstring.c:95
#1  0xb7c31414 in g_hash_table_lookup_node (hash_table=0x8693cf0, key=0x0) at ghash.c:195
#2  0xb7c313b3 in IA__g_hash_table_lookup (hash_table=0x8693cf0, key=0x0) at ghash.c:783
#3  0xb7f0c881 in contacts_context_fail (c=0x86a4420, error=0x86a2280) at contact.c:779
#4  0xb7f0e68d in contacts_got_attributes (connection=0x86a0c00, attributes=0x8693d80, error=0x0, user_data=0x86a4420, weak_object=0x0) at contact.c:1683
#5  0xb7f06e1d in connection_got_contact_attributes (self=0x86a0c00, attributes=0x8693d80, error=0x0, user_data=0x86a2150, weak_object=0x0)
    at connection-handles.c:721
#6  0xb7efc3db in _tp_cli_connection_interface_contacts_invoke_callback_get_contact_attributes (self=0x86a0c00, error=0x0, args=0x86a2270, 
    generic_callback=0xb7f06ca8 <connection_got_contact_attributes>, user_data=0x86a2150, weak_object=0x0) at _gen/tp-cli-connection-body.h:7840
#7  0xb7f3d0bb in tp_proxy_pending_call_idle_invoke (p=0x869ad50) at proxy-methods.c:153
#8  0xb7c45667 in g_idle_dispatch (source=0x86a3640, callback=0xb7f3cf6a <tp_proxy_pending_call_idle_invoke>, user_data=0x869ad50) at gmain.c:3924
#9  0xb7c4173e in g_main_dispatch (context=0x86967d0) at gmain.c:1814
#10 0xb7c42c16 in IA__g_main_context_dispatch (context=0x86967d0) at gmain.c:2367
#11 0xb7c4316a in g_main_context_iterate (context=0x86967d0, block=1, dispatch=1, self=0x869eb50) at gmain.c:2448
#12 0xb7c43941 in IA__g_main_loop_run (loop=0x8695840) at gmain.c:2656
#13 0x0804914f in main ()

Note that Will Thompson reports that this is not a problem in telepathy-glib 0.7.18.

main.c works for me with telepathy-glib 0.7.20 :-/

Oh, I'm an idiot and didn't change the account credentials. Let's see...

Nope, it does still work for me. I'll try to figure out how your backtrace could occur.

Ah! You're using a version of telepathy-gabble that misbehaved, and hence are hitting a different path in TpContact! If you upgrade your gabble, the problem should go away; I'll look into the bug in this path of tp-glib.

Oh, that's odd. That particular bug was in tp-glib, not gabble. Is your gabble using an older telepathy-glib?

commit d7d571e73817aa5b889f80c7185c89702d9c2f58
Author: Will Thompson <will.thompson@collabora.co.uk>
Date:   Wed Jan 7 15:51:13 2009 +0000

    Don't treat requested IDs' NULL terminator as an id.
    
    Fixes fd.o bug #19101
---
 telepathy-glib/contact.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/telepathy-glib/contact.c b/telepathy-glib/contact.c
index 9e19d64..faf99ec 100644
--- a/telepathy-glib/contact.c
+++ b/telepathy-glib/contact.c
@@ -796,7 +796,8 @@ contacts_context_fail (ContactsContext *c,
           error, c->user_data, c->weak_object);
       return;
     case CB_BY_ID:
-      for (i = 0; i < c->request_ids->len; i++)
+      /* -1 because NULL terminator is explicit */
+      for (i = 0; i < c->request_ids->len - 1; i++)
         {
           const gchar *id = g_ptr_array_index (c->request_ids, i);