fix: only validate mounts for new share

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

Author: icewind1991

apps/files_sharing/lib/Listener/SharesUpdatedListener.php

@@ -22,6 +22,7 @@
 use OCP\Share\Events\ShareCreatedEvent;
 use OCP\Share\Events\ShareTransferredEvent;
 use OCP\Share\IManager;
+use OCP\Share\IShare;
 
 /**
  * Listen to various events that can change what shares a user has access to
@@ -47,13 +48,15 @@ public function handle(Event $event): void {
 		if ($event instanceof UserAddedEvent || $event instanceof UserRemovedEvent) {
 			$this->updateForUser($event->getUser());
 		}
-		if (
-			$event instanceof ShareCreatedEvent
-			|| $event instanceof BeforeShareDeletedEvent
-			|| $event instanceof ShareTransferredEvent
-		) {
-			foreach ($this->shareManager->getUsersForShare($event->getShare()) as $user) {
-				$this->updateForUser($user);
+		if ($event instanceof ShareCreatedEvent || $event instanceof ShareTransferredEvent) {
+			$share = $event->getShare();
+			$shareTarget = $share->getTarget();
+			foreach ($this->shareManager->getUsersForShare($share) as $user) {
+				if ($share->getSharedBy() !== $user->getUID()) {
+					$this->updateForShare($user, $share);
+					// Share target validation might have changed the target, restore it for the next user
+					$share->setTarget($shareTarget);
+				}
 			}
 		}
 	}
@@ -82,4 +85,12 @@ private function updateForUser(IUser $user): void {
 
 		unset($this->inUpdate[$user->getUID()]);
 	}
+
+	private function updateForShare(IUser $user, IShare $share): void {
+		$cachedMounts = $this->userMountCache->getMountsForUser($user);
+		$mountPoints = array_map(fn (ICachedMountInfo $mount) => $mount->getMountPoint(), $cachedMounts);
+		$mountsByPath = array_combine($mountPoints, $cachedMounts);
+
+		$this->shareTargetValidator->verifyMountPoint($user, $share, $mountsByPath, [$share]);
+	}
 }