diff options
| author | Carsten Otte <cotte@de.ibm.com> | 2006-03-25 11:08:22 (GMT) |
|---|---|---|
| committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-03-25 16:23:01 (GMT) |
| commit | 55148548124e3e52e8921f1cb0e325111ef9cbb1 (patch) | |
| tree | b1a84317305d060b38bc6fcb1ae4006a76f22071 | |
| parent | 06842415149371127b91d15774cff95dcccc1845 (diff) | |
| download | linux-fsl-qoriq-55148548124e3e52e8921f1cb0e325111ef9cbb1.tar.xz | |
[PATCH] remove needless check in binfmt_elf.c
Local variable i is unsigned int and thus cannot be negative.
(akpm: unsigneds shouldn't be called `i'. This value cannot possibly be
negative anyway).
Signed-off-by: Carsten Otte <cotte@de.ibm.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
| -rw-r--r-- | fs/binfmt_elf.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/binfmt_elf.c b/fs/binfmt_elf.c index 61c21e7..4349113 100644 --- a/fs/binfmt_elf.c +++ b/fs/binfmt_elf.c @@ -1334,7 +1334,7 @@ static int fill_psinfo(struct elf_prpsinfo *psinfo, struct task_struct *p, i = p->state ? ffz(~p->state) + 1 : 0; psinfo->pr_state = i; - psinfo->pr_sname = (i < 0 || i > 5) ? '.' : "RSDTZW"[i]; + psinfo->pr_sname = (i > 5) ? '.' : "RSDTZW"[i]; psinfo->pr_zomb = psinfo->pr_sname == 'Z'; psinfo->pr_nice = task_nice(p); psinfo->pr_flag = p->flags; |