Revision: 23615 http://trac.macosforge.org/projects/launchd/changeset/23615 Author: zarzycki@apple.com Date: 2008-05-02 15:53:47 -0700 (Fri, 02 May 2008) Log Message: ----------- <rdar://problem/5906374> launchd calls __kill() on arm Modified Paths: -------------- branches/SULeopard/launchd/src/launchd_runtime_kill.c Modified: branches/SULeopard/launchd/src/launchd_runtime_kill.c =================================================================== --- branches/SULeopard/launchd/src/launchd_runtime_kill.c 2008-04-30 21:32:53 UTC (rev 23614) +++ branches/SULeopard/launchd/src/launchd_runtime_kill.c 2008-05-02 22:53:47 UTC (rev 23615) @@ -18,36 +18,27 @@ * @APPLE_APACHE_LICENSE_HEADER_END@ */ -#if !defined(__LP64__) && !defined(__arm__) -#define _NONSTD_SOURCE 1 -#define old_kill(x, y) kill(x, y) -#define old_killpg(x, y) killpg(x, y) -#else -/* ??? No blessed way to get the old behavior */ -extern int __kill(int, int, int); -#define old_kill(x, y) __kill(x, y, 0) -#define old_killpg(x, y) __kill(-(x), y, 0) -#endif +#include <sys/syscall.h> +#include <unistd.h> #include <signal.h> #include "launchd_runtime_kill.h" -/* - * POSIX defines consistency over correctness, and consequently kill/killpg now - * returns EPERM instead of ESRCH. - * - * I've filed 5487498 to get a non-portable kill() variant, but for now, - * defining _NONSTD_SOURCE gets us the old behavior. - */ - int runtime_kill(pid_t pid, int sig) { - return old_kill(pid, sig); + /* + * POSIX defines consistency over correctness, and consequently + * kill/killpg now returns EPERM instead of ESRCH. + * + * I've filed 5487498 to get a non-portable kill(). + * We'll regretfully take advantage of implementation details for now. + */ + return syscall(SYS_kill, pid, sig, 0); } int runtime_killpg(pid_t pgrp, int sig) { - return old_killpg(pgrp, sig); + return runtime_kill(-pgrp, sig); }