Package: upower
Version: 0.9.12-1
Tags: patch

maybe I have found the problem and the solution.

In the function up_backend_supports_sleep_state() the returnet value of g_spawn_command_line_sync() overwrite always the return value (wich is default FALSE) with a default TRUE value.

This value is always TRUE because /usr/bin/pm-is-supported exists (the case of non existent pm-is-supported is already managed after the valutation of retInt

So I have added a retInt variable who store the returned value of g_spawn_command_line_sync() and for his evaluation.

The patch wich I propose is written below, I hope it's safe, correct and applicable at the source code


--- src/linux/up-backend_old.c  2011-08-31 22:11:23.000000000 +0200
+++ src/linux/up-backend.c      2011-08-31 22:14:30.000000000 +0200
@@ -348,6 +348,7 @@

  up_backend_supports_sleep_state (const gchar *state)
 {
        gboolean ret = FALSE;
+       gboolean retInt;
        gchar *command;
        GError *error = NULL;
        gint exit_status;
@@ -355,8 +356,8 @@
        /* run script from pm-utils */
        command = g_strdup_printf ("/usr/bin/pm-is-supported --%s", state);
        g_debug ("excuting command: %s", command);
- ret = g_spawn_command_line_sync (command, NULL, NULL, &exit_status, &error);
-       if (!ret) {
+ retInt = g_spawn_command_line_sync (command, NULL, NULL, &exit_status, &error);
+       if (!retInt) {
                g_warning ("failed to run script: %s", error->message);
                g_error_free (error);
                goto out;





--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to