tags 483805 patch thanks Hi,
stumbled across the same bug. The problem is, that acpid connects stdout and stderr to /dev/null (only in daemon mode), but it opens /dev/null read-only. Thus, any output in /etc/init.d/gdm will fail, and as the script most probably does a ``set -e'', it fails altogether. The attached patch fixes this. In the meantime, you can work around the bug, by creating the following script as /etc/acpi/restart_gdm.sh: #!/bin/sh /etc/init.d/gdm restart > /dev/null 2>&1 < /dev/null and setting ``action=/etc/acpi/restart_gdm.sh''. Or, more generally, put the following line in any script in /etc/acpi at the top: #!/bin/sh exec > /dev/null 2>&1 < /dev/null #... Cheers, harry
diff -Nru a/acpid.c b/acpid.c --- a/acpid.c 2008-11-03 14:04:43.000000000 +0100 +++ b/acpid.c 2008-12-19 18:38:14.291127677 +0100 @@ -456,7 +456,7 @@ int log_opts; /* open /dev/null */ - nullfd = open("/dev/null", O_RDONLY); + nullfd = open("/dev/null", O_RDWR); if (nullfd < 0) { fprintf(stderr, "%s: can't open %s: %s\n", progname, "/dev/null", strerror(errno));
signature.asc
Description: PGP signature