Bug #64273
closedDownloading FAL media via FileDumpEID.php causes an out of memory error
100%
Description
When trying to download a large file (in my case a mp4 file) via FileDumpEID.php i get an out of memory error. The file is sent by the method dumpFileContents in class TYPO3\CMS\Core\Resource\Driver\LocalDriver. According to PHP manual the method readfile will not present any memory issues when output buffering is off, so i checked ob_get_level() in TYPO3\CMS\Core\Resource\ResourceStorage->dumpFileContents and for any reason it was 3.
Maybe it would be a good idea to disable output buffering before using readfile().
@public function dumpFileContents(FileInterface $file, $asDownload = FALSE, $alternativeFilename = NULL) {
[...]
while (ob_get_level() > 0) {
ob_end_clean();
}
$this->driver->dumpFileContents($file->getIdentifier());
}@
Updated by Mathias Schreiber almost 10 years ago
- Target version set to 7.1 (Cleanup)
- Sprint Focus set to On Location Sprint
Updated by Gerrit Code Review almost 10 years ago
- Status changed from New to Under Review
Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36485
Updated by Gerrit Code Review almost 10 years ago
Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36485
Updated by Gerrit Code Review almost 10 years ago
Patch set 3 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36485
Updated by Gerrit Code Review almost 10 years ago
Patch set 4 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36485
Updated by Gerrit Code Review almost 10 years ago
Patch set 5 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36485
Updated by Gerrit Code Review almost 10 years ago
Patch set 1 for branch TYPO3_6-2 of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/36533
Updated by Roman Schürmann almost 10 years ago
- Status changed from Under Review to Resolved
- % Done changed from 0 to 100
Applied in changeset b26f74850f6bae76ee650af4f96a26a5660f0c88.
Updated by Anja Leichsenring almost 9 years ago
- Sprint Focus deleted (
On Location Sprint)
Updated by Riccardo De Contardi about 7 years ago
- Status changed from Resolved to Closed