On Fri, 16 Mar 2018 14:23:29 -0500, Scott Cheloha wrote:

> Because we increment the timearg pointer, shutdown(8) accepts
> stuff like
>
>       shutdown ++10
>
> as strtonum(3) allows a single leading '+'.
>
> Not incrementing leaves the offset in a valid format for strtonum(3)
> and disallows stuff like the above.
>
> While here:
>
>   - time_t is 64-bit, so the maximum offset in minutes is LLONG_MAX / 60
>
>   - use errstr to say what was wrong with timearg, as we do elsewhere
>     with strtonum(3).

OK millert@

 - todd

Reply via email to