Merge lp:~ari-tczew/ubuntu/lucid/gnome-settings-daemon/lp-626379 into lp:ubuntu/lucid-proposed/gnome-settings-daemon

Proposed by Artur Rona
Status: Merged
Merge reported by: Martin Pitt
Merged at revision: not available
Proposed branch: lp:~ari-tczew/ubuntu/lucid/gnome-settings-daemon/lp-626379
Merge into: lp:ubuntu/lucid-proposed/gnome-settings-daemon
Diff against target: 54 lines (+33/-0)
3 files modified
debian/changelog (+8/-0)
debian/patches/92_return_if_nitems_is_0.patch (+24/-0)
debian/patches/series (+1/-0)
To merge this branch: bzr merge lp:~ari-tczew/ubuntu/lucid/gnome-settings-daemon/lp-626379
Reviewer Review Type Date Requested Status
Martin Pitt Approve
Review via email: mp+41410@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Martin Pitt (pitti) wrote :

This was uploaded to lucid-proposed, but without using bzr branch merge.

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 2010-04-28 10:46:53 +0000
3+++ debian/changelog 2010-11-21 13:47:55 +0000
4@@ -1,3 +1,11 @@
5+gnome-settings-daemon (2.30.1-0ubuntu1.1) lucid-proposed; urgency=low
6+
7+ [ Krzysztof Klimonda ]
8+ * debian/patches/92_return_if_nitems_is_0.patch: (LP: #626379)
9+ - Fix handle clipboard with KDE apps on Gnome, e.g. kolourpaint4.
10+
11+ -- Artur Rona <ari-tczew@ubuntu.com> Sun, 21 Nov 2010 14:21:45 +0100
12+
13 gnome-settings-daemon (2.30.1-0ubuntu1) lucid-proposed; urgency=low
14
15 * New upstream version (LP: #571250):
16
17=== added file 'debian/patches/92_return_if_nitems_is_0.patch'
18--- debian/patches/92_return_if_nitems_is_0.patch 1970-01-01 00:00:00 +0000
19+++ debian/patches/92_return_if_nitems_is_0.patch 2010-11-21 13:47:55 +0000
20@@ -0,0 +1,24 @@
21+From: Artur Rona <ari-tczew@ubuntu.com>
22+Description: Fix handle clipboard with KDE apps on Gnome, e.g. kolourpaint4.
23+Bug: https://bugzilla.gnome.org/628526
24+Bug-Ubuntu: https://launchpad.net/bugs/626379
25+Forwarded: yes
26+Author: Krzysztof Klimonda <kklimonda@ubuntu.com>
27+
28+Index: gnome-settings-daemon/plugins/clipboard/gsd-clipboard-manager.c
29+===================================================================
30+--- gnome-settings-daemon.orig/plugins/clipboard/gsd-clipboard-manager.c 2010-09-14 17:26:18.077966003 +0200
31++++ gnome-settings-daemon/plugins/clipboard/gsd-clipboard-manager.c 2010-09-14 17:35:07.117966003 +0200
32+@@ -599,8 +599,11 @@
33+ &type, &format, &nitems, &remaining,
34+ (unsigned char **) &multiple);
35+
36+- if (type != XA_ATOM_PAIR)
37++ if (type != XA_ATOM_PAIR || nitems == 0) {
38++ if (multiple)
39++ free (multiple);
40+ return;
41++ }
42+
43+ for (i = 0; i < nitems; i += 2) {
44+ rdata = (IncrConversion *) malloc (sizeof (IncrConversion));
45
46=== modified file 'debian/patches/series'
47--- debian/patches/series 2010-04-28 10:46:53 +0000
48+++ debian/patches/series 2010-11-21 13:47:55 +0000
49@@ -15,4 +15,5 @@
50 90_autoreconf.patch
51 91_bugzilla_update_touchpad_icons.patch
52 91_update_gvc_source.patch
53+92_return_if_nitems_is_0.patch
54 99_ltmain_as-needed.patch

Subscribers

People subscribed via source and target branches

to all changes: