CVSROOT:        /cvs
Module name:    src
Changes by:     k...@cvs.openbsd.org    2020/08/08 06:38:21

Modified files:
        sbin/ifconfig  : brconfig.c 

Log message:
Replace strto*l(3) with strtonum(3)

Some bridge options allow values bigger than the corresponding datatype
and thus pass overflowed values to the kernel, not all options to range
checks and setting invalid "portno" complains about "portidx" values.

Use the same, simpler strtonum() idiom across all options with consistent
error messages and proper [U]INT*_MAX defines as max values to match the
datatype and all option handlers to be consistent (in less code).

Keep deferring option dependent min/max value checking to the ioctl(2)
interface, i.e. values documented in ifconfig(8) are not duplicated in
strtonum() calls.

OK millert

Reply via email to