>From upstream:
"[Richard Hughes]: This looks like a problem in dbus-glib:
/* set spindown timeouts */
ret = dbus_g_proxy_call (disks->priv->proxy, "DriveSetAllSpindownTimeouts",
&error,
G_TYPE_INT, timeout,
G_TYPE_STRV, options,
G_TYPE_INVALID,
G_TYPE_STRING, disks->priv->cookie,
G_TYPE_INVALID);
if (!ret) {
egg_warning ("failed to set spindown timeout: %s", error->message);
^- GOES BANG
g_error_free (error);
goto out;
}
But in the docs:
dbus_g_proxy_call()...
Returns : FALSE if an error is set, TRUE otherwise.
So it's returning FALSE without setting ERROR non-NULL."
"[Scott Howard]: Thanks for looking at this, Richard. I found this, and think
we're on to
something:
http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=541632
"By passing dbus_g_proxy_call an incorrect signature, we can cause
the function to fail but not provide any error message. This isn't
helpful. This patch causes it to set an error when demarshalling
a type that it's not expecting. Instead of a NULL error, one instead
sees something like the following when the method returns a path
object but the client expected a string:"
I think we're passing "timeout" as a guint when it is expecting a gint
according to its signature."
Testing this is important for getting this fixed.
** Bug watch added: Debian Bug tracker #541632
http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=541632
** Changed in: gnome-power-manager (Ubuntu)
Importance: Medium => High
** Changed in: gnome-power-manager (Ubuntu)
Status: Confirmed => Triaged
--
gnome-power-manager when removing or inserting power cable
https://bugs.launchpad.net/bugs/447304
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
--
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
