diff options
| author | John Baldwin <jhb@FreeBSD.org> | 2006-02-22 17:20:37 +0000 |
|---|---|---|
| committer | John Baldwin <jhb@FreeBSD.org> | 2006-02-22 17:20:37 +0000 |
| commit | 7a61c1a3cb2f87030e284e4ab6b9b29fa1721415 (patch) | |
| tree | 83a00beb9a6c297a91bb98288c559eb899cd4789 /sys/fs/procfs | |
| parent | 3df4e47b1e0ebaa2ae04b752662482c84d64aded (diff) | |
| download | src-7a61c1a3cb2f87030e284e4ab6b9b29fa1721415.tar.gz src-7a61c1a3cb2f87030e284e4ab6b9b29fa1721415.zip | |
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
Notes
Notes:
svn path=/head/; revision=155918
Diffstat (limited to 'sys/fs/procfs')
| -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 13334fdece6c..65f39f6caa1d 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; |