Fixed: FS#3603 - system.inc.php:1827 is_allowed_user regex typo?

Author: Till Brehm

interface/lib/classes/functions.inc.php

@@ -430,7 +430,7 @@ public function is_allowed_user($username, $restrict_names = false) {
 		$name_blacklist = array('root','ispconfig','vmail','getmail');
 		if(in_array($username,$name_blacklist)) return false;
 
-		if(preg_match('/^[a-zA-Z0-9\.\-]{1,32}$/', $username) == false) return false;
+		if(preg_match('/^[a-zA-Z0-9\.\-_]{1,32}$/', $username) == false) return false;
 
 		if($restrict_names == true && preg_match('/^web\d+$/', $username) == false) return false;
 
@@ -443,7 +443,7 @@ public function is_allowed_group($groupname, $restrict_names = false) {
 		$name_blacklist = array('root','ispconfig','vmail','getmail');
 		if(in_array($groupname,$name_blacklist)) return false;
 
-		if(preg_match('/^[a-zA-Z0-9\.\-]{1,32}$/', $groupname) == false) return false;
+		if(preg_match('/^[a-zA-Z0-9\.\-_]{1,32}$/', $groupname) == false) return false;
 
 		if($restrict_names == true && preg_match('/^client\d+$/', $groupname) == false) return false;
 

server/lib/classes/system.inc.php

@@ -1824,7 +1824,7 @@ public function is_allowed_user($username, $check_id = true, $restrict_names = f
 		$name_blacklist = array('root','ispconfig','vmail','getmail');
 		if(in_array($username,$name_blacklist)) return false;
 
-		if(preg_match('/^[a-zA-Z0-9\.\-]{1,32}$/', $username) == false) return false;
+		if(preg_match('/^[a-zA-Z0-9\.\-_]{1,32}$/', $username) == false) return false;
 
 		if($check_id && intval($this->getuid($username)) < $this->min_uid) return false;
 
@@ -1839,7 +1839,7 @@ public function is_allowed_group($groupname, $check_id = true, $restrict_names =
 		$name_blacklist = array('root','ispconfig','vmail','getmail');
 		if(in_array($groupname,$name_blacklist)) return false;
 
-		if(preg_match('/^[a-zA-Z0-9\.\-]{1,32}$/', $groupname) == false) return false;
+		if(preg_match('/^[a-zA-Z0-9\.\-_]{1,32}$/', $groupname) == false) return false;
 
 		if($check_id && intval($this->getgid($groupname)) < $this->min_gid) return false;