[Filesystem] Safeguard (sym)link calls

derrabus · derrabus · commit 17e612adbaab · 2022-05-19T15:04:31.000+02:00
diff --git a/src/Symfony/Component/Filesystem/Filesystem.php b/src/Symfony/Component/Filesystem/Filesystem.php
@@ -318,6 +318,8 @@ private function isReadable(string $filename): bool
      */
     public function symlink($originDir, $targetDir, $copyOnWindows = false)
     {
+        self::assertOperationIsAvailable('symlink');
+
         if ('\\' === \DIRECTORY_SEPARATOR) {
             $originDir = strtr($originDir, '/', '\\');
             $targetDir = strtr($targetDir, '/', '\\');
@@ -354,6 +356,8 @@ public function symlink($originDir, $targetDir, $copyOnWindows = false)
      */
     public function hardlink($originFile, $targetFiles)
     {
+        self::assertOperationIsAvailable('link');
+
         if (!$this->exists($originFile)) {
             throw new FileNotFoundException(null, 0, null, $originFile);
         }
@@ -737,6 +741,13 @@ private function getSchemeAndHierarchy(string $filename): array
         return 2 === \count($components) ? [$components[0], $components[1]] : [null, $components[0]];
     }
 
+    private static function assertOperationIsAvailable(string $functionName): void
+    {
+        if (!\function_exists($functionName)) {
+            throw new IOException(sprintf('Unable to perform filesystem operation because the "%s()" function has been disabled.', $functionName));
+        }
+    }
+
     /**
      * @param mixed ...$args
      *

Original file line number	Diff line number	Diff line change
`@@ -318,6 +318,8 @@ private function isReadable(string $filename): bool`
`318`	`318`	`*/`
`319`	`319`	`public function symlink($originDir, $targetDir, $copyOnWindows = false)`
`320`	`320`	`{`
	`321`	`+ self::assertOperationIsAvailable('symlink');`
	`322`	`+`
`321`	`323`	`if ('\\' === \DIRECTORY_SEPARATOR) {`
`322`	`324`	`$originDir = strtr($originDir, '/', '\\');`
`323`	`325`	`$targetDir = strtr($targetDir, '/', '\\');`
`@@ -354,6 +356,8 @@ public function symlink($originDir, $targetDir, $copyOnWindows = false)`
`354`	`356`	`*/`
`355`	`357`	`public function hardlink($originFile, $targetFiles)`
`356`	`358`	`{`
	`359`	`+ self::assertOperationIsAvailable('link');`
	`360`	`+`
`357`	`361`	`if (!$this->exists($originFile)) {`
`358`	`362`	`throw new FileNotFoundException(null, 0, null, $originFile);`
`359`	`363`	`}`
`@@ -737,6 +741,13 @@ private function getSchemeAndHierarchy(string $filename): array`
`737`	`741`	`return 2 === \count($components) ? [$components[0], $components[1]] : [null, $components[0]];`
`738`	`742`	`}`
`739`	`743`
	`744`	`+ private static function assertOperationIsAvailable(string $functionName): void`
	`745`	`+ {`
	`746`	`+ if (!\function_exists($functionName)) {`
	`747`	`+ throw new IOException(sprintf('Unable to perform filesystem operation because the "%s()" function has been disabled.', $functionName));`
	`748`	`+ }`
	`749`	`+ }`
	`750`	`+`
`740`	`751`	`/**`
`741`	`752`	`* @param mixed ...$args`
`742`	`753`	`*`