diff options
| author | Mark Johnston <markj@FreeBSD.org> | 2023-12-26 01:42:04 +0000 |
|---|---|---|
| committer | Mark Johnston <markj@FreeBSD.org> | 2023-12-26 02:04:00 +0000 |
| commit | f450277f7a608f26624384e046c1987490c51296 (patch) | |
| tree | f29978c7e576444f1dfff6fed1ef16205a01bc84 | |
| parent | abbc260f74b203646dd108ad4038627fcfb0ded7 (diff) | |
| download | src-f450277f7a608f26624384e046c1987490c51296.tar.gz src-f450277f7a608f26624384e046c1987490c51296.zip | |
thread: Ignore errors when copying out during thr_exit()
It does not seem reasonable to return to userspace after calling
umtx_thread_exit().
This is in preparation for annotating copyin() and related functions
with __result_use_check.
Reviewed by: olce, kib
MFC after: 1 week
Differential Revision: https://reviews.freebsd.org/D43143
| -rw-r--r-- | sys/kern/kern_thr.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sys/kern/kern_thr.c b/sys/kern/kern_thr.c index 544479fc9f13..e2f1b6baca4b 100644 --- a/sys/kern/kern_thr.c +++ b/sys/kern/kern_thr.c @@ -315,8 +315,8 @@ sys_thr_exit(struct thread *td, struct thr_exit_args *uap) /* Signal userland that it can free the stack. */ if ((void *)uap->state != NULL) { - suword_lwpid(uap->state, 1); - kern_umtx_wake(td, uap->state, INT_MAX, 0); + (void)suword_lwpid(uap->state, 1); + (void)kern_umtx_wake(td, uap->state, INT_MAX, 0); } return (kern_thr_exit(td)); |