Re: CVS commit: [netbsd-7] src/sbin/raidctl

[christos%astron.com@localhost (Christos Zoulas)]

In article <20150608204801.C85E79A%cvs.netbsd.org@localhost>,
Soren Jacobsen <source-changes-d%NetBSD.org@localhost> wrote:
>-=-=-=-=-=-
>
>Module Name:	src
>Committed By:	snj
>Date:		Mon Jun  8 20:48:01 UTC 2015
>
>Modified Files:
>	src/sbin/raidctl [netbsd-7]: raidctl.c
>
>Log Message:
>Pull up following revision(s) (requested by manu in ticket #827):
>	sbin/raidctl/raidctl.c: revision 1.58
>Better sanity check numbers given to raidctl(8)
>Replace atoi(3) by strtol(3), and check that numbers are valid,
>positive, and in int32_t range. The previous lack of check could
>silently lead to the same serial being set to all RAID volumes
>for instance because given numbers were bigger than INT_MAX. The
>consequence is in an awful mess when RAIDframe would mix volumes...

This has typos and it is easier to use strtou like in the next version.

christos