diff options
author | ed <ed@FreeBSD.org> | 2009-02-26 15:51:54 +0000 |
---|---|---|
committer | ed <ed@FreeBSD.org> | 2009-02-26 15:51:54 +0000 |
commit | b3ddcfe1f75495bb47bf5eaba6333e4822eadae7 (patch) | |
tree | d89376ead0217f0b0f64e8d12308efc93860e35a /sys/kern/kern_exec.c | |
parent | fb70a002f2760a40274da9dacaa3edd815884665 (diff) | |
download | FreeBSD-src-b3ddcfe1f75495bb47bf5eaba6333e4822eadae7.zip FreeBSD-src-b3ddcfe1f75495bb47bf5eaba6333e4822eadae7.tar.gz |
Remove even more unneeded variable assignments.
kern_time.c:
- Unused variable `p'.
kern_thr.c:
- Variable `error' is always caught immediately, so no reason to
initialize it. There is no way that error != 0 at the end of
create_thread().
kern_sig.c:
- Unused variable `code'.
kern_synch.c:
- `rval' is always assigned in all different cases.
kern_rwlock.c:
- `v' is always overwritten with RW_UNLOCKED further on.
kern_malloc.c:
- `size' is always initialized with the proper value before being used.
kern_exit.c:
- `error' is always caught and returned immediately. abort2() never
returns a non-zero value.
kern_exec.c:
- `len' is always assigned inside the if-statement right below it.
tty_info.c:
- `td' is always overwritten by FOREACH_THREAD_IN_PROC().
Found by: LLVM's scan-build
Diffstat (limited to 'sys/kern/kern_exec.c')
-rw-r--r-- | sys/kern/kern_exec.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/sys/kern/kern_exec.c b/sys/kern/kern_exec.c index c89c9a6..36a9fc8 100644 --- a/sys/kern/kern_exec.c +++ b/sys/kern/kern_exec.c @@ -591,7 +591,6 @@ interpret: len = min(ndp->ni_cnd.cn_namelen,MAXCOMLEN); bcopy(ndp->ni_cnd.cn_nameptr, p->p_comm, len); } else { - len = MAXCOMLEN; if (vn_commname(binvp, p->p_comm, MAXCOMLEN + 1) == 0) len = MAXCOMLEN; else { @@ -1061,8 +1060,6 @@ exec_copyin_args(struct image_args *args, char *fname, int error; size_t length; - error = 0; - bzero(args, sizeof(*args)); if (argv == NULL) return (EFAULT); |