Check whether path is file in DataPart::fromPath()

freiondrej · fabpot · commit 9e3670e14057 · 2020-06-22T16:55:08.000+02:00
diff --git a/src/Symfony/Component/Mime/Part/DataPart.php b/src/Symfony/Component/Mime/Part/DataPart.php
@@ -56,6 +56,10 @@ public static function fromPath(string $path, string $name = null, string $conte
             $contentType = self::$mimeTypes->getMimeTypes($ext)[0] ?? 'application/octet-stream';
         }
 
+        if (false === is_readable($path)) {
+            throw new InvalidArgumentException(sprintf('Path "%s" is not readable.', $path));
+        }
+        
         if (false === $handle = @fopen($path, 'r', false)) {
             throw new InvalidArgumentException(sprintf('Unable to open path "%s".', $path));
         }

Original file line number	Diff line number	Diff line change
`@@ -56,6 +56,10 @@ public static function fromPath(string $path, string $name = null, string $conte`
`56`	`56`	`$contentType = self::$mimeTypes->getMimeTypes($ext)[0] ?? 'application/octet-stream';`
`57`	`57`	`}`
`58`	`58`
	`59`	`+ if (false === is_readable($path)) {`
	`60`	`+ throw new InvalidArgumentException(sprintf('Path "%s" is not readable.', $path));`
	`61`	`+ }`
	`62`	`+`
`59`	`63`	`if (false === $handle = @fopen($path, 'r', false)) {`
`60`	`64`	`throw new InvalidArgumentException(sprintf('Unable to open path "%s".', $path));`
`61`	`65`	`}`