diff options
| author |   Baptiste Daroussin <bapt@FreeBSD.org> | 2015-12-02 22:35:25 +0000 |
|---|---|---|
| committer | Baptiste Daroussin <bapt@FreeBSD.org> | 2015-12-02 22:35:25 +0000 |
| commit | 37f903568b33e005010ceea4921219edbe23ba39 (patch) | |
| tree | 3a3613975222074634102105d1b5cf5fc9405847 | |
| parent | 42c74a30927e0f5a7c28809ed3df548911bf17a1 (diff) | |
| download | pw-darwin-37f903568b33e005010ceea4921219edbe23ba39.tar.gz pw-darwin-37f903568b33e005010ceea4921219edbe23ba39.tar.zst pw-darwin-37f903568b33e005010ceea4921219edbe23ba39.zip | |
pw_checkname since the beginning if too strict on GECOS field,
relax it a bit so gecos can be used to store multibytes data.
This was unseen before FreeBSD 10.2 as this validation function was motly unused
since FreeBSD 10.2 the usage of this function has been generalized to improve
validation.
Reported by: des
MFC after: 1 week
| -rw-r--r-- | pw/pw_user.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/pw/pw_user.c b/pw/pw_user.c index 61c2440..30a2749 100644 --- a/pw/pw_user.c +++ b/pw/pw_user.c @@ -642,7 +642,8 @@ pw_checkname(char *name, int gecos) } if (!reject) { while (*ch) { - if (strchr(badchars, *ch) != NULL || *ch < ' ' || + if (strchr(badchars, *ch) != NULL || + (!gecos && *ch < ' ') || *ch == 127) { reject = 1; break; |