Bug #47675 [NoF->ReO]: File descriptor leaked due to HAVE_BROKEN_GETCWD

From: Date: Wed, 28 Sep 2011 01:07:35 +0000
Subject: Bug #47675 [NoF->ReO]: File descriptor leaked due to HAVE_BROKEN_GETCWD
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-163933@lists.php.net to get a copy of this message
Edit report at https://bugs.php.net/bug.php?id=47675&edit=1

 ID:                 47675
 Updated by:         jsjohnst@php.net
 Reported by:        cs at ecn dot purdue dot edu
 Summary:            File descriptor leaked due to HAVE_BROKEN_GETCWD
-Status:             No Feedback
+Status:             Re-Opened
 Type:               Bug
 Package:            Apache2 related
 Operating System:   Solaris 10
 PHP Version:        5.2.9
 Block user comment: N
 Private report:     N



Previous Comments:
------------------------------------------------------------------------
[2011-09-28 00:58:18] jsjohnst@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.

------------------------------------------------------------------------
[2010-01-09 06:59:22] bryan at stansell dot org

I've encountered this problem using OpenSolaris (snv_115), apache 1.3.41 and php-5.2.12 (via
mod_php5).  It was also a problem with php 5.2.9.  My apache processes continue to accumulate open
files pointing to the directory which contains the php script.

I am using gcc 4.3.3, gnu as, and solaris ld.  It makes me wonder if it's a compiler-related
thing.

I was also talking to a friend and we checked his httpd processes and saw the same file descriptor
leak.  His setup is Solaris 9 (sparc), apache 1.3.41, php 4.4.8, and gcc 4.0.2.

I worked around my problem by unsetting HAVE_BROKEN_GETCWD.  I have a couple other ideas on possibly
narrowing down the problem, but I haven't had a chance to try them.

------------------------------------------------------------------------


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


Thread (11 messages)

« previous php.bugs (#163933) next »