Remove the uploaded file if there is a checksum error

owncloud · Jun 5, 2019 · cbab58b · cbab58b
1 parent 5550d82
commit cbab58b
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/apps/dav/lib/Connector/Sabre/File.php b/apps/dav/lib/Connector/Sabre/File.php
@@ -208,6 +208,7 @@ public function put($data) {
 			}
 
 			if (!self::isChecksumValid($partStorage, $internalPartPath)) {
+				$partStorage->unlink($internalPartPath);  // remove the uploaded file on checksum error
 				throw new BadRequest('The computed checksum does not match the one received from the client.');
 			}
 
@@ -535,6 +536,7 @@ private function createFileChunked($data) {
 					$chunk_handler->file_assemble($partStorage, $partInternalPath);
 
 					if (!self::isChecksumValid($partStorage, $partInternalPath)) {
+						$partStorage->unlink($partInternalPath);  // remove the uploaded file on checksum error
 						throw new BadRequest('The computed checksum does not match the one received from the client.');
 					}