diff options
| author | nectar <nectar@FreeBSD.org> | 2003-08-07 16:42:27 +0000 |
|---|---|---|
| committer | nectar <nectar@FreeBSD.org> | 2003-08-07 16:42:27 +0000 |
| commit | df9de6c5cddb086d8d623c55d1cf800864f9fb29 (patch) | |
| tree | b243f33137fa414f8c884b0de9a58ef0bf90260f /sys/kern/sysv_msg.c | |
| parent | 0e7767264fa208c91b875189149b064c580b630a (diff) | |
| download | FreeBSD-src-df9de6c5cddb086d8d623c55d1cf800864f9fb29.zip FreeBSD-src-df9de6c5cddb086d8d623c55d1cf800864f9fb29.tar.gz | |
Update some argument-documenting comments to match reality.
Add an explicit range check to those same arguments to reduce risk of
cardiac arrest in future code readers.
Diffstat (limited to 'sys/kern/sysv_msg.c')
| -rw-r--r-- | sys/kern/sysv_msg.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/sys/kern/sysv_msg.c b/sys/kern/sysv_msg.c index 51a0c5c..b138d98 100644 --- a/sys/kern/sysv_msg.c +++ b/sys/kern/sysv_msg.c @@ -285,7 +285,7 @@ msgsys(td, uap) struct thread *td; /* XXX actually varargs. */ struct msgsys_args /* { - u_int which; + int which; int a2; int a3; int a4; @@ -297,7 +297,8 @@ msgsys(td, uap) if (!jail_sysvipc_allowed && jailed(td->td_ucred)) return (ENOSYS); - if (uap->which >= sizeof(msgcalls)/sizeof(msgcalls[0])) + if (uap->which < 0 || + uap->which >= sizeof(msgcalls)/sizeof(msgcalls[0])) return (EINVAL); error = (*msgcalls[uap->which])(td, &uap->a2); return (error); |
