Edit report at https://bugs.php.net/bug.php?id=47675&edit=1
ID: 47675 Updated by: ras...@php.net Reported by: cs at ecn dot purdue dot edu Summary: File descriptor leaked due to HAVE_BROKEN_GETCWD -Status: Re-Opened +Status: Closed Type: Bug Package: Apache2 related Operating System: Solaris 10 PHP Version: 5.2.9 -Assigned To: +Assigned To: rasmus Block user comment: N Private report: N New Comment: Will be in the next batch of releases Previous Comments: ------------------------------------------------------------------------ [2013-04-04 20:56:28] davek at gamehouse dot com Related To: Bug #64577 ------------------------------------------------------------------------ [2011-09-28 00:58:18] jsjoh...@php.net I've heard that this was fixed in PHP 5.3.5. It's not listed in the release notes from what I can see, so can someone confirm if 5.3.5 addresses this issue? ------------------------------------------------------------------------ [2011-05-18 18:23:29] pyorke at joyent dot com This still broken in PHP 5.3.3 When is it going to be fixed ------------------------------------------------------------------------ [2010-08-08 10:20:55] php at marino dot st I've been trying to track down this file descriptor leakage problem for months. I was stuck on 5.2.8 because of it. I confirm that the issue is specifically with Solaris 10. I have opensolaris sxce nevada 130 locally and I've not seen FD leakage on it. I confirm that patch suggested by bryan at stansell dot org seemed to correct the problem. FYI, PHP was spawned and remains persistent for use with the Litespeed web server (uses the LSAPI interface), so it would run out of file descriptors between 1 and 12 hours on my site. It's a bit disappointing that this error has been present for 5 releases and was never fixed. ------------------------------------------------------------------------ [2010-01-12 15:40:45] bryan at stansell dot org I finally got a chance to test a theory. Looks like the volatile attribute fixed things for me. #if HAVE_BROKEN_GETCWD volatile int old_cwd_fd = -1; #else Once I added that, the setjmp/longjmp worked as expected. I got the idea from the manpage on Solaris: The values of register and automatic variables are unde- fined. Register or automatic variables whose value must be relied upon must be declared as volatile. Perhaps it's a gcc/gas/Solaris/x86 optimization somewhere that overlooked the case, but this is a workaround. Of course, undefining HAVE_BROKEN_GETCWD for Solaris also works, if you have a web tree that isn't restricted in some way. ------------------------------------------------------------------------ The remainder of the comments for this report are too long. To view the rest of the comments, please view the bug report online at https://bugs.php.net/bug.php?id=47675 -- Edit this bug report at https://bugs.php.net/bug.php?id=47675&edit=1