fix: address review

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

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/console.php b/console.php
@@ -42,7 +42,10 @@ function exceptionHandler($exception) {
 	}
 
 	$user = posix_getuid();
-	$userName = posix_getpwuid($user) !== false ? posix_getpwuid($user)['name'] : '';
+	$userNameArray = posix_getpwuid($user);
+	if ($userNameArray !== false) {
+		$userName = $userNameArray['name'];
+	}
 	$configUser = fileowner(OC::$configDir . 'config.php');
 	$configuredUser = $config->getSystemValueString('php.user', '');
 	if ($user !== $configUser && $userName !== $configuredUser) {

diff --git a/cron.php b/cron.php
@@ -130,7 +130,10 @@
 		}
 
 		$user = posix_getuid();
-		$userName = posix_getpwuid($user) !== false ? posix_getpwuid($user)['name'] : '';
+		$userNameArray = posix_getpwuid($user);
+		if ($userNameArray !== false)) {
+			$userName = $userNameArray['name'];
+		}
 		$configUser = fileowner(OC::$configDir . 'config.php');
 		$configuredUser = $config->getSystemValueString('php.user', '');
 		if ($user !== $configUser && $userName !== $configuredUser) {