diff --git a/src/Storage/Validators/FileType.php b/src/Storage/Validators/FileType.php index 0f1870e704..711bdb8263 100644 --- a/src/Storage/Validators/FileType.php +++ b/src/Storage/Validators/FileType.php @@ -64,6 +64,10 @@ class FileType extends Validator */ public function isValid($path) { + if(!\is_readable($path)) { + return false; + } + $handle = fopen($path, 'r'); if (!$handle) { diff --git a/tests/resources/disk-b/kitten-1.png b/tests/resources/disk-b/kitten-1.png new file mode 100644 index 0000000000..25df7c3431 Binary files /dev/null and b/tests/resources/disk-b/kitten-1.png differ diff --git a/tests/resources/disk-b/kitten-2.png b/tests/resources/disk-b/kitten-2.png new file mode 100644 index 0000000000..8629a6a768 Binary files /dev/null and b/tests/resources/disk-b/kitten-2.png differ diff --git a/tests/unit/Storage/Validators/FileTypeTest.php b/tests/unit/Storage/Validators/FileTypeTest.php index 24eb16a4ce..a6cb115aaf 100644 --- a/tests/unit/Storage/Validators/FileTypeTest.php +++ b/tests/unit/Storage/Validators/FileTypeTest.php @@ -25,6 +25,7 @@ class FileTypeTest extends TestCase { $this->assertEquals($this->object->isValid(__DIR__ . '/../../../resources/disk-a/kitten-1.jpg'), true); $this->assertEquals($this->object->isValid(__DIR__ . '/../../../resources/disk-a/kitten-2.jpg'), true); - $this->assertEquals($this->object->isValid(__DIR__ . '/../../../resources/disk-a/kitten-2.jpg'), true); + $this->assertEquals($this->object->isValid(__DIR__ . '/../../../resources/disk-b/kitten-1.png'), false); + $this->assertEquals($this->object->isValid(__DIR__ . '/../../../resources/disk-b/kitten-2.png'), false); } } \ No newline at end of file