Merge lp:~om26er/ubuntu/oneiric/empathy/fix-crashes into lp:~ubuntu-desktop/empathy/ubuntu

Proposed by Omer Akram
Status: Merged
Approved by: Marc Deslauriers
Approved revision: 214
Merged at revision: 214
Proposed branch: lp:~om26er/ubuntu/oneiric/empathy/fix-crashes
Merge into: lp:~ubuntu-desktop/empathy/ubuntu
Diff against target: 59 lines (+39/-0)
3 files modified
debian/changelog (+8/-0)
debian/patches/70_lp_832378.patch (+30/-0)
debian/patches/series (+1/-0)
To merge this branch: bzr merge lp:~om26er/ubuntu/oneiric/empathy/fix-crashes
Reviewer Review Type Date Requested Status
Marc Deslauriers Approve
Review via email: mp+72761@code.launchpad.net

Description of the change

empathy is crashing alot few nasty crashes are fixed in master so I backported a commit that should fix the two linked bugs. The main reason is to have Oneiric beta with a working empathy in case upstream version comes after our beta release.

To post a comment you must log in.
Revision history for this message
Marc Deslauriers (mdeslaur) wrote :

Look good. ACK. Thanks!
Pending archive freeze exception approval, I'll upload it.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2011-08-22 19:03:49 +0000
3+++ debian/changelog 2011-08-24 17:25:23 +0000
4@@ -1,3 +1,11 @@
5+empathy (3.1.5.1-1ubuntu2) oneiric; urgency=low
6+
7+ * debian/patches/70_lp_832378.patch:
8+ - empathy crashed with SIGSEGV in __memcpy_ssse3() (LP: #828802)
9+ - empathy crashed with SIGSEGV in g_memdup() (LP: #832378)
10+
11+ -- Omer Akram <om26er@ubuntu.com> Wed, 24 Aug 2011 22:15:51 +0500
12+
13 empathy (3.1.5.1-1ubuntu1) oneiric; urgency=low
14
15 * New upstream version
16
17=== added file 'debian/patches/70_lp_832378.patch'
18--- debian/patches/70_lp_832378.patch 1970-01-01 00:00:00 +0000
19+++ debian/patches/70_lp_832378.patch 2011-08-24 17:25:23 +0000
20@@ -0,0 +1,30 @@
21+From: Omer Akram <om26er@ubuntu.com>
22+Author: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>
23+Subject: Don't crash if we failed to load the avatar.
24+Origin: http://git.gnome.org/browse/empathy/commit/?id=f60338e75ad72300e1f93ea38ce17284d50d698c
25+Bug: https://bugzilla.gnome.org/656829
26+Bug-Ubuntu: https://launchpad.net/bugs/828802
27+
28+--- a/libempathy/empathy-contact.c
29++++ b/libempathy/empathy-contact.c
30+@@ -1811,8 +1811,17 @@ contact_set_avatar_from_tp_contact (EmpathyContact *contact)
31+ gchar *data;
32+ gsize len;
33+ gchar *path;
34++ GError *error = NULL;
35++
36++ if (!g_file_load_contents (file, NULL, &data, &len, NULL, &error))
37++ {
38++ DEBUG ("Failed to load avatar: %s", error->message);
39++
40++ g_error_free (error);
41++ contact_set_avatar (contact, NULL);
42++ return;
43++ }
44+
45+- g_file_load_contents (file, NULL, &data, &len, NULL, NULL);
46+ path = g_file_get_path (file);
47+
48+ avatar = empathy_avatar_new ((guchar *) data, len, mime, path);
49+--
50+1.7.4.1
51
52=== modified file 'debian/patches/series'
53--- debian/patches/series 2011-08-17 16:57:16 +0000
54+++ debian/patches/series 2011-08-24 17:25:23 +0000
55@@ -6,3 +6,4 @@
56 37_facebook_default.patch
57 38_lp_569289.patch
58 41_unity_launcher_progress.patch
59+70_lp_832378.patch

Subscribers

People subscribed via source and target branches