ID: 40098
Updated by: [email][/email]
Reported By: wharmby at uk dot ibm dot com
-Status: Open
+Status: Assigned
-Bug Type: Scripting Engine problem
+Bug Type: Streams related
Operating System: Linux RHEL4
PHP Version: 5CVS-2007-01-11 (snap)
-Assigned To:
+Assigned To: iliaa

Previous Comments:

[2007-01-11 11:59:44] wharmby at uk dot ibm dot com

The current implementation of php_fopen_primary_script()
uses the non-reentrant getpwnam() rather than the re-entrant
getpwnam_r(). Therefore calling it on a Linux ZTS enabled
build could lead to unpredictable/undesirable results. The
code should use the re-entrant version if it is available.

The following patch which were built against the latest snapshot (Jan
11 2007, 0730 GMT) modifies the code in
main/fopen_wrappers.c to use the re-entrant getpwnam_r if
its available:


Fix tested on Linux RHEL4 with Apache 2.0.52 and user_dir set in
php.ini The modified code was then driven by specifying the ~user_id in
the URL e.g


Reproduce code:
Problem found by code inspection. As with most thread safety
issues difficult to produce a simple testcase which will show
a reproducible crash but current Linux executable is clearly
not reentrant and therefore not thread safe.

Expected result:

Actual result:


Edit this bug report at [url][/url]