diff options
author | andersen <andersen@69ca8d6d-28ef-0310-b511-8ec308f3f277> | 2001-12-06 14:52:32 +0000 |
---|---|---|
committer | andersen <andersen@69ca8d6d-28ef-0310-b511-8ec308f3f277> | 2001-12-06 14:52:32 +0000 |
commit | 16213be079ec7df974070d467db62e889a44d46d (patch) | |
tree | 430b543e3b98a9028f5c0435b1363b2256d26a24 /procps/pidof.c | |
parent | 75dc7d8fde627d6afef9b9bbcbcbec3cac07499b (diff) | |
download | busybox-w32-16213be079ec7df974070d467db62e889a44d46d.tar.gz busybox-w32-16213be079ec7df974070d467db62e889a44d46d.tar.bz2 busybox-w32-16213be079ec7df974070d467db62e889a44d46d.zip |
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
git-svn-id: svn://busybox.net/trunk/busybox@3854 69ca8d6d-28ef-0310-b511-8ec308f3f277
Diffstat (limited to 'procps/pidof.c')
-rw-r--r-- | procps/pidof.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/procps/pidof.c b/procps/pidof.c index 5a40288dc..941582704 100644 --- a/procps/pidof.c +++ b/procps/pidof.c | |||
@@ -58,7 +58,7 @@ extern int pidof_main(int argc, char **argv) | |||
58 | 58 | ||
59 | /* Looks like everything is set to go. */ | 59 | /* Looks like everything is set to go. */ |
60 | while(optind < argc) { | 60 | while(optind < argc) { |
61 | pid_t* pidList; | 61 | long* pidList; |
62 | 62 | ||
63 | pidList = find_pid_by_name( argv[optind]); | 63 | pidList = find_pid_by_name( argv[optind]); |
64 | if (!pidList || *pidList<=0) { | 64 | if (!pidList || *pidList<=0) { |