diff options
| author | Konstantin Belousov <kib@FreeBSD.org> | 2024-04-29 18:51:53 +0000 |
|---|---|---|
| committer | Konstantin Belousov <kib@FreeBSD.org> | 2024-04-30 07:06:52 +0000 |
| commit | 1e01650a787028fd558e8cb709c232e335fc2cd9 (patch) | |
| tree | fe1872abdfc980d9849aaa60718151d09713fb02 | |
| parent | 11d79c4756b7c85faf7c4a268c301a41ce0887b9 (diff) | |
| download | src-1e01650a787028fd558e8cb709c232e335fc2cd9.tar.gz src-1e01650a787028fd558e8cb709c232e335fc2cd9.zip | |
kcmp_pget(): add an assert that we did not hold the current process
Tested by: pho
Sponsored by: The FreeBSD Foundation
MFC after: 3 days
| -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 b926f98892a5..cc1126ac4dc4 100644 --- a/sys/kern/sys_generic.c +++ b/sys/kern/sys_generic.c @@ -2092,11 +2092,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 |