diff options
| author | Eric Andersen <andersen@codepoet.org> | 2001-12-06 14:52:32 +0000 |
|---|---|---|
| committer | Eric Andersen <andersen@codepoet.org> | 2001-12-06 14:52:32 +0000 |
| commit | b24d65659f193cd7497dfdae4d8aa1bc91dbf343 (patch) | |
| tree | 430b543e3b98a9028f5c0435b1363b2256d26a24 /init/halt.c | |
| parent | 06656f363de87846086f03c5e12541a66637858e (diff) | |
| download | 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/halt.c')
| -rw-r--r-- | init/halt.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/init/halt.c b/init/halt.c index 307c1022d..917de8b5a 100644 --- a/init/halt.c +++ b/init/halt.c @@ -28,7 +28,7 @@ extern int halt_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) |