Merge pull request #10939 from nextcloud/bugfix/10930/perm-checker-wrong-path

Fix path when app has wrong permission

1 files changed, 5 insertions, 6 deletions

diff --git a/settings/Controller/CheckSetupController.php b/settings/Controller/CheckSetupController.php
index 747e60c7cb2..9169808456f 100644
--- a/settings/Controller/CheckSetupController.php
+++ b/settings/Controller/CheckSetupController.php
@@ -544,18 +544,17 @@ Raw output
 	 */
 	protected function getAppDirsWithDifferentOwner(): array {
 		$currentUser = posix_getpwuid(posix_getuid());
-		$appDirsWithDifferentOwner = [];
+		$appDirsWithDifferentOwner = [[]];
 
 		foreach (OC::$APPSROOTS as $appRoot) {
 			if ($appRoot['writable'] === true) {
-				$appDirsWithDifferentOwner = array_merge(
-					$appDirsWithDifferentOwner,
-					$this->getAppDirsWithDifferentOwnerForAppRoot($currentUser, $appRoot)
-				);
+				$appDirsWithDifferentOwner[] = $this->getAppDirsWithDifferentOwnerForAppRoot($currentUser, $appRoot);
 			}
 		}
 
+		$appDirsWithDifferentOwner = array_merge(...$appDirsWithDifferentOwner);
 		sort($appDirsWithDifferentOwner);
+
 		return $appDirsWithDifferentOwner;
 	}
 
@@ -576,7 +575,7 @@ Raw output
 				$absAppPath = $appsPath . DIRECTORY_SEPARATOR . $fileInfo->getFilename();
 				$appDirUser = posix_getpwuid(fileowner($absAppPath));
 				if ($appDirUser !== $currentUser) {
-					$appDirsWithDifferentOwner[] = $absAppPath . DIRECTORY_SEPARATOR . $fileInfo->getFilename();
+					$appDirsWithDifferentOwner[] = $absAppPath;
 				}
 			}
 		}