Use regex to detect part files

owncloud · Mar 24, 2017 · 6799445 · 6799445
1 parent 4c2358c
commit 6799445
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/lib/private/legacy/util.php b/lib/private/legacy/util.php
@@ -1439,8 +1439,8 @@ public static function isValidFileName($file) {
 			return false;
 		}
 
-		$path_parts = pathinfo($trimmed);
-		if ($path_parts['extension'] === 'part') {
+		// detect part files
+		if (preg_match('/' . \OCP\Files\FileInfo::BLACKLIST_FILES_REGEX . '/', $trimmed) !== 0) {
 			return false;
 		}
 

diff --git a/tests/lib/UtilTest.php b/tests/lib/UtilTest.php
@@ -10,6 +10,9 @@
 
 use OC_Util;
 
+/**
+ * @group DB
+ */
 class UtilTest extends \Test\TestCase {
 	public function testGetVersion() {
 		$version = \OCP\Util::getVersion();
@@ -234,6 +237,13 @@ public function filenameValidationProvider() {
 			['.. ', false],
 			['. ', false],
 			[' .', false],
+			// part files not allowed
+			['.part', false],
+			['notallowed.part', false],
+			['neither.filepart', false],
+			// part in the middle is ok
+			['super movie part one.mkv', true],
+			['super.movie.part.mkv', true],
 		];
 	}