Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:6213
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Message-ID: <20031204150617.1165.qmail@pb1.pair.com>
To: internals@lists.php.net
Reply-To: jay@php.net
Mail-Copies-To: jay@php.net
Date: Thu, 04 Dec 2003 10:06:12 -0500
References: <6.0.1.1.0.20031203175354.039cac08@127.0.0.1> <6.0.1.1.0.20031203181532.039d2858@127.0.0.1> <20031203182039.91511.qmail@pb1.pair.com> <6.0.1.1.0.20031203193419.03a13eb8@127.0.0.1>
Lines: 58
User-Agent: KNode/0.7.2
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7Bit
Subject: Re: [PHP-DEV] browscap and nesting level too deep (bug #25916)
From: jay@php.net (Jay Smith)


Glad to see it's finally working. I was never able to get those errors, but
it probably had something to do with the thread pooling you had configured.
And seeing as it's working now, I guess I don't need to bother trying to
get them. 

J

Uwe Schindler wrote:

> On my solaris box the fix does it. I tested it by hammering the same PHP
> script using get_browser() with and without patch. Without patch it gets
> this error. With not and script works :)
> 
> The big problem with this bug is that when the error happens the first
> time (3 threads using get_browser()), the thread which produces the error
> does not reset the recursion counter (because of the error) and after
> finishing all threads it is left at 1. When then 2 more threads use
> get_browser() at the same time the second one gets the error and at the
> end the counter is left at 2. And then all further calls fail...
> 
> In my special case today when the error occured it was:
> 
> 168.221.143.68 www.pangaea.de - [03/Dec/2003:17:10:09 +0100] "GET /
> HTTP/1.1" 200 1985
> "http://www.google.com/search?hl=en&ie=UTF-8&oe=UTF-8&q=pangaea"
> "Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.1; .NET CLR 1.1.4322)"
> 168.221.143.68 www.pangaea.de - [03/Dec/2003:17:10:09 +0100] "GET /
> HTTP/1.1" 200 1985
> "http://www.google.com/search?hl=en&ie=UTF-8&oe=UTF-8&q=pangaea"
> "Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.1; .NET CLR 1.1.4322)"
> 168.221.143.68 www.pangaea.de - [03/Dec/2003:17:10:09 +0100] "GET /
> HTTP/1.1" 200 1985
> "http://www.google.com/search?hl=en&ie=UTF-8&oe=UTF-8&q=pangaea"
> "Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.1; .NET CLR 1.1.4322)"
> 168.221.143.68 www.pangaea.de - [03/Dec/2003:17:10:09 +0100] "GET /
> HTTP/1.1" 200 150
> "http://www.google.com/search?hl=en&ie=UTF-8&oe=UTF-8&q=pangaea"
> "Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.1; .NET CLR 1.1.4322)"
> 168.221.143.68 www.pangaea.de - [03/Dec/2003:17:10:09 +0100] "GET /
> HTTP/1.1" 200 150
> "http://www.google.com/search?hl=en&ie=UTF-8&oe=UTF-8&q=pangaea"
> "Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.1; .NET CLR 1.1.4322)"
> 168.221.143.68 www.pangaea.de - [03/Dec/2003:17:10:09 +0100] "GET /
> HTTP/1.1" 200 150
> "http://www.google.com/search?hl=en&ie=UTF-8&oe=UTF-8&q=pangaea"
> "Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.1; .NET CLR 1.1.4322)"
> 
> 6 threads accessed the get_browser() at the same time - 3 of them failed
> (content-length=150... and visible in error log), counter left at 3 ->
> from this time on get_browser failed to work.
> 
> I think I should apply the patch to 4.3 and 5 (same problem there) and
> close the bug.
> 
> Uwe
>