[PATCH] pw_scan: remove checks for (u|g)ids > USHRT_MAX.
Remove the check and simplify the logic for checking
UID/GID validity in `pw_scan()` by calling `strtonum`
instead of `strtoul`.
I ran into this because I use a non-default UID/GID
numbering scheme where both are typically greater
than 2^16 for normal users. The width of UIDs has
been 32 bits since before 4.4BSD in 1994, almost 27
years ago; if larger UIDs were going to show up as
a problem it would have already happened.
Updated by dancrossnyc 6 months ago
A gentle ping on this? I should note that FreeBSD effectively disabled this check in 2009: https://cgit.freebsd.org/src/commit/lib/libc/gen/pw_scan.c?id=764eca2591d4b1b0fdb45e4e839b1163bf76219f
- Status changed from New to Closed
I've pushed it (with some minor adjustments in manual pages). Thanks!