Project

General

Profile

Actions

Bug #16521

closed

Several issues with Safe Mode On and testing for files using is_file()

Added by Rune Nilssen over 17 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
Install Tool
Target version:
-
Start date:
2006-09-02
Due date:
% Done:

0%

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

Description

Due to the way is_file() works it will always return false if the file is not owned by the user running the php script. This means in essense that if the file isnt owned by webserver user is_file() will always return false when testing for it..

Thus far I have found two issues regarding this;

First default_config.php is unable to detect localconf.php even if it is sitting there and is read and writeable by the webserver.

Second in the install script testing for ImageMagick executables always fails even if they are available. This can be worked around by setting the path manually in the all configurastion page and enabling IM.

My suggestion is that Typo3 considers a different testing mechanism when running under safemode, like actually executing the file in question or something similar. Perhaps reading a few bytes from the beginning of the file if that can be done. This is ofcourse more expensive in terms of cpu cycles, but makes typo3 alot more safemode friendly.

I believe this bug/flaw/limitation is valid in the latest release as of today.
(issue imported from #M4142)


Related issues 2 (0 open2 closed)

Related to TYPO3 Core - Bug #14830: Typo3 Imagemagick static detection not workingClosedMichael Stucki2005-06-19

Actions
Related to TYPO3 Core - Bug #16697: typo3 v4.0.2 requires gmClosedMichael Stucki2006-11-07

Actions
Actions #1

Updated by Chris topher almost 14 years ago

This is still an issue in current trunk (as of TYPO3 4.4beta3).

E.g. in sysext/install/mod/class.tx_install.php -> checkImageMagick()
the following line might return the wrong value:

file_exists($v) && @is_file($v.$filename.$isExt)

Actions #2

Updated by Wouter Wolters about 11 years ago

  • Status changed from New to Resolved
  • Target version deleted (0)

In INSTALL.txt is the following line
Safe mode is not supported, thus it must not be activated

We do not support safemode since some versions.

Actions #3

Updated by Benni Mack over 5 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF