Merge pull request #1614 from revoltek-daniel/patch-1

Check if path is a file in FileSystemLoader
liip · Dec 12, 2024 · 6d8dee9 · 6d8dee9
2 parents 98e0318 + 59a78cd
commit 6d8dee9
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/Binary/Loader/FileSystemLoader.php b/Binary/Loader/FileSystemLoader.php
@@ -12,6 +12,7 @@
 namespace Liip\ImagineBundle\Binary\Loader;
 
 use Liip\ImagineBundle\Binary\Locator\LocatorInterface;
+use Liip\ImagineBundle\Exception\Binary\Loader\NotLoadableException;
 use Liip\ImagineBundle\Exception\InvalidArgumentException;
 use Liip\ImagineBundle\Model\FileBinary;
 use Symfony\Component\HttpFoundation\File\MimeType\ExtensionGuesserInterface as DeprecatedExtensionGuesserInterface;
@@ -69,6 +70,11 @@ public function __construct(
     public function find($path)
     {
         $path = $this->locator->locate($path);
+
+        if (\is_file($path) === false) {
+            throw new NotLoadableException(\sprintf('Source image: "%s" is no file.', $path));
+        }
+
         $mimeType = $this->mimeTypeGuesser instanceof DeprecatedMimeTypeGuesserInterface ? $this->mimeTypeGuesser->guess($path) : $this->mimeTypeGuesser->guessMimeType($path);
         $extension = $this->getExtension($mimeType);