Mailing-List: contact dev-help@tomcat.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Tomcat Developers List" <dev@tomcat.apache.org>
Message-ID: <52492924.1040205@apache.org>
Date: Mon, 30 Sep 2013 08:32:52 +0100
From: Mark Thomas <markt@apache.org>
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64;
 rv:17.0) Gecko/20130801 Thunderbird/17.0.8
MIME-Version: 1.0
To: Tomcat Developers List <dev@tomcat.apache.org>
Subject: Re: Recent tcnative null-dereference with 8.0.0-RC3 and 7.0.45
 [tcnative-1.dll+0x7e23]
References: <5246199A.70000@christopherschultz.net>
 <5246AEB6.5050509@kippdata.de> <5246B613.6010504@apache.org>
In-Reply-To: <5246B613.6010504@apache.org>
Content-Type: text/plain; charset=ISO-8859-15
Content-Transfer-Encoding: 7bit

On 28/09/2013 11:57, Mladen Turk wrote:
> On 09/28/2013 12:25 PM, Rainer Jung wrote:
>>
>>> I can't seem to reproduce the problem using an APR connector on Linux:
>>> reports seems to indicate that trivially accessing Tomcat via an HTTP
>>> APR connector will cause a crash, and I was able to run the following
>>> command without bringing down my instance:
>>>
>>> $ ab -n 10000 -c 50 http://localhost:28215/my-webapp/index.html
>>
>>
>> Still the whole poll stuff is platform dependent, so it might be you
>> will not be able to reproduce the Windows crash even when the test
>> scenario is exactly the same as there.
>>
> 
> Almost every APR crash error is related to reusing closed
> object/descriptor/pointer.
> Eg, crashing inside poll can be caused by closing the socket that is
> inside poller.
> The fact that it doesn't crash on linux might be just because of "close"
> order.
> Even on the same OS it doesn't have to crash all the time.
> 
> Anyhow, I'll try to check the 8-RC3 with the latest native on windows.
> Seems there are some conceptual problem used inside TC8 probably with
> async sockets and double close.

I suspect multiple factors:
- the refactoring to a single poller thread may have introduced some
additional issues we haven't tracked down yet
- allowing two threads (one read, one write) to work with the same
socket introduces lots of opportunities to close a socket in one thread
while it is still in use in another

I've been mulling over refactoring the connector code so all actions on
the socket go via the socket wrapper so it is easier to track true
socket state in one place. Now might be the time to implement that change.

Mark


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org