Bug #30623: call cObj->includeLibs without check if $conf is an array - TYPO3 Core - TYPO3 Forge

Actions

Bug #30623

closed

call cObj->includeLibs without check if $conf is an array

Added by Frank Nägler about 13 years ago. Updated about 6 years ago.

Status:

Closed

Priority:

Must have

Assignee:

-

Category:

-

Target version:

-

Start date:

2011-10-07

Due date:

% Done:

0%

Estimated time:

TYPO3 Version:

4.5

PHP Version:

Tags:

Complexity:

Is Regression:

Sprint Focus:

Description

In some situations $conf is not an array. e.g. if an extension has a wrong typoscript configuration.
in this case there are hundreds of error_log entries like this:

Core: Error handler (FE): PHP Catchable Fatal Error: Argument 1 passed to tslib_cObj::includeLibs() must be an array, string given, called in /.../htdocs/typo3-src/typo3_src-4.5.6/typo3/sysext/cms/tslib/content/class.tslib_content_user.php on line 50 and defined in /.../htdocs/typo3-src/typo3_src-4.5.6/typo3/sysext/cms/tslib/class.tslib_content.php line 6783

On line 50 in cms/tslib/content/class.tslib_content_user.php the function cObj->includeLibs is called without a check if the given argument ($conf) is an array.

I have fixed this, the patch file is attached.

Files

bug_30623.diff (1 KB) bug_30623.diff

Patch file for this issue

Frank Nägler, 2011-10-07 13:00

Related issues 1 (0 open — 1 closed)

Actions

#1

Updated by Frank Nägler about 13 years ago

File bug_30623.diff bug_30623.diff added

Actions

#2

Updated by Mr. Hudson about 13 years ago

Patch set 1 of change Id98c1b584969082f72f55372618f9e72b210e019 has been pushed to the review server.
It is available at http://review.typo3.org/5611

Actions

#3

Updated by Frank Nägler about 13 years ago

please see the http://review.typo3.org/5610 this is the main change, sorry for the double commit, it was my first git/review patch :(

Actions

#4

Updated by Mr. Hudson about 13 years ago

Patch set 2 of change Id98c1b584969082f72f55372618f9e72b210e019 has been pushed to the review server.
It is available at http://review.typo3.org/5611

Actions

#5

Updated by Mr. Hudson about 13 years ago

Patch set 3 of change I8ed20a4fd5335d59cb9150334ed218be0bc759cc has been pushed to the review server.
It is available at http://review.typo3.org/5610

Actions

#6

Updated by Mr. Hudson about 13 years ago

Patch set 4 of change I8ed20a4fd5335d59cb9150334ed218be0bc759cc has been pushed to the review server.
It is available at http://review.typo3.org/5610

Actions

#7

Updated by Mr. Hudson about 13 years ago

Patch set 5 of change I8ed20a4fd5335d59cb9150334ed218be0bc759cc has been pushed to the review server.
It is available at http://review.typo3.org/5610

Actions

#8

Updated by Mr. Hudson about 13 years ago

Status changed from New to Under Review

Patch set 6 of change I8ed20a4fd5335d59cb9150334ed218be0bc759cc has been pushed to the review server.
It is available at http://review.typo3.org/5610

Actions

#9

Updated by Mr. Hudson about 13 years ago

Patch set 7 of change I8ed20a4fd5335d59cb9150334ed218be0bc759cc has been pushed to the review server.
It is available at http://review.typo3.org/5610

Actions

#10

Updated by Gerrit Code Review almost 13 years ago

Patch set 8 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/5610

Actions

#11

Updated by Gerrit Code Review almost 13 years ago

Patch set 9 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/5610

Actions

#12

Updated by Gerrit Code Review almost 13 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/9009

Actions

#13

Updated by Philipp Gampe over 12 years ago

Status changed from Under Review to Resolved

Applied in changeset a43c75c0140585fd15161e9551bc110be62c52ec.

Actions

#14

Updated by Benni Mack about 6 years ago

Status changed from Resolved to Closed

Actions

Also available in: Atom PDF