** Changed in: gnome-settings-daemon (Ubuntu)
Status: Triaged => In Progress
--
You received this bug notification because you are a member of Desktop
Packages, which is subscribed to gnome-settings-daemon in Ubuntu.
https://bugs.launchpad.net/bugs/1339244
Title:
Comparison of exit_status in run_custom_command is wrong
Status in “gnome-settings-daemon” package in Ubuntu:
In Progress
Status in “gnome-settings-daemon” source package in Trusty:
New
Bug description:
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);
To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/gnome-settings-daemon/+bug/1339244/+subscriptions
--
Mailing list: https://launchpad.net/~desktop-packages
Post to : [email protected]
Unsubscribe : https://launchpad.net/~desktop-packages
More help : https://help.launchpad.net/ListHelp
