diff options
| author | Konstantin Belousov <kib@FreeBSD.org> | 2024-04-29 18:51:53 +0000 |
|---|---|---|
| committer | Konstantin Belousov <kib@FreeBSD.org> | 2024-05-03 00:26:25 +0000 |
| commit | 6b667b46ab192d00a280ffdab60ff523a88495e4 (patch) | |
| tree | b936fe5464ec49ac5bde93f5b2a0761710d33114 | |
| parent | 91e0b44c159f71ea3d27297b250bdcd318c0fe89 (diff) | |
| download | src-6b667b46ab192d00a280ffdab60ff523a88495e4.tar.gz src-6b667b46ab192d00a280ffdab60ff523a88495e4.zip | |
kcmp_pget(): add an assert that we did not hold the current process
(cherry picked from commit 1e01650a787028fd558e8cb709c232e335fc2cd9)
| -rw-r--r-- | sys/kern/sys_generic.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/sys/kern/sys_generic.c b/sys/kern/sys_generic.c index 32f137f0cb4c..f2d099f7a0c4 100644 --- a/sys/kern/sys_generic.c +++ b/sys/kern/sys_generic.c @@ -2018,11 +2018,15 @@ kcmp_cmp(uintptr_t a, uintptr_t b) static int kcmp_pget(struct thread *td, pid_t pid, struct proc **pp) { + int error; + if (pid == td->td_proc->p_pid) { *pp = td->td_proc; return (0); } - return (pget(pid, PGET_CANDEBUG | PGET_NOTWEXIT | PGET_HOLD, pp)); + error = pget(pid, PGET_CANDEBUG | PGET_NOTWEXIT | PGET_HOLD, pp); + MPASS(*pp != td->td_proc); + return (error); } int |