From 9259c33814862711fc6dc6c07c171c31d1789cde Mon Sep 17 00:00:00 2001 From: Sheldon Hearn Date: Mon, 15 Nov 1999 16:45:37 +0000 Subject: Add to pwd_mkdb a -q option to silence warnings about large IDs. Add a suitably ominous warning in the manual page. The diff applied is not the one provided in the attributed PR. PR: 13344 Reviewed by: bde --- libc/gen/pw_scan.c | 15 +++++++++++---- libc/gen/pw_scan.h | 4 ++++ 2 files changed, 15 insertions(+), 4 deletions(-) (limited to 'libc/gen') diff --git a/libc/gen/pw_scan.c b/libc/gen/pw_scan.c index 0ce9b1d..a24380f 100644 --- a/libc/gen/pw_scan.c +++ b/libc/gen/pw_scan.c @@ -56,6 +56,13 @@ static const char rcsid[] = #include "pw_scan.h" +/* + * Some software assumes that IDs are short. We should emit warnings + * for id's which can not be stored in a short, but we are more liberal + * by default, warning for IDs greater than USHRT_MAX. + */ +int pw_big_ids_warning = 1; + int pw_scan(bp, pw) char *bp; @@ -89,8 +96,8 @@ pw_scan(bp, pw) warnx("root uid should be 0"); return (0); } - if (id > USHRT_MAX) { - warnx("%s > max uid value (%d)", p, USHRT_MAX); + if (pw_big_ids_warning && id > USHRT_MAX) { + warnx("%s > max uid value (%u)", p, USHRT_MAX); /*return (0);*/ /* THIS SHOULD NOT BE FATAL! */ } pw->pw_uid = id; @@ -99,8 +106,8 @@ pw_scan(bp, pw) goto fmt; if(p[0]) pw->pw_fields |= _PWF_GID; id = atol(p); - if (id > USHRT_MAX) { - warnx("%s > max gid value (%d)", p, USHRT_MAX); + if (pw_big_ids_warning && id > USHRT_MAX) { + warnx("%s > max gid value (%u)", p, USHRT_MAX); /* return (0); This should not be fatal! */ } pw->pw_gid = id; diff --git a/libc/gen/pw_scan.h b/libc/gen/pw_scan.h index d1d4bc1..2519bd4 100644 --- a/libc/gen/pw_scan.h +++ b/libc/gen/pw_scan.h @@ -31,6 +31,10 @@ * SUCH DAMAGE. * * @(#)pw_scan.h 8.1 (Berkeley) 4/1/94 + * + * $FreeBSD$ */ +extern int pw_big_ids_warning; + extern int pw_scan __P((char *, struct passwd *)); -- cgit v1.2.3-56-ge451