fix(posix_getpwuid): handle false return value

Signed-off-by: Simon L <szaimen@e.mail.de>
nextcloud · May 22, 2024 · f49d3f0 · f49d3f0
1 parent 087ba07
commit f49d3f0
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/console.php b/console.php
@@ -42,10 +42,10 @@ function exceptionHandler($exception) {
 	}
 
 	$user = posix_getuid();
-	$userName = posix_getpwuid($user)['name'];
+	$userName = posix_getpwuid($user) !== false ? posix_getpwuid($user)['name'] : '';
 	$configUser = fileowner(OC::$configDir . 'config.php');
 	$configuredUser = $config->getSystemValueString('php.user', '');
-	if ($user !== $configUser && $userName !== $configuredUser) {
+	if ($user !== $configUser && $configuredUser === '' && $userName !== $configuredUser) {
 		echo "Console has to be executed with the user that owns the file config/config.php" . PHP_EOL;
 		echo "Current user id: " . $user . PHP_EOL;
 		echo "Owner id of config.php: " . $configUser . PHP_EOL;

diff --git a/cron.php b/cron.php
@@ -130,10 +130,10 @@
 		}
 
 		$user = posix_getuid();
-		$userName = posix_getpwuid($user)['name'];
+		$userName = posix_getpwuid($user) !== false ? posix_getpwuid($user)['name'] : '';
 		$configUser = fileowner(OC::$configDir . 'config.php');
 		$configuredUser = $config->getSystemValueString('php.user', '');
-		if ($user !== $configUser && $userName !== $configuredUser) {
+		if ($user !== $configUser && $configuredUser === '' && $userName !== $configuredUser) {
 			echo "Console has to be executed with the user that owns the file config/config.php" . PHP_EOL;
 			echo "Current user id: " . $user . PHP_EOL;
 			echo "Owner id of config.php: " . $configUser . PHP_EOL;