Bug 57521

Summary: free(): invalid pointer
Product: Telepathy Reporter: christian
Component: gabbleAssignee: Will Thompson <will>
Status: RESOLVED FIXED QA Contact: Telepathy bugs list <telepathy-bugs>
Severity: major    
Priority: medium CC: will
Version: git masterKeywords: patch
Hardware: x86-64 (AMD64)   
OS: Linux (All)   
URL: http://cgit.collabora.com/git/user/wjt/telepathy-gabble/log/?h=57521-crash-when-failing-to-create-invisible-list
Whiteboard: r+
i915 platform: i915 features:
Attachments: debug output

Description christian 2012-11-25 16:05:09 UTC
Created attachment 70542 [details]
debug output

telepathy-gabble crashes when trying to connect to the League Of Legends XMPP Server.
Using pidgin with the same configuration settings succeeds.
See attached log file for more information
Comment 1 Will Thompson 2012-11-27 16:58:26 UTC
I have a test case reproducing this crash…
Comment 2 christian 2012-11-28 13:01:04 UTC
(In reply to comment #1)
> I have a test case reproducing this crash…

Great. Could you please provide a bit more details about it?
Comment 3 Will Thompson 2012-11-28 15:12:40 UTC
…and here's a branch which fixes it.

The bug is in the error-handling path in Gabble's privacy list code. Specifically, a GError is being freed with g_free (which is incorrect). http://cgit.collabora.com/git/user/wjt/telepathy-gabble/commit/?h=57521-crash-when-failing-to-create-invisible-list&id=3ada6c09df28114b120d2e3b91f232f17c231ec2 fixes it.
Comment 4 Jonny Lamb 2012-11-28 15:31:34 UTC
yes.
Comment 5 christian 2012-11-29 13:06:08 UTC
(In reply to comment #3)
> http://cgit.collabora.com/git/user/wjt/telepathy-gabble/commit/?h=57521-
> crash-when-failing-to-create-invisible-
> list&id=3ada6c09df28114b120d2e3b91f232f17c231ec2 fixes it.

Thank you very much, this works fine for me
Comment 6 Will Thompson 2012-12-06 16:29:50 UTC
Crash fixes merged to 0.16 for 0.16.5; everything merged to master for 0.17.2.

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.