[HttpCache] Unlink tmp file on error

Chansig · Chansig · commit c9a03558830a · 2018-03-31T09:59:49.000+02:00
diff --git a/src/Symfony/Component/HttpKernel/HttpCache/Store.php b/src/Symfony/Component/HttpKernel/HttpCache/Store.php
@@ -387,16 +387,22 @@ private function save($key, $data)
 
             $tmpFile = tempnam(dirname($path), basename($path));
             if (false === $fp = @fopen($tmpFile, 'wb')) {
+                @unlink($tmpFile);
+
                 return false;
             }
             @fwrite($fp, $data);
             @fclose($fp);
 
             if ($data != file_get_contents($tmpFile)) {
+                @unlink($tmpFile);
+
                 return false;
             }
 
             if (false === @rename($tmpFile, $path)) {
+                @unlink($tmpFile);
+
                 return false;
             }
         }

Original file line number	Diff line number	Diff line change
`@@ -387,16 +387,22 @@ private function save($key, $data)`
`387`	`387`
`388`	`388`	`$tmpFile = tempnam(dirname($path), basename($path));`
`389`	`389`	`if (false === $fp = @fopen($tmpFile, 'wb')) {`
	`390`	`+ @unlink($tmpFile);`
	`391`	`+`
`390`	`392`	`return false;`
`391`	`393`	`}`
`392`	`394`	`@fwrite($fp, $data);`
`393`	`395`	`@fclose($fp);`
`394`	`396`
`395`	`397`	`if ($data != file_get_contents($tmpFile)) {`
	`398`	`+ @unlink($tmpFile);`
	`399`	`+`
`396`	`400`	`return false;`
`397`	`401`	`}`
`398`	`402`
`399`	`403`	`if (false === @rename($tmpFile, $path)) {`
	`404`	`+ @unlink($tmpFile);`
	`405`	`+`
`400`	`406`	`return false;`
`401`	`407`	`}`
`402`	`408`	`}`