Stable update for gdm3
Hi,
please find attached a proposed stable update for gdm3.
gdm3 (2.30.5-6squeeze4) stable; urgency=low
* 35_double_free.patch: stolen from 2.30.7. Fix a double free issue in
the chooser code.
* 36_windowpath.patch: stolen from 2.30.7. Only set the WINDOWPATH
variable if not null.
* 37_shutdown_buttons.patch: stolen from upstream git. Only show
shutdown options when requested. Closes: #628032.
* 14_pam_dialog.patch: remove the beep, since it happens after the
session has been reaped and can lock the sound device.
Cheers,
--
.''`. Josselin Mouette
: :' :
`. `'
`-
diff -Nru gdm3-2.30.5/debian/changelog gdm3-2.30.5/debian/changelog
--- gdm3-2.30.5/debian/changelog 2011-04-20 21:31:43.000000000 +0200
+++ gdm3-2.30.5/debian/changelog 2011-08-18 18:21:18.000000000 +0200
@@ -1,3 +1,16 @@
+gdm3 (2.30.5-6squeeze4) stable; urgency=low
+
+ * 35_double_free.patch: stolen from 2.30.7. Fix a double free issue in
+ the chooser code.
+ * 36_windowpath.patch: stolen from 2.30.7. Only set the WINDOWPATH
+ variable if not null.
+ * 37_shutdown_buttons.patch: stolen from upstream git. Only show
+ shutdown options when requested. Closes: #628032.
+ * 14_pam_dialog.patch: remove the beep, since it happens after the
+ session has been reaped and can lock the sound device.
+
+ -- Josselin Mouette <joss@debian.org> Thu, 18 Aug 2011 18:21:15 +0200
+
gdm3 (2.30.5-6squeeze3) stable; urgency=low
* 33_reset_signal_handler.patch: stolen upstream. Reset SIGPIPE
diff -Nru gdm3-2.30.5/debian/patches/14_pam_dialog.patch gdm3-2.30.5/debian/patches/14_pam_dialog.patch
--- gdm3-2.30.5/debian/patches/14_pam_dialog.patch 2011-04-20 21:28:24.000000000 +0200
+++ gdm3-2.30.5/debian/patches/14_pam_dialog.patch 2011-08-18 18:21:18.000000000 +0200
@@ -3,9 +3,9 @@
Index: gdm-2.30.5/daemon/gdm-session-worker.c
===================================================================
---- gdm-2.30.5.orig/daemon/gdm-session-worker.c 2010-09-16 11:29:04.000000000 +0200
-+++ gdm-2.30.5/daemon/gdm-session-worker.c 2010-09-16 11:31:42.000000000 +0200
-@@ -815,7 +815,7 @@
+--- gdm-2.30.5.orig/daemon/gdm-session-worker.c 2011-08-18 18:18:46.369202613 +0200
++++ gdm-2.30.5/daemon/gdm-session-worker.c 2011-08-18 18:18:46.689204182 +0200
+@@ -815,7 +815,7 @@ gdm_session_worker_process_pam_message (
res = gdm_session_worker_ask_for_secret (worker, utf8_msg, &user_answer);
break;
case PAM_TEXT_INFO:
@@ -17,12 +17,13 @@
Index: gdm-2.30.5/gui/simple-greeter/gdm-greeter-login-window.c
===================================================================
--- gdm-2.30.5.orig/gui/simple-greeter/gdm-greeter-login-window.c 2010-08-11 19:40:07.000000000 +0200
-+++ gdm-2.30.5/gui/simple-greeter/gdm-greeter-login-window.c 2010-09-16 11:31:06.000000000 +0200
-@@ -667,9 +667,68 @@
++++ gdm-2.30.5/gui/simple-greeter/gdm-greeter-login-window.c 2011-08-18 18:19:25.741393775 +0200
+@@ -667,8 +667,65 @@ gdm_greeter_login_window_problem (GdmGre
g_debug ("GdmGreeterLoginWindow: problem: %s", text);
- set_message (GDM_GREETER_LOGIN_WINDOW (login_window), text);
+- gdk_window_beep (GTK_WIDGET (login_window)->window);
+ _gdm_greeter_login_window_set_interactive (login_window, FALSE);
+
+ GtkWidget *dialog;
@@ -81,10 +82,7 @@
+ gtk_dialog_run (GTK_DIALOG (dialog));
+ gtk_widget_destroy (dialog);
+
- gdk_window_beep (GTK_WIDGET (login_window)->window);
-
+ _gdm_greeter_login_window_set_interactive (login_window, TRUE);
-+
+
return TRUE;
}
-
diff -Nru gdm3-2.30.5/debian/patches/35_double_free.patch gdm3-2.30.5/debian/patches/35_double_free.patch
--- gdm3-2.30.5/debian/patches/35_double_free.patch 1970-01-01 01:00:00.000000000 +0100
+++ gdm3-2.30.5/debian/patches/35_double_free.patch 2011-08-18 18:21:18.000000000 +0200
@@ -0,0 +1,20 @@
+From f33b0e8ccac7aa1719af42ec268071c8dcf6a395 Mon Sep 17 00:00:00 2001
+From: Brian Cameron <brian.cameron@oracle.com>
+Date: Thu, 02 Jun 2011 14:50:50 +0000
+Subject: Fix bug #650659 - double free issue in chooser.
+
+---
+diff --git a/gui/simple-greeter/gdm-chooser-widget.c b/gui/simple-greeter/gdm-chooser-widget.c
+index d4b0a4a..3c6ec4c 100644
+--- a/gui/simple-greeter/gdm-chooser-widget.c
++++ b/gui/simple-greeter/gdm-chooser-widget.c
+@@ -2355,6 +2355,7 @@ gdm_chooser_widget_lookup_item (GdmChooserWidget *widget,
+
+ if (active_item_id == NULL || strcmp (active_item_id, id) != 0) {
+ g_free (active_item_id);
++ active_item_id = NULL;
+
+ if (!find_item (widget, id, &iter)) {
+ return FALSE;
+--
+cgit v0.9.0.2
diff -Nru gdm3-2.30.5/debian/patches/36_windowpath.patch gdm3-2.30.5/debian/patches/36_windowpath.patch
--- gdm3-2.30.5/debian/patches/36_windowpath.patch 1970-01-01 01:00:00.000000000 +0100
+++ gdm3-2.30.5/debian/patches/36_windowpath.patch 2011-08-18 18:21:18.000000000 +0200
@@ -0,0 +1,38 @@
+From eef9fec895f1411066cb15c64be10e5e84b565e0 Mon Sep 17 00:00:00 2001
+From: Brian Cameron <brian.cameron@oracle.com>
+Date: Thu, 02 Jun 2011 14:56:41 +0000
+Subject: Only set WINDOWPATH if it is not NULL.
+
+---
+diff --git a/daemon/gdm-session-direct.c b/daemon/gdm-session-direct.c
+index d1fec90..73c34e2 100644
+--- a/daemon/gdm-session-direct.c
++++ b/daemon/gdm-session-direct.c
+@@ -2047,6 +2047,8 @@ gdm_session_direct_set_environment_variable (GdmSessionDirect *session,
+ static void
+ setup_session_environment (GdmSessionDirect *session)
+ {
++ char *windowpath;
++
+ gdm_session_direct_set_environment_variable (session,
+ "GDMSESSION",
+ get_session_name (session));
+@@ -2078,10 +2080,12 @@ setup_session_environment (GdmSessionDirect *session)
+ session->priv->user_x11_authority_file);
+ }
+
+- gdm_session_direct_set_environment_variable (session,
+- "WINDOWPATH",
+- g_getenv ("WINDOWPATH"));
+-
++ windowpath = g_getenv ("WINDOWPATH");
++ if (windowpath != NULL) {
++ gdm_session_direct_set_environment_variable (session,
++ "WINDOWPATH",
++ windowpath);
++ }
+
+ /* FIXME: We do this here and in the session worker. We should consolidate
+ * somehow.
+--
+cgit v0.9.0.2
diff -Nru gdm3-2.30.5/debian/patches/37_shutdown_buttons.patch gdm3-2.30.5/debian/patches/37_shutdown_buttons.patch
--- gdm3-2.30.5/debian/patches/37_shutdown_buttons.patch 1970-01-01 01:00:00.000000000 +0100
+++ gdm3-2.30.5/debian/patches/37_shutdown_buttons.patch 2011-08-18 18:21:18.000000000 +0200
@@ -0,0 +1,33 @@
+From d8a75a806215cf95832de70da7fdb9c7592d697f Mon Sep 17 00:00:00 2001
+From: Ray Strode <rstrode@redhat.com>
+Date: Wed, 15 Sep 2010 18:54:17 +0000
+Subject: Don't show shutdown options when not available
+
+The logic for showing the shut down menu was a little
+iffy before. This is apparently causing issues on solaris,
+and probably elsewhere.
+
+https://bugzilla.gnome.org/show_bug.cgi?id=624298
+---
+Index: gdm-2.30.5/gui/simple-greeter/gdm-greeter-panel.c
+===================================================================
+--- gdm-2.30.5.orig/gui/simple-greeter/gdm-greeter-panel.c 2011-08-18 18:19:35.393442281 +0200
++++ gdm-2.30.5/gui/simple-greeter/gdm-greeter-panel.c 2011-08-18 18:19:47.869503273 +0200
+@@ -838,7 +838,7 @@ setup_panel (GdmGreeterPanel *panel)
+ gtk_widget_show (panel->priv->hostname_label);
+ }
+
+- if (panel->priv->display_is_local || get_show_restart_buttons (panel)) {
++ if (!panel->priv->display_is_local || get_show_restart_buttons (panel)) {
+ GtkWidget *menu_item;
+ GtkWidget *image;
+
+@@ -869,7 +869,7 @@ setup_panel (GdmGreeterPanel *panel)
+ menu_item = gtk_menu_item_new_with_label (_("Quit"));
+ g_signal_connect (G_OBJECT (menu_item), "activate", G_CALLBACK (do_disconnect), NULL);
+ gtk_menu_shell_append (GTK_MENU_SHELL (panel->priv->shutdown_menu), menu_item);
+- } else {
++ } else if (get_show_restart_buttons (panel)) {
+ if (can_suspend ()) {
+ menu_item = gtk_menu_item_new_with_label (_("Suspend"));
+ g_signal_connect (G_OBJECT (menu_item), "activate", G_CALLBACK (do_system_suspend), NULL);
diff -Nru gdm3-2.30.5/debian/patches/series gdm3-2.30.5/debian/patches/series
--- gdm3-2.30.5/debian/patches/series 2011-04-20 21:31:43.000000000 +0200
+++ gdm3-2.30.5/debian/patches/series 2011-08-18 18:21:18.000000000 +0200
@@ -32,5 +32,8 @@
30_utf8_locale.patch
33_reset_signal_handler.patch
34_postsession_shutdown.patch
+35_double_free.patch
+36_windowpath.patch
+37_shutdown_buttons.patch
90_relibtoolize.patch
99_CVE-2011-0727.patch
Reply to: