** Bug watch added: GNOME Bug Tracker #710791 https://bugzilla.gnome.org/show_bug.cgi?id=710791
** Also affects: gnome-settings-daemon via https://bugzilla.gnome.org/show_bug.cgi?id=710791 Importance: Unknown Status: Unknown ** Also affects: unity-settings-daemon (Ubuntu) Importance: Undecided Status: New ** Branch linked: lp:~robert-ancell/unity-settings-daemon/check-exit- status ** Changed in: gnome-settings-daemon (Ubuntu) Importance: Undecided => Medium ** Changed in: gnome-settings-daemon (Ubuntu Trusty) Importance: Undecided => Medium ** Changed in: unity-settings-daemon (Ubuntu Trusty) Importance: Undecided => Medium ** Changed in: unity-settings-daemon (Ubuntu Trusty) Status: New => Triaged ** Changed in: unity-settings-daemon (Ubuntu) Importance: Undecided => Medium ** Changed in: unity-settings-daemon (Ubuntu) Status: New => Triaged ** Changed in: gnome-settings-daemon (Ubuntu Trusty) Status: New => Triaged -- 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: Unknown Status in “gnome-settings-daemon” package in Ubuntu: Fix Released Status in “unity-settings-daemon” package in Ubuntu: Triaged Status in “gnome-settings-daemon” source package in Trusty: Triaged Status in “unity-settings-daemon” source package in Trusty: Triaged 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/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
