rpms/xfce4-xkb-plugin/devel xfce4-xkb-plugin-0.5.3.3-fix-various-segfaults.patch, NONE, 1.1 xfce4-xkb-plugin.spec, 1.23, 1.24

Christoph Wickert cwickert at fedoraproject.org
Fri Feb 12 00:22:35 UTC 2010


Author: cwickert

Update of /cvs/pkgs/rpms/xfce4-xkb-plugin/devel
In directory cvs1.fedora.phx.redhat.com:/tmp/cvs-serv9885/devel

Modified Files:
	xfce4-xkb-plugin.spec 
Added Files:
	xfce4-xkb-plugin-0.5.3.3-fix-various-segfaults.patch 
Log Message:
* Fri Feb 12 2010 Christoph Wickert <cwickert at fedoraproject.org> - 0.5.3.3-3
- Fix various segfaults. Thanks to Lionel Le Folgoc (#525471, #547553)


xfce4-xkb-plugin-0.5.3.3-fix-various-segfaults.patch:
 b/panel-plugin/xkb-config.c |    3 +--
 panel-plugin/xkb-config.c   |    5 ++---
 2 files changed, 3 insertions(+), 5 deletions(-)

--- NEW FILE xfce4-xkb-plugin-0.5.3.3-fix-various-segfaults.patch ---
>From 2856ccfd12b7d9844538ea43a34e9f7af9e945a6 Mon Sep 17 00:00:00 2001
From: Lionel Le Folgoc <mrpouit at ubuntu.com>
Date: Mon, 8 Feb 2010 20:05:12 +0100
Subject: [PATCH 1/2] Use g_strdup("") for options, instead of "", because it is later free'd (thus segfaults).

---
 panel-plugin/xkb-config.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/panel-plugin/xkb-config.c b/panel-plugin/xkb-config.c
index bff77dd..020e915 100644
--- a/panel-plugin/xkb-config.c
+++ b/panel-plugin/xkb-config.c
@@ -290,7 +290,7 @@ xkb_config_update_settings (t_xkb_settings *settings)
         if (settings->kbd_config->toggle_option
                 && strlen (settings->kbd_config->toggle_option) > 0)
             options = g_strdup (settings->kbd_config->toggle_option);
-        else options = "";
+        else options = g_strdup ("");
 
         if (settings->kbd_config->compose_key_position
                 && strlen (settings->kbd_config->compose_key_position) > 0)
-- 
1.6.3.3


>From 04022389eee43503381f88f4f14ea76ec4941464 Mon Sep 17 00:00:00 2001
From: Lionel Le Folgoc <mrpouit at ubuntu.com>
Date: Mon, 8 Feb 2010 20:28:37 +0100
Subject: [PATCH 2/2] Fix another segfault.

---
 panel-plugin/xkb-config.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/panel-plugin/xkb-config.c b/panel-plugin/xkb-config.c
index 020e915..8546b31 100644
--- a/panel-plugin/xkb-config.c
+++ b/panel-plugin/xkb-config.c
@@ -312,11 +312,11 @@ xkb_config_update_settings (t_xkb_settings *settings)
     {
         prefix = g_strsplit(*opt, ":", 2);
         if (settings->kbd_config->toggle_option == NULL
-                && prefix && strcmp(*prefix, "grp") == 0)
+                && prefix && *prefix && strcmp(*prefix, "grp") == 0)
         {
             settings->kbd_config->toggle_option = g_strdup (*opt);
         }
-        else if (prefix && strcmp(*prefix, "compose") == 0)
+        else if (prefix && *prefix && strcmp(*prefix, "compose") == 0)
         {
             settings->kbd_config->compose_key_position = g_strdup (*opt);
         }
-- 
1.6.3.3



Index: xfce4-xkb-plugin.spec
===================================================================
RCS file: /cvs/pkgs/rpms/xfce4-xkb-plugin/devel/xfce4-xkb-plugin.spec,v
retrieving revision 1.23
retrieving revision 1.24
diff -u -p -r1.23 -r1.24
--- xfce4-xkb-plugin.spec	25 Jan 2010 23:43:23 -0000	1.23
+++ xfce4-xkb-plugin.spec	12 Feb 2010 00:22:35 -0000	1.24
@@ -1,6 +1,6 @@
 Name:		xfce4-xkb-plugin
 Version:	0.5.3.3
-Release:	2%{?dist}
+Release:	3%{?dist}
 Summary:	XKB layout switcher for the Xfce panel
 
 Group:		User Interface/Desktops
@@ -9,6 +9,9 @@ URL:		http://goodies.xfce.org/projects/p
 Source0:	http://goodies.xfce.org/releases/%{name}/%{name}-%{version}.tar.gz
 Patch0:		xfce4-xkb-plugin-0.5.2-xklavier-api.patch
 Patch1:		xfce4-xkb-plugin-0.5.3.3-libxklavier5.patch
+# https://bugzilla.redhat.com/show_bug.cgi?id=525471
+# fix at http://bugzilla.xfce.org/show_bug.cgi?id=6122#c1
+Patch2:		xfce4-xkb-plugin-0.5.3.3-fix-various-segfaults.patch
 BuildRoot:	%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
 
 BuildRequires:	xfce4-panel-devel >= 4.3.20, libxfcegui4-devel >= 4.3.90.2, libxml2-devel
@@ -25,6 +28,7 @@ XF86Config file or some other way (e.g. 
 
 %prep
 %setup -q
+%patch2 -p1 -b .orig
 # Fedora > 11 has libxklavier 4.0
 %if 0%{?fedora} > 11
 %patch0 -p1 -b .xklavier4
@@ -56,6 +60,9 @@ rm -rf $RPM_BUILD_ROOT
 %{_datadir}/xfce4/xkb/flags/*.svg
 
 %changelog
+* Fri Feb 12 2010 Christoph Wickert <cwickert at fedoraproject.org> - 0.5.3.3-3
+- Fix various segfaults. Thanks to Lionel Le Folgoc (#525471, #547553)
+
 * Mon Jan 25 2010 Christoph Wickert <cwickert at fedoraproject.org> - 0.5.3.3-2
 - Add patch for libxklavier 5.0. Thanks to Caolan McNamara (#558083)
 



More information about the scm-commits mailing list