]> git.cameronkatri.com Git - pw-darwin.git/commitdiff
Allow the "@" and "!" characters in passwd file GECOS fields.
authorDavid Bright <dab@FreeBSD.org>
Thu, 1 Mar 2018 17:47:28 +0000 (17:47 +0000)
committerDavid Bright <dab@FreeBSD.org>
Thu, 1 Mar 2018 17:47:28 +0000 (17:47 +0000)
Two PRs (152084 & 210187) request allowing the "@" and/or "!"
characters in the passwd file GECOS field. The man page for pw does
not mention that those characters are disallowed, Linux supports those
characters in this field, and the "@" character in particular would be
useful for storing email addresses in that field.

PR: 152084, 210187
Submitted by: jschauma@netmeister.org, Dave Cottlehuber <dch@freebsd.org>
Reported by: jschauma@netmeister.org, Dave Cottlehuber <dch@freebsd.org>
Reviewed by: delphij (secteam), vangyzen
MFC after: 1 week
Sponsored by: Dell EMC
Differential Revision: https://reviews.freebsd.org/D14519

pw/pw_user.c

index 62a1613322db9a6f17878eb4612733658a1285d7..148239ebd2b5f8a9e48af738b6bf34125cf60482 100644 (file)
@@ -632,7 +632,7 @@ pw_checkname(char *name, int gecos)
        reject = 0;
        if (gecos) {
                /* See if the name is valid as a gecos (comment) field. */
-               badchars = ":!@";
+               badchars = ":";
                showtype = "gecos field";
        } else {
                /* See if the name is valid as a userid or group. */