[BUGFIX] SimpleFileBackend::setCacheDirectory returns valid path name 58/46158/2
authorJan Helke <typo3@helke.de>
Fri, 22 Jan 2016 09:58:24 +0000 (10:58 +0100)
committerFrank Nägler <frank.naegler@typo3.org>
Fri, 22 Jan 2016 14:37:14 +0000 (15:37 +0100)
commita59665c164c2f57ee2f97e2b0bde84d52238d2a8
tree33812571f0011f2146847791931feb73c3dc99f7
parent941308f68650e730b72ca4a5c00944484c90e296
[BUGFIX] SimpleFileBackend::setCacheDirectory returns valid path name

The comment for the method
\TYPO3\CMS\Core\Cache\Backend\SimpleFileBackend::setCacheDirectory()
states clearly, that it is possible to provide an absolute path as
cache directory. However, in the lower part of the function, it is
stated, that if ($cacheDirectory0 == '/') the documentRoot should be
set to '/'. That results in a returned path like '//...'.
This causes problems within
\TYPO3\CMS\Core\Utility\GeneralUtility::validPathStr(), because pathes
with '//' in it are always invalid.

Resolves: #72635
Releases: master, 7.6, 6.2
Change-Id: I30e7743ba9835c99382b7c04153f91e688ead5fc
Reviewed-on: https://review.typo3.org/46158
Reviewed-by: Jan Helke <typo3@helke.de>
Reviewed-by: Frank Nägler <frank.naegler@typo3.org>
Tested-by: Frank Nägler <frank.naegler@typo3.org>
typo3/sysext/core/Classes/Cache/Backend/SimpleFileBackend.php
typo3/sysext/core/Tests/Unit/Cache/Backend/FileBackendTest.php