diff options
| author | jake <jake@FreeBSD.org> | 2000-11-22 07:42:04 +0000 |
|---|---|---|
| committer | jake <jake@FreeBSD.org> | 2000-11-22 07:42:04 +0000 |
| commit | 0c0be4e82677a2d8fb49b7a70676bf359e7360b0 (patch) | |
| tree | a0c32260edfde6d861e66b11172147608684043e /sys/kern/kern_exit.c | |
| parent | 1b544875743d53491a096b0c757bf30ce73bb83b (diff) | |
| download | FreeBSD-src-0c0be4e82677a2d8fb49b7a70676bf359e7360b0.zip FreeBSD-src-0c0be4e82677a2d8fb49b7a70676bf359e7360b0.tar.gz | |
Protect the following with a lockmgr lock:
allproc
zombproc
pidhashtbl
proc.p_list
proc.p_hash
nextpid
Reviewed by: jhb
Obtained from: BSD/OS and netbsd
Diffstat (limited to 'sys/kern/kern_exit.c')
| -rw-r--r-- | sys/kern/kern_exit.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/sys/kern/kern_exit.c b/sys/kern/kern_exit.c index 489633a..9e5d488 100644 --- a/sys/kern/kern_exit.c +++ b/sys/kern/kern_exit.c @@ -264,11 +264,13 @@ exit1(p, rv) * Remove proc from allproc queue and pidhash chain. * Place onto zombproc. Unlink from parent's child list. */ + lockmgr(&allproc_lock, LK_EXCLUSIVE, NULL, CURPROC); LIST_REMOVE(p, p_list); LIST_INSERT_HEAD(&zombproc, p, p_list); p->p_stat = SZOMB; LIST_REMOVE(p, p_hash); + lockmgr(&allproc_lock, LK_RELEASE, NULL, CURPROC); q = LIST_FIRST(&p->p_children); if (q) /* only need this if any child is S_ZOMB */ @@ -510,7 +512,9 @@ loop: * Unlink it from its process group and free it. */ leavepgrp(p); + lockmgr(&allproc_lock, LK_EXCLUSIVE, NULL, CURPROC); LIST_REMOVE(p, p_list); /* off zombproc */ + lockmgr(&allproc_lock, LK_RELEASE, NULL, CURPROC); LIST_REMOVE(p, p_sibling); if (--p->p_procsig->ps_refcnt == 0) { |
