Bug#506549: suspendorhibernate: dbus-pm method is inherently broken

2008-11-24 Thread Loïc Minier
On Sat, Nov 22, 2008, Bart Samwel wrote: dbus-pm) if [ -x /usr/bin/dbus-send ] ; then +getXconsole +. $userhome/.dbus/session-bus/* # Call the power management daemon (which, if it is # running, we probably don't

Bug#506549: suspendorhibernate: dbus-pm method is inherently broken

2008-11-23 Thread Raphael Hertzog
On Sun, 23 Nov 2008, Bart Samwel wrote: 1. This program is running as root, right? I would be very careful with sourcing arbitrary shell commands from a users home directory then. I agree that that would be risky. However, on my system the .dbus directory is owned by root and not

Bug#506549: suspendorhibernate: dbus-pm method is inherently broken

2008-11-23 Thread Bart Samwel
Raphael Hertzog wrote: On Sun, 23 Nov 2008, Bart Samwel wrote: 1. This program is running as root, right? I would be very careful with sourcing arbitrary shell commands from a users home directory then. I agree that that would be risky. However, on my system the .dbus directory is owned by

Bug#506549: suspendorhibernate: dbus-pm method is inherently broken

2008-11-23 Thread Raphael Hertzog
On Sun, 23 Nov 2008, Bart Samwel wrote: Sounds like a definite possibility. The export DISPLAY=... stuff won't work however, because dbus doesn't derive it's session bus from that AFAICT. The documentation doesn't say so indeed but in my .dbus/session-bus file there's a comment that says: #

Bug#506549: suspendorhibernate: dbus-pm method is inherently broken

2008-11-22 Thread Fredrik Tolf
Package: acpi-support Version: 0.109-9 Severity: normal The dbus-pm suspend method in the suspendorhibernate script (which is enabled by default) is inherently broken and, I think, should be removed altogether. It uses the dbus-send program with the --session option, but since it does not run in

Bug#506549: suspendorhibernate: dbus-pm method is inherently broken

2008-11-22 Thread Bart Samwel
Hi Fredrik, Fredrik Tolf wrote: The dbus-pm suspend method in the suspendorhibernate script (which is enabled by default) is inherently broken and, I think, should be removed altogether. It uses the dbus-send program with the --session option, but since it does not run in the desktop user's

Bug#506549: suspendorhibernate: dbus-pm method is inherently broken

2008-11-22 Thread Fredrik Tolf
On Sat, 2008-11-22 at 20:31 +0100, Bart Samwel wrote: Anyway, I don't think the suspend method is inherently broken, although it is broken as it is now. :-) We do things inside X sessions as well (such as locking all screens when suspending), so I expect it should be possible to get dbus-send

Bug#506549: suspendorhibernate: dbus-pm method is inherently broken

2008-11-22 Thread Bart Samwel
Hi Fredrik, Fredrik Tolf wrote: On Sat, 2008-11-22 at 20:31 +0100, Bart Samwel wrote: Anyway, I don't think the suspend method is inherently broken, although it is broken as it is now. :-) We do things inside X sessions as well (such as locking all screens when suspending), so I expect it