Don't check aq64_minfree which is unsigned for negative values.

This fixes a tautological comparison warning.

Reviewed by:	rwatson
Differential Revision:	https://reviews.freebsd.org/D7682
This commit is contained in:
John Baldwin 2016-09-08 19:47:57 +00:00
parent c97a3872fe
commit b3db2736b1

View file

@ -299,12 +299,12 @@ sys_auditon(struct thread *td, struct auditon_args *uap)
case A_OLDSETQCTRL:
case A_SETQCTRL:
if (uap->length == sizeof(udata.au_qctrl64)) {
/* NB: aq64_minfree is unsigned unlike aq_minfree. */
if ((udata.au_qctrl64.aq64_hiwater > AQ_MAXHIGH) ||
(udata.au_qctrl64.aq64_lowater >=
udata.au_qctrl.aq_hiwater) ||
(udata.au_qctrl64.aq64_bufsz > AQ_MAXBUFSZ) ||
(udata.au_qctrl64.aq64_minfree > 100) ||
(udata.au_qctrl64.aq64_minfree < 0))
(udata.au_qctrl64.aq64_minfree > 100))
return (EINVAL);
audit_qctrl.aq_hiwater =
(int)udata.au_qctrl64.aq64_hiwater;