From 19aeffa8b56e7bcc31c773e64c7519733f7426e2 Mon Sep 17 00:00:00 2001
From: Robin Appelman <robin@icewind.nl>
Date: Thu, 3 Oct 2024 15:12:09 +0200
Subject: [PATCH] fix: better error when trying to move a shared mount into a
 different storage

Signed-off-by: Robin Appelman <robin@icewind.nl>
---
 lib/private/Files/View.php | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/lib/private/Files/View.php b/lib/private/Files/View.php
index 80679a9481fa5..f99bfdef64dc5 100644
--- a/lib/private/Files/View.php
+++ b/lib/private/Files/View.php
@@ -26,6 +26,7 @@
 use OCP\Files\InvalidPathException;
 use OCP\Files\Mount\IMountPoint;
 use OCP\Files\NotFoundException;
+use OCP\Files\NotPermittedException;
 use OCP\Files\ReservedWordException;
 use OCP\IUser;
 use OCP\IUserManager;
@@ -751,16 +752,18 @@ public function rename($source, $target) {
 					$mount2 = $this->getMount($target);
 					$storage1 = $mount1->getStorage();
 					$storage2 = $mount2->getStorage();
+					$sourceParent = dirname($source);
+					$targetParent = dirname($target);
 					$internalPath1 = $mount1->getInternalPath($absolutePath1);
 					$internalPath2 = $mount2->getInternalPath($absolutePath2);
 
-					$this->changeLock($source, ILockingProvider::LOCK_EXCLUSIVE, true);
+					$this->changeLock($source, ILockingProvider::LOCK_EXCLUSIVE, true);s
 					try {
 						$this->changeLock($target, ILockingProvider::LOCK_EXCLUSIVE, true);
 
 						if ($internalPath1 === '') {
 							if ($mount1 instanceof MoveableMount) {
-								$sourceParentMount = $this->getMount(dirname($source));
+								$sourceParentMount = $this->getMount($sourceParent);
 								if ($sourceParentMount === $mount2 && $this->targetIsNotShared($targetUser, $absolutePath2)) {
 									/**
 									 * @var \OC\Files\Mount\MountPoint | \OC\Files\Mount\MoveableMount $mount1
@@ -769,7 +772,7 @@ public function rename($source, $target) {
 									$result = $mount1->moveMount($absolutePath2);
 									$manager->moveMount($sourceMountPoint, $mount1->getMountPoint());
 								} else {
-									$result = false;
+									throw new NotPermittedException("Not allowed to move $source to $target as $sourceParent is not the same storage as $targetParent");
 								}
 							} else {
 								$result = false;