diff options
| author | mjg <mjg@FreeBSD.org> | 2012-11-13 22:01:25 +0000 |
|---|---|---|
| committer | mjg <mjg@FreeBSD.org> | 2012-11-13 22:01:25 +0000 |
| commit | 72315ca4845472c75a46a47dcd5cbf4da84c63a1 (patch) | |
| tree | c0dc36214c6be9429c2ebc566b2f14167e45e3f9 /sys/kern/kern_proc.c | |
| parent | 56fb6a5bbf100c74b469b2cd20a535f51e522585 (diff) | |
| download | FreeBSD-src-72315ca4845472c75a46a47dcd5cbf4da84c63a1.zip FreeBSD-src-72315ca4845472c75a46a47dcd5cbf4da84c63a1.tar.gz | |
enterpgrp: get rid of pgrp2 variable and use KASSERT directly on pgfind result.
pgrp2 was used only for debugging, but pgrp2 = pgfind(..) was present in compiled code even for kernels without INVARIANTS
Approved by: trasz (mentor)
MFC after: 1 week
Diffstat (limited to 'sys/kern/kern_proc.c')
| -rw-r--r-- | sys/kern/kern_proc.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/sys/kern/kern_proc.c b/sys/kern/kern_proc.c index 8ee043e..d6db62f 100644 --- a/sys/kern/kern_proc.c +++ b/sys/kern/kern_proc.c @@ -421,17 +421,13 @@ enterpgrp(p, pgid, pgrp, sess) struct pgrp *pgrp; struct session *sess; { - struct pgrp *pgrp2; sx_assert(&proctree_lock, SX_XLOCKED); KASSERT(pgrp != NULL, ("enterpgrp: pgrp == NULL")); KASSERT(p->p_pid == pgid, ("enterpgrp: new pgrp and pid != pgid")); - - pgrp2 = pgfind(pgid); - - KASSERT(pgrp2 == NULL, + KASSERT(pgfind(pgid) == NULL, ("enterpgrp: pgrp with pgid exists")); KASSERT(!SESS_LEADER(p), ("enterpgrp: session leader attempted setpgrp")); |
