lib/chkname.c: is_valid_name(): Use ispfchar() to simplify

In the first case, we can do the transformation because a few lines above, we explicitly reject a name starting with a '-'. In the second case, we're obviously using ispfchar() instead of its pattern. Signed-off-by: Alejandro Colomar <[email protected]>
shadow-maint · Dec 23, 2024 · 2ff431a · 2ff431a
1 parent e75e6c4
commit 2ff431a
Showing 1 changed file with 2 additions and 10 deletions.
diff --git a/lib/chkname.c b/lib/chkname.c
@@ -84,10 +84,7 @@ is_valid_name(const char *name)
          * sake of Samba 3.x "add machine script"
          */
 
-	if (!(isalnum(*name) ||
-	      *name == '_' ||
-	      *name == '.'))
-	{
+	if (!ispfchar(*name)) {
 		errno = EILSEQ;
 		return false;
 	}
@@ -96,12 +93,7 @@ is_valid_name(const char *name)
 		if (streq(name, "$"))  // Samba
 			return true;
 
-		if (!(isalnum(*name) ||
-		      *name == '_' ||
-		      *name == '.' ||
-		      *name == '-'
-		     ))
-		{
+		if (!ispfchar(*name)) {
 			errno = EILSEQ;
 			return false;
 		}