Public bug reported:

In file /plugins/common/gsd-input-helper.c in function
run_custom_command the variable exit_status doesn't contain the exit
status of the process, so the statement "return (exit_status == 1);" is
wrong.

The documentation 
(https://developer.gnome.org/glib/stable/glib-Spawning-Processes.html#g-spawn-sync)
 says:
exit_status    return location for child exit status, as returned by waitpid(), 
or NULL.

And in the documentation of waitpid (http://linux.die.net/man/2/waitpid)
exit_status is described.

You have to do something like this:
if (WIFEXITED(exit_status) == FALSE) {
    return FALSE;
}
return (WEXITSTATUS(exit_status) == 1);

** Affects: gnome-settings-daemon (Ubuntu)
     Importance: Undecided
         Status: New

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1339244

Title:
  Comparison of exit_status in run_custom_command is wrong

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/gnome-settings-daemon/+bug/1339244/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to