Set the exclusive file lock earlier.

owncloud · Aug 2, 2018 · ddfdcfd · ddfdcfd
1 parent 3eee971
commit ddfdcfd
Showing 1 changed file with 10 additions and 9 deletions.
diff --git a/apps/dav/lib/Connector/Sabre/File.php b/apps/dav/lib/Connector/Sabre/File.php
@@ -188,6 +188,16 @@ public function put($data) {
 				// because we have no clue about the cause we can only throw back a 500/Internal Server Error
 				throw new Exception('Could not write file contents');
 			}
+
+			try {
+				$this->changeLock(ILockingProvider::LOCK_EXCLUSIVE);
+			} catch (LockedException $e) {
+				if ($needsPartFile) {
+					$partStorage->unlink($internalPartPath);
+				}
+				throw new FileLocked($e->getMessage(), $e->getCode(), $e);
+			}
+
 			list($count, $result) = \OC_Helper::streamCopy($data, $target);
 			\fclose($target);
 
@@ -227,15 +237,6 @@ public function put($data) {
 				$run = true;
 			}
 
-			try {
-				$this->changeLock(ILockingProvider::LOCK_EXCLUSIVE);
-			} catch (LockedException $e) {
-				if ($needsPartFile) {
-					$partStorage->unlink($internalPartPath);
-				}
-				throw new FileLocked($e->getMessage(), $e->getCode(), $e);
-			}
-
 			if ($needsPartFile) {
 				// rename to correct path
 				try {