diff options
author | jhb <jhb@FreeBSD.org> | 2006-02-22 17:20:37 +0000 |
---|---|---|
committer | jhb <jhb@FreeBSD.org> | 2006-02-22 17:20:37 +0000 |
commit | d6902d680b5355d12a5cff6cc8777a9d20506082 (patch) | |
tree | 83a00beb9a6c297a91bb98288c559eb899cd4789 /sys | |
parent | d4a7c7535cf6e31dc7883e0239d15caf261363a0 (diff) | |
download | FreeBSD-src-d6902d680b5355d12a5cff6cc8777a9d20506082.zip FreeBSD-src-d6902d680b5355d12a5cff6cc8777a9d20506082.tar.gz |
Hold the proc lock while calling proc_sstep() since the function asserts
it and remove a PRELE() that didn't have a matching PHOLD(). The calling
code already has a PHOLD anyway.
MFC after: 1 week
Diffstat (limited to 'sys')
-rw-r--r-- | sys/fs/procfs/procfs_ctl.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/sys/fs/procfs/procfs_ctl.c b/sys/fs/procfs/procfs_ctl.c index 13334fd..65f39f6 100644 --- a/sys/fs/procfs/procfs_ctl.c +++ b/sys/fs/procfs/procfs_ctl.c @@ -245,9 +245,8 @@ out: * What does it mean to single step a threaded program? */ case PROCFS_CTL_STEP: - PROC_UNLOCK(p); error = proc_sstep(FIRST_THREAD_IN_PROC(p)); /* XXXKSE */ - PRELE(p); + PROC_UNLOCK(p); if (error) return (error); break; |