[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#780090: marked as done (unblock: mate-session-manager/1.8.1-8)



Your message dated Mon, 09 Mar 2015 18:38:36 +0100
with message-id <54FDDA9C.2020808@thykier.net>
and subject line Re: Bug#780090: unblock: mate-session-manager/1.8.1-8
has caused the Debian Bug report #780090,
regarding unblock: mate-session-manager/1.8.1-8
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
780090: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=780090
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message ---
Package: release.debian.org
Severity: normal
User: release.debian.org@packages.debian.org
Usertags: unblock

Please unblock package mate-session-manager

+  * debian/patches:
+    + Add 0004_no-crash-if-no-req-apps-found.patch. Avoid dereferencing NULL
+      pointer, avoid m-s-m crashing if no required applications can be found.
+      (Closes: #778817).

-> Crashes in MATE session manager must be prevented whereever possible as a crash
in mate-session-manager will crash the complete desktop session (or leave
it in an unusable state).

+    + Update 0001_msmgnome-Allow-users-to-disable-one-or-both-compatib.patch.
+      Fix typo in gsettings schema file. (Closes: #778826).

-> Typo fix on a non-i18n gsettings string.

As hinted by Niels in a private mail exchange, the fixes for the above
issues have been uploaded already.

light+love,
Mike

unblock mate-session-manager/1.8.1-8

-- System Information:
Debian Release: 8.0
  APT prefers stable
  APT policy: (990, 'stable'), (500, 'testing-updates'), (500, 'testing-proposed-updates'), (500, 'testing')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 3.16.0-4-amd64 (SMP w/4 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: sysvinit (via /sbin/init)
diff -Nru mate-session-manager-1.8.1/debian/changelog mate-session-manager-1.8.1/debian/changelog
--- mate-session-manager-1.8.1/debian/changelog	2015-03-09 08:20:38.000000000 +0100
+++ mate-session-manager-1.8.1/debian/changelog	2015-03-09 08:18:58.000000000 +0100
@@ -1,3 +1,14 @@
+mate-session-manager (1.8.1-8) unstable; urgency=medium
+
+  * debian/patches:
+    + Add 0004_no-crash-if-no-req-apps-found.patch. Avoid dereferencing NULL
+      pointer, avoid m-s-m crashing if no required applications can be found.
+      (Closes: #778817).
+    + Update 0001_msmgnome-Allow-users-to-disable-one-or-both-compatib.patch.
+      Fix typo in gsettings schema file. (Closes: #778826).
+
+ -- Mike Gabriel <sunweaver@debian.org>  Mon, 09 Mar 2015 08:14:50 +0100
+
 mate-session-manager (1.8.1-7) unstable; urgency=medium
 
   * debian/patches:
diff -Nru mate-session-manager-1.8.1/debian/patches/0001_msmgnome-Allow-users-to-disable-one-or-both-compatib.patch mate-session-manager-1.8.1/debian/patches/0001_msmgnome-Allow-users-to-disable-one-or-both-compatib.patch
--- mate-session-manager-1.8.1/debian/patches/0001_msmgnome-Allow-users-to-disable-one-or-both-compatib.patch	2015-03-09 08:20:38.000000000 +0100
+++ mate-session-manager-1.8.1/debian/patches/0001_msmgnome-Allow-users-to-disable-one-or-both-compatib.patch	2015-03-09 08:16:57.000000000 +0100
@@ -19,7 +19,7 @@
      </key>
 +    <key name="gnome-compat-startup" type="as">
 +      <default>[ 'keyring', 'smproxy' ]</default>
-+      <_summary>Control gnome compatibility componnent startup</_summary>
++      <_summary>Control gnome compatibility component startup</_summary>
 +      <_description>Control which compatibility components to start.</_description>
 +    </key>
      <child name="required-components" schema="org.mate.session.required-components"/>
diff -Nru mate-session-manager-1.8.1/debian/patches/0004_no-crash-if-no-req-apps-found.patch mate-session-manager-1.8.1/debian/patches/0004_no-crash-if-no-req-apps-found.patch
--- mate-session-manager-1.8.1/debian/patches/0004_no-crash-if-no-req-apps-found.patch	1970-01-01 01:00:00.000000000 +0100
+++ mate-session-manager-1.8.1/debian/patches/0004_no-crash-if-no-req-apps-found.patch	2015-03-09 06:02:49.000000000 +0100
@@ -0,0 +1,43 @@
+
+m 3d54b96a285272cf5ac6a191a1b53ce018305b3e Mon Sep 17 00:00:00 2001
+From: monsta <monsta@inbox.ru>
+Date: Thu, 19 Feb 2015 16:05:52 +0300
+Subject: [PATCH] avoid dereferencing NULL pointer
+
+---
+ mate-session/main.c | 8 +++++---
+ 1 file changed, 5 insertions(+), 3 deletions(-)
+
+diff --git a/mate-session/main.c b/mate-session/main.c
+index 63cc5fc..f5f5734 100644
+--- a/mate-session/main.c
++++ b/mate-session/main.c
+@@ -249,15 +249,16 @@ static void append_required_apps(GsmManager* manager)
+ 	{
+ 		g_warning("No required applications specified");
+ 	}
+-
+-	for (i = 0; required_components[i]; i++)
++	else
+ 	{
++		for (i = 0; required_components[i]; i++)
++		{
+ 			char* default_provider;
+ 			const char* component;
+ 
+ 			if (IS_STRING_EMPTY((char*) required_components[i]))
+ 			{
+-					continue;
++				continue;
+ 			}
+ 
+ 			component = required_components[i];
+@@ -285,6 +286,7 @@ static void append_required_apps(GsmManager* manager)
+ 			}
+ 
+ 			g_free(default_provider);
++		}
+ 	}
+ 
+ 	g_debug("main: *** Done adding required apps");
+
diff -Nru mate-session-manager-1.8.1/debian/patches/series mate-session-manager-1.8.1/debian/patches/series
--- mate-session-manager-1.8.1/debian/patches/series	2015-03-09 08:20:38.000000000 +0100
+++ mate-session-manager-1.8.1/debian/patches/series	2015-03-09 06:03:04.000000000 +0100
@@ -1,4 +1,5 @@
 0001_msmgnome-Allow-users-to-disable-one-or-both-compatib.patch
 0002_msmgnome-Allow-users-to-disable-one-or-both-compatib.patch
 0003_set-XDG-CURRENT-DESKTOP-if-empty.patch
+0004_no-crash-if-no-req-apps-found.patch
 1001_hypen-used-as-minus-sign-man-page-fix.patch

--- End Message ---
--- Begin Message ---
On 2015-03-09 08:32, Mike Gabriel wrote:
> Package: release.debian.org
> Severity: normal
> User: release.debian.org@packages.debian.org
> Usertags: unblock
> 
> Please unblock package mate-session-manager
> 
> +  * debian/patches:
> +    + Add 0004_no-crash-if-no-req-apps-found.patch. Avoid dereferencing NULL
> +      pointer, avoid m-s-m crashing if no required applications can be found.
> +      (Closes: #778817).
> 
> -> Crashes in MATE session manager must be prevented whereever possible as a crash
> in mate-session-manager will crash the complete desktop session (or leave
> it in an unusable state).
> 
> +    + Update 0001_msmgnome-Allow-users-to-disable-one-or-both-compatib.patch.
> +      Fix typo in gsettings schema file. (Closes: #778826).
> 
> -> Typo fix on a non-i18n gsettings string.
> 
> As hinted by Niels in a private mail exchange, the fixes for the above
> issues have been uploaded already.
> 
> light+love,
> Mike
> 
> unblock mate-session-manager/1.8.1-8
> 
> [...]

Unblocked, thanks.

~Niels

--- End Message ---

Reply to: