summaryrefslogtreecommitdiffstats
path: root/adduser
diff options
context:
space:
mode:
authorYaroslav Tykhiy <ytykhiy@gmail.com>2002-01-28 17:30:12 +0000
committerYaroslav Tykhiy <ytykhiy@gmail.com>2002-01-28 17:30:12 +0000
commit980c0e8f0f4043752a535aa6b5fc181e9f7daca3 (patch)
treeac857bae9ce4cc009703af37185bff11004139fe /adduser
parentd43f593f9ea43553b1d83f258996e0709161ab9b (diff)
downloadpw-darwin-980c0e8f0f4043752a535aa6b5fc181e9f7daca3.tar.gz
pw-darwin-980c0e8f0f4043752a535aa6b5fc181e9f7daca3.tar.zst
pw-darwin-980c0e8f0f4043752a535aa6b5fc181e9f7daca3.zip
Disallow adding duplicate roots, toors, or other users w/uid 0.
Previously, a truth check instead if defined() check erroneously allowed that. PR: bin/8745
Diffstat (limited to 'adduser')
-rw-r--r--adduser/adduser.perl2
1 files changed, 1 insertions, 1 deletions
diff --git a/adduser/adduser.perl b/adduser/adduser.perl
index 2728eff..2aa9c31 100644
--- a/adduser/adduser.perl
+++ b/adduser/adduser.perl
@@ -345,7 +345,7 @@ sub new_users_name_valid {
warn "Username doesn't match the regexp /$usernameregexp/\a\n";
}
return 0;
- } elsif ($username{$name}) {
+ } elsif (defined($username{$name})) {
warn "Username ``$name'' already exists!\a\n"; return 0;
}
return 1;