Project

General

Profile

Actions

Bug #51761

closed

PHP Warning for open_basedir in filelist for every file after updating 6.1.3 -> 6.1.4

Added by Simon Schick over 10 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
File Abstraction Layer (FAL)
Target version:
Start date:
2013-09-05
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
6.1
PHP Version:
5.3
Tags:
Complexity:
Is Regression:
Yes
Sprint Focus:

Description

I don't know in which change this was included ... but since the update to 6.1.4, I get a PHP Warning for every file listed there.

The warning is:
PHP Warning: is_dir(): open_basedir restriction in effect. File(/srv/http/domain.org/www/fileadmin/user_upload/someimage.jpg/) is not within the allowed path(s): (/srv/http/domain.org/:/srv/http/localhost/www/piwik/:/srv/http/typo3/:/srv/http/default/xhprof/:/srv/xhprof/:/usr/share/php/xhprof_lib/utils/) in /srv/http/typo3/typo3_src-6.1.4-patched/typo3/sysext/core/Classes/Resource/Driver/LocalDriver.php line 399

I don't know whether this is a PHP bug or not, but it just appears after updating to 6.1.4. After downgrading to 6.1.3, it disappeared.


Related issues 1 (0 open1 closed)

Has duplicate TYPO3 Core - Bug #51771: Warnings after upgrading to 6.0.9 and browsing filelist with open_basedir in useClosed2013-09-05

Actions
Actions #1

Updated by Simon Schick over 10 years ago

FYI: the version I have is called "patched", because I patched it with this bugfix: https://review.typo3.org/#/c/20981/
Unpatching doesn't help.

Actions #2

Updated by Markus Klein over 10 years ago

Thx for reporting.
Indeed these path checks were introduced in 6.1.4.

Obviously the tests with open_basedir were not sufficient.

/srv/http/domain.org/www/fileadmin/user_upload/someimage.jpg/

Actions #3

Updated by Marcus Krause over 10 years ago

Root cause is obviously an is_dir() call on a file resource with a trailing slash.

Actions #4

Updated by Ernesto Baschny over 10 years ago

  • Category set to 1394
  • Target version deleted (2463)
Actions #5

Updated by Ernesto Baschny over 10 years ago

  • Is Regression set to Yes
Actions #6

Updated by Ernesto Baschny over 10 years ago

  • Category changed from 1394 to File Abstraction Layer (FAL)
  • Status changed from New to Accepted
  • Target version set to next-patchlevel
Actions #7

Updated by Gerrit Code Review over 10 years ago

  • Status changed from Accepted to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at https://review.typo3.org/23696

Actions #8

Updated by Helmut Hummel over 10 years ago

Can you check if the change in Gerrit fixes your problem?

Actions #9

Updated by Simon Schick over 10 years ago

@helmut, thanks - works!

Actions #10

Updated by Gerrit Code Review over 10 years ago

Patch set 1 for branch TYPO3_6-1 has been pushed to the review server.
It is available at https://review.typo3.org/23698

Actions #11

Updated by Gerrit Code Review over 10 years ago

Patch set 1 for branch TYPO3_6-0 has been pushed to the review server.
It is available at https://review.typo3.org/23699

Actions #12

Updated by Helmut Hummel over 10 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #13

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF