fix: track already set up paths to avoid loops

Signed-off-by: Salvatore Martire <[email protected]>

Author: salmart-dev

lib/private/Files/SetupManager.php

@@ -72,6 +72,12 @@ class SetupManager {
 	 * @var array<string, class-string<IMountProvider>[]>
 	 */
 	private array $setupUserMountProviders = [];
+	/**
+	 * An array of paths that have already been set up
+	 *
+	 * @var array<string, class-string<IMountProvider>[]>
+	 */
+	private array $setupMountProviderPaths = [];
 	private ICache $cache;
 	private bool $listeningForProviders;
 	private array $fullSetupRequired = [];
@@ -512,7 +518,7 @@ public function setupForPathAuthoritative(string $path, bool $includeChildren =
 			return;
 		}
 
-		if ($this->isSetupComplete($user)) {
+		if ($this->isSetupComplete($user) || isset($this->setupMountProviderPaths[$path])) {
 			return;
 		}
 
@@ -633,6 +639,7 @@ public function tearDown() {
 		$this->setupUsers = [];
 		$this->setupUsersComplete = [];
 		$this->setupUserMountProviders = [];
+		$this->setupMountProviderPaths = [];
 		$this->fullSetupRequired = [];
 		$this->rootSetup = false;
 		$this->mountManager->clear();
@@ -724,6 +731,10 @@ public function getMountsFromProviders(
 			}
 
 			if (is_a($providerClass, IPartialMountProvider::class, true)) {
+				if (isset($this->setupMountProviderPaths[$path])) {
+					continue; // skip already set up paths
+				}
+				$this->setupMountProviderPaths[$path] = true;
 				// mount provider capable of returning mount-points specific to
 				// this path
 				$mounts[] = $this->mountProviderCollection