diff options
author | Eric Andersen | 2001-12-06 14:52:32 +0000 |
---|---|---|
committer | Eric Andersen | 2001-12-06 14:52:32 +0000 |
commit | b24d65659f193cd7497dfdae4d8aa1bc91dbf343 (patch) | |
tree | 430b543e3b98a9028f5c0435b1363b2256d26a24 /procps/kill.c | |
parent | 06656f363de87846086f03c5e12541a66637858e (diff) | |
download | busybox-b24d65659f193cd7497dfdae4d8aa1bc91dbf343.zip busybox-b24d65659f193cd7497dfdae4d8aa1bc91dbf343.tar.gz |
If find_pid_by_name() had an error, it was returning -1, but storing
that into a pid_t, which is unsigned on a number archs. Furthermore,
find_pid_by_name() would _never_ return an error if the intended proces
was "init", but instead would return 1, meaning we would fail to work
on 2.4.x kernels running an initrd...
-Erik
Diffstat (limited to 'procps/kill.c')
-rw-r--r-- | procps/kill.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/procps/kill.c b/procps/kill.c index 8b8a992..7147b57 100644 --- a/procps/kill.c +++ b/procps/kill.c @@ -114,7 +114,7 @@ extern int kill_main(int argc, char **argv) pid_t myPid=getpid(); /* Looks like they want to do a killall. Do that */ while (--argc >= 0) { - pid_t* pidList; + long* pidList; pidList = find_pid_by_name( *argv); if (!pidList || *pidList<=0) { |