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 /init/poweroff.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 'init/poweroff.c')
| -rw-r--r-- | init/poweroff.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/init/poweroff.c b/init/poweroff.c index cec2d6d..d99b556 100644 --- a/init/poweroff.c +++ b/init/poweroff.c @@ -28,7 +28,7 @@ extern int poweroff_main(int argc, char **argv) { #ifdef CONFIG_FEATURE_INITRD /* don't assume init's pid == 1 */ - pid_t *pid = find_pid_by_name("init"); + long *pid = find_pid_by_name("init"); if (!pid || *pid<=0) { pid = find_pid_by_name("linuxrc"); if (!pid || *pid<=0) |