Only the foreground user should be able to trigger suspends

Bug #26618 reported by Matthew Garrett
8
Affects Status Importance Assigned to Milestone
gdm (Ubuntu)
Fix Released
Medium
Ubuntu Desktop Bugs

Bug Description

The following patch checks that the foreground vt corresponds to the vt making
the suspend request. This prevents background users from triggering suspends
when idle.

Revision history for this message
Matthew Garrett (mjg59) wrote :

Created an attachment (id=5193)
Patch to do this

Revision history for this message
Carthik Sharma (carthik) wrote :

There is a patch, so is this to be applied sometime? Looks like it slipped through the cracks :)

Changed in gdm:
status: Unconfirmed → Needs Info
Revision history for this message
doclist (dclist) wrote :

I am also waiting for this bug to be addressed.

Revision history for this message
Daniel Holbach (dholbach) wrote :

Marking as Fix Committed - we should look at it after Herd2 release.

Changed in gdm:
assignee: seb128 → desktop-bugs
status: Needs Info → Fix Committed
Revision history for this message
Sebastien Bacher (seb128) wrote :

That upload fixes the problem:

 gdm (2.17.7-0ubuntu1) feisty; urgency=low
 .
   * New upstream version:
     - New sockets command FLEXI_XNEST_USER and FLEXI_XSERVER_USER. This is
       used to start login procedure by programs like gnome-screensaver and the
       Fast User Switch applet.
     - Improvements to PAM logic so that it works more generically with PAM
       modules.
     - Better fix for not showing the face browser if the userlist is empty.
       This avoids a needless selection "changed" signal from getting generated.
     - Fix so ConsoleKit logic does not use dbus-glib bindings to using the
       lower level API instead. Fixes bug #400793.
     - Fixes for compiler warnings and for c99 compile issues.
     - Avoid catching sigchld when trying to call XOpenDisplay. This corrects
       an issue where GDM catches a second SIGCHLD when trying to restart.
       This causes the slave to freeze even though the Xserver and the session
       are killed, causing the display to no longer manage.
     - Improvements to PAM documentation in GDM docs.
     - Translation updates
   * debian/patches/11_powermanagement.patch:
     - updated with change by Matthew Garrett to limit suspend and hibernate
       to the foreground user (Ubuntu: #26618)

Changed in gdm:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.