Bug #54750: Wrong use of tempnam() in LocalPrieviewHelper - TYPO3 Core - TYPO3 Forge

Actions

Copy link

Bug #54750

closed

Wrong use of tempnam() in LocalPrieviewHelper

Added by Frans Saris over 10 years ago. Updated over 6 years ago.

Status:

Closed

Priority:

Must have

Assignee:

Frans Saris

Category:

File Abstraction Layer (FAL)

Target version:

6.2.0

Start date:

2014-01-05

Due date:

% Done:

100%

Estimated time:

TYPO3 Version:

6.2

PHP Version:

Tags:

Complexity:

Is Regression:

Sprint Focus:

Description

In TYPO3\CMS\Core\Resource\Processing\LocalPrieviewHelper Utility\GeneralUtility::tempnam() is used to create a temp file.

$temporaryFileName = Utility\GeneralUtility::tempnam('preview_') . '.' . $task->getTargetFileExtension();

The problem here is that tempnam() creates a file int typo3temp but that one isn't used because the file extension is added to the $temporyFileName.

This results in a empty file in typo3temp every time an processed file is created.

Actions

Copy link

Updated by Gerrit Code Review over 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 https://review.typo3.org/26650

Actions

Copy link

Updated by Gerrit Code Review over 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 https://review.typo3.org/26650

Actions

Copy link

Updated by Gerrit Code Review over 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 https://review.typo3.org/26650

Actions

Copy link

Updated by Gerrit Code Review over 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 https://review.typo3.org/26650

Actions

Copy link

Updated by Gerrit Code Review over 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 https://review.typo3.org/26650

Actions

Copy link

Updated by Gerrit Code Review over 10 years ago

Patch set 6 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/26650

Actions

Copy link

Updated by Gerrit Code Review over 10 years ago

Patch set 7 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at https://review.typo3.org/26650

Actions

Copy link