fix: add some extra checks for getMountsForPath arguments

Signed-off-by: Robin Appelman <[email protected]>

Author: icewind1991

lib/private/Files/Config/MountProviderCollection.php

@@ -84,9 +84,12 @@ public function getMountsForUser(IUser $user): array {
 	}
 
 	/**
-	 * @param MountProviderArgs[] $mountProviderArgs
-	 * @return array<string, IMountPoint> IMountPoint array indexed by mount
-	 *                                    point.
+	 * The caller is responsible to ensure that all provided MountProviderArgs
+	 * are for the same user.
+	 * And that the `$providerClass` implements IPartialMountProvider.
+	 *
+	 * @param list<MountProviderArgs> $mountProviderArgs
+	 * @return array<string, IMountPoint> IMountPoint array indexed by mount point.
 	 */
 	public function getUserMountsFromProviderByPath(
 		string $providerClass,
@@ -98,14 +101,16 @@ public function getUserMountsFromProviderByPath(
 		if ($provider === null) {
 			return [];
 		}
+		if (count($mountProviderArgs) === 0) {
+			return [];
+		}
 
-		if (!is_a($providerClass, IPartialMountProvider::class, true)) {
+		if (!$provider instanceof IPartialMountProvider) {
 			throw new \LogicException(
 				'Mount provider does not support partial mounts'
 			);
 		}
 
-		/** @var IPartialMountProvider $provider */
 		return $provider->getMountsForPath(
 			$path,
 			$forChildren,

lib/private/Files/SetupManager.php

@@ -600,7 +600,7 @@ public function setupForPath(string $path, bool $includeChildren = false): void
 				}
 				$this->setupMountProviderPaths[$mountPoint] = self::SETUP_WITH_CHILDREN;
 				foreach ($authoritativeCachedMounts as $providerClass => $cachedMounts) {
-					$providerArgs = array_filter(array_map(
+					$providerArgs = array_values(array_filter(array_map(
 						static function (ICachedMountInfo $info) use ($rootsMetadata) {
 							$rootMetadata = $rootsMetadata[$info->getRootId()] ?? null;
 
@@ -609,7 +609,7 @@ static function (ICachedMountInfo $info) use ($rootsMetadata) {
 								: null;
 						},
 						$cachedMounts
-					));
+					)));
 					$authoritativeMounts[] = $this->mountProviderCollection->getUserMountsFromProviderByPath(
 						$providerClass,
 						$path,