fix: only validate mounts for new share

Signed-off-by: Robin Appelman <robin@icewind.nl>

Author: icewind1991

apps/files_sharing/lib/Listener/SharesUpdatedListener.php

@@ -23,6 +23,7 @@
 use OCP\Share\Events\BeforeShareDeletedEvent;
 use OCP\Share\Events\ShareCreatedEvent;
 use OCP\Share\IManager;
+use OCP\Share\IShare;
 
 /**
  * Listen to various events that can change what shares a user has access to
@@ -52,9 +53,13 @@ public function handle(Event $event): void {
 		if ($event instanceof UserAddedEvent || $event instanceof UserRemovedEvent) {
 			$this->updateForUser($event->getUser());
 		}
-		if ($event instanceof ShareCreatedEvent || $event instanceof BeforeShareDeletedEvent) {
+		if ($event instanceof ShareCreatedEvent) {
+			$share = $event->getShare();
+			$shareTarget = $share->getTarget();
 			foreach ($this->shareManager->getUsersForShare($event->getShare()) as $user) {
-				$this->updateForUser($user);
+				$this->updateForShare($user, $share);
+				// Share target validation might have changed the target, restore it for the next user
+				$share->setTarget($shareTarget);
 			}
 		}
 	}
@@ -80,4 +85,21 @@ private function updateForUser(IUser $user): void {
 			}
 		}
 	}
+
+	private function updateForShare(IUser $user, IShare $share): void {
+		if (isset($this->updatedUsers[$user->getUID()])) {
+			return;
+		}
+		$this->updatedUsers[$user->getUID()] = true;
+
+		$cachedMounts = $this->userMountCache->getMountsForUser($user);
+		$mountPoints = array_map(fn (ICachedMountInfo $mount) => $mount->getMountPoint(), $cachedMounts);
+		$mountsByPath = array_combine($mountPoints, $cachedMounts);
+
+		$mountPoint = '/' . $user->getUID() . '/files/' . trim($share->getTarget(), '/') . '/';
+		$mountKey = $share->getNodeId() . '::' . $mountPoint;
+		if (!isset($cachedMounts[$mountKey])) {
+			$this->shareTargetValidator->verifyMountPoint($user, $share, $mountsByPath, [$share]);
+		}
+	}
 }