Zend Framework

strpos() operations on objects cause segfaults

Details

  • Type: Bug Bug
  • Status: Resolved Resolved
  • Priority: Trivial Trivial
  • Resolution: Fixed
  • Affects Version/s: 1.8.4
  • Fix Version/s: 1.9.1
  • Component/s: Zend_Locale
  • Labels:
    None

Description

It has come to my attention that Zend_Date() causes a segfault when you pass another Zend_Date object to its constructor. After some debugging I realized that this segfault is caused by Zend_Locale calling strpos() on that Zend_Date object in _prepareLocale().

I realize that this is probably really a PHP bug, but it seems cleaner if _prepareLocale() casts $locale to a string first before performing any string operations on it.

I was able to reproduce this by executing the following command:

php -r 'require_once "Zend/Date.php"; $x = new Zend_Date(); $y = new Zend_Date($x);'

Note that I'm only able to reproduce this problem on one out of three servers I've tested this on. I was able to reproduce it under 32-bit Debian Linux 5.0. I was not able to reproduce it under 64-bit Debian Linux 4.0 or 5.0.

I am attaching a patch that will fix the segmentation fault by casting $locale to a string at an earlier point.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Thomas Weidner added a comment -

Which PHP release are you using on Debian ?

Show
Thomas Weidner added a comment - Which PHP release are you using on Debian ?
Hide
Permalink
Alex Adriaanse added a comment -

5.2.9 on all three servers that I've tested. The PHP packages were provided by dotdeb.org (the server that produced the segfault had the 5.2.9-0.dotdeb.2 packages installed).

Show
Alex Adriaanse added a comment - 5.2.9 on all three servers that I've tested. The PHP packages were provided by dotdeb.org (the server that produced the segfault had the 5.2.9-0.dotdeb.2 packages installed).
Hide
Permalink
Thomas Weidner added a comment -

This seems to be the related PHP bug: http://bugs.php.net/bug.php?id=47522

Show
Thomas Weidner added a comment - This seems to be the related PHP bug: http://bugs.php.net/bug.php?id=47522
Hide
Permalink
Thomas Weidner added a comment -

The reason of this segfault seem to be the huge memory footprint used when initiating date objects.

Some performance increasements have been implemented which prevent that this error occurs in this place.

Still, ZF is not able to solve a PHP internal error. The given code should have worked even with the used memory.

Show
Thomas Weidner added a comment - The reason of this segfault seem to be the huge memory footprint used when initiating date objects. Some performance increasements have been implemented which prevent that this error occurs in this place. Still, ZF is not able to solve a PHP internal error. The given code should have worked even with the used memory.
Hide
Permalink
Alex Adriaanse added a comment -

Understood. But wouldn't it be cleaner to move the string cast up a few lines anyway? The workaround seems easy and clean.

Show
Alex Adriaanse added a comment - Understood. But wouldn't it be cleaner to move the string cast up a few lines anyway? The workaround seems easy and clean.

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved:
Atlassian JIRA (v4.4.3#663-r165197) | Report a problem
Powered by a free Atlassian JIRA open source license for Zend Framework. Try JIRA - bug tracking software for your team.