diff options
| author | Konstantin Belousov <kib@FreeBSD.org> | 2026-04-25 09:46:50 +0000 |
|---|---|---|
| committer | Konstantin Belousov <kib@FreeBSD.org> | 2026-04-27 03:23:19 +0000 |
| commit | 912f9dfca451e359dda7cdf45539b7c19764f54d (patch) | |
| tree | 9d3057994570773579e6333d4c23026dc62b075d | |
| parent | e378d97eeb2b8961f312cc04d4751579a7b70741 (diff) | |
amd64: ia32_fetch_syscall_args() does not need to check params != NULL
Whatever params pointer is, it does not matter. copyin() handles any
values. In fact, params cannot be ever NULL.
Reviewed by: markj
Sponsored by: The FreeBSD Foundation
MFC after: 1 week
Differential revision: https://reviews.freebsd.org/D56630
| -rw-r--r-- | sys/amd64/ia32/ia32_syscall.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sys/amd64/ia32/ia32_syscall.c b/sys/amd64/ia32/ia32_syscall.c index c61587a93d1f..85e3d8f8e920 100644 --- a/sys/amd64/ia32/ia32_syscall.c +++ b/sys/amd64/ia32/ia32_syscall.c @@ -187,7 +187,7 @@ ia32_fetch_syscall_args(struct thread *td) else sa->callp = &p->p_sysent->sv_table[sa->code]; - if (params != NULL && sa->callp->sy_narg != 0) + if (sa->callp->sy_narg != 0) error = copyin(params, (caddr_t)args, (u_int)(sa->callp->sy_narg * sizeof(int))); else |