Re: Recent tcnative null-dereference with 8.0.0-RC3 and 7.0.45 [tcnative-1.dll+0x7e23]

2013-09-30 Thread Mladen Turk
On 09/30/2013 08:50 PM, Christopher Schultz wrote: Mladen, Unless there is significant objection, Yes there is. The transition from Java to JNI costs 10 times then then a simple 'if (socket == OL)' in java I'd like to add such NULL checks to limit JVM crashes in cases where the Java code is

RE: [VOTE] Release Apache Tomcat 7.0.45

2013-09-30 Thread Konstantin Preißer
> -Original Message- > From: Mark Thomas [mailto:ma...@apache.org] > Sent: Monday, September 30, 2013 9:31 PM > To: Tomcat Developers List > Subject: Re: [VOTE] Release Apache Tomcat 7.0.45 > > On 30/09/2013 09:00, Mark Thomas wrote: > > On 26/09/2013 19:46, Konstantin Preißer wrote: > >

Re: [VOTE] Release Apache Tomcat 7.0.45

2013-09-30 Thread Mark Thomas
On 30/09/2013 20:31, Mark Thomas wrote: > On 30/09/2013 09:00, Mark Thomas wrote: >> It looks like my recent fixes for APR/native unit test failures/crashes >> are the culprit. Hopefully it won't take too long to find a fix. > I *think* I have a fix for this. If folks that have seen this error >

buildbot success in ASF Buildbot on tomcat-7-trunk

2013-09-30 Thread buildbot
The Buildbot has detected a restored build on builder tomcat-7-trunk while building ASF Buildbot. Full details are available at: http://ci.apache.org/builders/tomcat-7-trunk/builds/1434 Buildbot URL: http://ci.apache.org/ Buildslave for this Build: bb-vm_ubuntu Build Reason: scheduler Build So

[Bug 55602] JRE crashes during websocket communication

2013-09-30 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=55602 Mark Thomas changed: What|Removed |Added Status|NEW |RESOLVED Version|unspecif

svn commit: r1527745 - /tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml

2013-09-30 Thread markt
Author: markt Date: Mon Sep 30 19:39:50 2013 New Revision: 1527745 URL: http://svn.apache.org/r1527745 Log: Update changelog Modified: tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml Modified: tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml URL: http://svn.apache.org/viewvc/tomcat/tc7.0.x/t

svn commit: r1527742 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/tomcat/util/net/AprEndpoint.java

2013-09-30 Thread markt
Author: markt Date: Mon Sep 30 19:38:05 2013 New Revision: 1527742 URL: http://svn.apache.org/r1527742 Log: Switch to destroySocket in a few places where it is safe to do so. Modified: tomcat/tc7.0.x/trunk/ (props changed) tomcat/tc7.0.x/trunk/java/org/apache/tomcat/util/net/AprEndpoint

svn commit: r1527740 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/tomcat/util/net/AprEndpoint.java

2013-09-30 Thread markt
Author: markt Date: Mon Sep 30 19:37:33 2013 New Revision: 1527740 URL: http://svn.apache.org/r1527740 Log: Refactor the method names some more. Modified: tomcat/tc7.0.x/trunk/ (props changed) tomcat/tc7.0.x/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java Propchange: tomcat/tc7.

svn commit: r1527738 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/tomcat/util/net/AprEndpoint.java

2013-09-30 Thread markt
Author: markt Date: Mon Sep 30 19:34:42 2013 New Revision: 1527738 URL: http://svn.apache.org/r1527738 Log: Add a warning comment and simplify the code a little. Modified: tomcat/tc7.0.x/trunk/ (props changed) tomcat/tc7.0.x/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java Propch

svn commit: r1527737 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/tomcat/util/net/AprEndpoint.java

2013-09-30 Thread markt
Author: markt Date: Mon Sep 30 19:34:17 2013 New Revision: 1527737 URL: http://svn.apache.org/r1527737 Log: Refactor the remove list in the Poller to be a close list. The poller now removes the socket if it is in the Poller then destroys it. Note the socket is always destroyed whether it is found

svn commit: r1527735 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/tomcat/util/net/AprEndpoint.java

2013-09-30 Thread markt
Author: markt Date: Mon Sep 30 19:33:41 2013 New Revision: 1527735 URL: http://svn.apache.org/r1527735 Log: Rename method Modified: tomcat/tc7.0.x/trunk/ (props changed) tomcat/tc7.0.x/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java Propchange: tomcat/tc7.0.x/trunk/

svn commit: r1527734 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/tomcat/util/net/AprEndpoint.java

2013-09-30 Thread markt
Author: markt Date: Mon Sep 30 19:33:10 2013 New Revision: 1527734 URL: http://svn.apache.org/r1527734 Log: Move the point where the socket is removed from the connections Map so all the actions apart from removal from the Poller are in a single method to facilitate refactoring removal of the sock

Re: [VOTE] Release Apache Tomcat 7.0.45

2013-09-30 Thread Mark Thomas
On 30/09/2013 09:00, Mark Thomas wrote: > On 26/09/2013 19:46, Konstantin Preißer wrote: >> Hi Rainer, >> >>> -Original Message- >>> From: Rainer Jung [mailto:rainer.j...@kippdata.de] >>> Sent: Thursday, September 26, 2013 8:22 PM >> >>> Any chance you can try current trunk? Mark added more

svn commit: r1527733 - /tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java

2013-09-30 Thread markt
Author: markt Date: Mon Sep 30 19:27:02 2013 New Revision: 1527733 URL: http://svn.apache.org/r1527733 Log: Switch to destroySocket in a few places where it is safe to do so. Modified: tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java Modified: tomcat/trunk/java/org/apache/tomcat

svn commit: r1527732 - /tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java

2013-09-30 Thread markt
Author: markt Date: Mon Sep 30 19:26:27 2013 New Revision: 1527732 URL: http://svn.apache.org/r1527732 Log: Refactor the method names some more. Modified: tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java Modified: tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java URL

svn commit: r1527731 - /tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java

2013-09-30 Thread markt
Author: markt Date: Mon Sep 30 19:25:52 2013 New Revision: 1527731 URL: http://svn.apache.org/r1527731 Log: Add a warning comment and simplify the code a little. Modified: tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java Modified: tomcat/trunk/java/org/apache/tomcat/util/net/Apr

svn commit: r1527730 - /tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java

2013-09-30 Thread markt
Author: markt Date: Mon Sep 30 19:25:18 2013 New Revision: 1527730 URL: http://svn.apache.org/r1527730 Log: Refactor the remove list in the Poller to be a close list. The poller now removes the socket if it is in the Poller then destroys it. Note the socket is always destroyed whether it is found

svn commit: r1527728 - /tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java

2013-09-30 Thread markt
Author: markt Date: Mon Sep 30 19:24:43 2013 New Revision: 1527728 URL: http://svn.apache.org/r1527728 Log: Rename method Modified: tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java Modified: tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java URL: http://svn.apache.or

svn commit: r1527727 - /tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java

2013-09-30 Thread markt
Author: markt Date: Mon Sep 30 19:24:07 2013 New Revision: 1527727 URL: http://svn.apache.org/r1527727 Log: Move the point where the socket is removed from the connections Map so all the actions apart from removal from the Poller are in a single method to facilitate refactoring removal of the sock

Re: Possible IIS SPDY Redirector for Tomcat

2013-09-30 Thread Christopher Schultz
Konstantin, On 9/30/13 11:32 AM, Konstantin Preißer wrote: > Hi Chris, > >> -Original Message- >> From: Christopher Schultz [mailto:ch...@christopherschultz.net] >> Sent: Monday, September 30, 2013 5:19 PM >> To: Tomcat Developers List >> Subject: Re: Possible IIS SPDY Redirector for Tomc

Re: Recent tcnative null-dereference with 8.0.0-RC3 and 7.0.45 [tcnative-1.dll+0x7e23]

2013-09-30 Thread Christopher Schultz
Mladen, On 9/30/13 1:42 PM, Mladen Turk wrote: > On 09/30/2013 05:33 PM, Christopher Schultz wrote: >> >> I may need a little more guidance, since adding NULL-checks for >> everything is a little silly. Here is the function declaration in >> tcnative: >> >> TCN_IMPLEMENT_CALL(jint, Poll, poll)(TCN

[Bug 55613] Contexts in webapp/META-INF should be observed even if an equivalent Context appears in server.xml

2013-09-30 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=55613 Mark Thomas changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Bug 55613] Contexts in webapp/META-INF should be observed even if an equivalent Context appears in server.xml

2013-09-30 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=55613 Tony Weston changed: What|Removed |Added OS||All --- Comment #1 from Tony Weston

[Bug 55613] New: Contexts in webapp/META-INF should be observed even if an equivalent Context appears in server.xml

2013-09-30 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=55613 Bug ID: 55613 Summary: Contexts in webapp/META-INF should be observed even if an equivalent Context appears in server.xml Product: Tomcat 7 Version: trunk Hardware: P

Re: Recent tcnative null-dereference with 8.0.0-RC3 and 7.0.45 [tcnative-1.dll+0x7e23]

2013-09-30 Thread Mladen Turk
On 09/30/2013 05:33 PM, Christopher Schultz wrote: I may need a little more guidance, since adding NULL-checks for everything is a little silly. Here is the function declaration in tcnative: TCN_IMPLEMENT_CALL(jint, Poll, poll)(TCN_STDARGS, jlong pollset, j

Re: Recent tcnative null-dereference with 8.0.0-RC3 and 7.0.45 [tcnative-1.dll+0x7e23]

2013-09-30 Thread Christopher Schultz
Mark, On 9/30/13 9:34 AM, Mark Thomas wrote: > On 30/09/2013 08:32, Mark Thomas wrote: >> 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 tr

RE: Possible IIS SPDY Redirector for Tomcat

2013-09-30 Thread Konstantin Preißer
Hi Chris, > -Original Message- > From: Christopher Schultz [mailto:ch...@christopherschultz.net] > Sent: Monday, September 30, 2013 5:19 PM > To: Tomcat Developers List > Subject: Re: Possible IIS SPDY Redirector for Tomcat > > Konstantin, > > On 9/28/13 1:40 PM, Konstantin Preißer wrote

Re: Recent tcnative null-dereference with 8.0.0-RC3 and 7.0.45 [tcnative-1.dll+0x7e23]

2013-09-30 Thread Mladen Turk
On 09/30/2013 03:34 PM, Mark Thomas wrote: I'd like to try and make the changes as a series of small, incremental, easy to review changes. I'm not sure how feasible that will be. I hope to have this fixed in the next day or so. Cool. Not sure if socket wrapper is created for each native socke

Re: Possible IIS SPDY Redirector for Tomcat

2013-09-30 Thread Christopher Schultz
Konstantin, On 9/28/13 1:40 PM, Konstantin Preißer wrote: > Contra: > - Worse performance than the native ISAPI redirector. I made a test by a > Servlet that produces 700 MB of random data, and on a separate machine I used > a program to make a HTTP request and read the data. The average transfe

[Bug 55608] fmt:bundle tag unnecessarily buffers body content

2013-09-30 Thread bugzilla
https://issues.apache.org/bugzilla/show_bug.cgi?id=55608 --- Comment #3 from Christopher Schultz --- One could argue that might actually be used to intentionally buffer content a bit, just in case an error occurs. Once the response is committed, errors occurring during rendering are usually cata

Re: Recent tcnative null-dereference with 8.0.0-RC3 and 7.0.45 [tcnative-1.dll+0x7e23]

2013-09-30 Thread Mark Thomas
On 30/09/2013 08:32, Mark Thomas wrote: > 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 connect

buildbot failure in ASF Buildbot on tomcat-7-trunk

2013-09-30 Thread buildbot
The Buildbot has detected a new failure on builder tomcat-7-trunk while building ASF Buildbot. Full details are available at: http://ci.apache.org/builders/tomcat-7-trunk/builds/1433 Buildbot URL: http://ci.apache.org/ Buildslave for this Build: bb-vm_ubuntu Build Reason: scheduler Build Sourc

svn commit: r1527504 - /tomcat/tc7.0.x/trunk/java/org/apache/tomcat/util/net/res/LocalStrings.properties

2013-09-30 Thread markt
Author: markt Date: Mon Sep 30 10:01:38 2013 New Revision: 1527504 URL: http://svn.apache.org/r1527504 Log: Better debug messages Modified: tomcat/tc7.0.x/trunk/java/org/apache/tomcat/util/net/res/LocalStrings.properties Modified: tomcat/tc7.0.x/trunk/java/org/apache/tomcat/util/net/res/Lo

svn commit: r1527503 - /tomcat/trunk/java/org/apache/tomcat/util/net/res/LocalStrings.properties

2013-09-30 Thread markt
Author: markt Date: Mon Sep 30 10:00:35 2013 New Revision: 1527503 URL: http://svn.apache.org/r1527503 Log: Better debug messages Modified: tomcat/trunk/java/org/apache/tomcat/util/net/res/LocalStrings.properties Modified: tomcat/trunk/java/org/apache/tomcat/util/net/res/LocalStrings.proper

svn commit: r1527495 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/tomcat/util/net/res/LocalStrings.properties

2013-09-30 Thread markt
Author: markt Date: Mon Sep 30 09:16:58 2013 New Revision: 1527495 URL: http://svn.apache.org/r1527495 Log: Add missing bracket Modified: tomcat/tc7.0.x/trunk/ (props changed) tomcat/tc7.0.x/trunk/java/org/apache/tomcat/util/net/res/LocalStrings.properties Propchange: tomcat/tc7.0.x/t

svn commit: r1527493 - /tomcat/trunk/java/org/apache/tomcat/util/net/res/LocalStrings.properties

2013-09-30 Thread markt
Author: markt Date: Mon Sep 30 09:16:24 2013 New Revision: 1527493 URL: http://svn.apache.org/r1527493 Log: Add missing bracket Modified: tomcat/trunk/java/org/apache/tomcat/util/net/res/LocalStrings.properties Modified: tomcat/trunk/java/org/apache/tomcat/util/net/res/LocalStrings.properti

svn commit: r1527483 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/tomcat/util/net/AprEndpoint.java

2013-09-30 Thread markt
Author: markt Date: Mon Sep 30 08:18:54 2013 New Revision: 1527483 URL: http://svn.apache.org/r1527483 Log: Fix copy/paste error Modified: tomcat/tc7.0.x/trunk/ (props changed) tomcat/tc7.0.x/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java Propchange: tomcat/tc7.0.x/trunk/ -

svn commit: r1527480 - /tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java

2013-09-30 Thread markt
Author: markt Date: Mon Sep 30 08:11:34 2013 New Revision: 1527480 URL: http://svn.apache.org/r1527480 Log: Fix copy/paste error Modified: tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java Modified: tomcat/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java URL: http://svn.ap

Re: [VOTE] Release Apache Tomcat 7.0.45

2013-09-30 Thread Mark Thomas
On 26/09/2013 19:46, Konstantin Preißer wrote: > Hi Rainer, > >> -Original Message- >> From: Rainer Jung [mailto:rainer.j...@kippdata.de] >> Sent: Thursday, September 26, 2013 8:22 PM > >> Any chance you can try current trunk? Mark added more fixes to >> AprEndpoint after r1523781. It wou

svn commit: r1527475 - in /tomcat/tc7.0.x/trunk/webapps/docs: changelog.xml cluster-howto.xml config/cluster-valve.xml

2013-09-30 Thread kfujino
Author: kfujino Date: Mon Sep 30 07:54:55 2013 New Revision: 1527475 URL: http://svn.apache.org/r1527475 Log: Add document for sessionIdAttribute attribute in JvmRouteBinderValve. Modified: tomcat/tc7.0.x/trunk/webapps/docs/changelog.xml tomcat/tc7.0.x/trunk/webapps/docs/cluster-howto.xml

svn commit: r1527474 - /tomcat/trunk/webapps/docs/cluster-howto.xml

2013-09-30 Thread kfujino
Author: kfujino Date: Mon Sep 30 07:46:14 2013 New Revision: 1527474 URL: http://svn.apache.org/r1527474 Log: Fix default value of sessionIdAttribute. Modified: tomcat/trunk/webapps/docs/cluster-howto.xml Modified: tomcat/trunk/webapps/docs/cluster-howto.xml URL: http://svn.apache.org/viewv

svn commit: r1527472 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/tomcat/util/net/AprEndpoint.java webapps/docs/changelog.xml

2013-09-30 Thread markt
Author: markt Date: Mon Sep 30 07:45:09 2013 New Revision: 1527472 URL: http://svn.apache.org/r1527472 Log: Removal of a socket from the poller needs to happen on the poller thread as removal, like addition, is not thread safe. Removal of a socket from the poller needs to remove the socket from t

svn commit: r1527470 - in /tomcat/tc7.0.x/trunk: ./ java/org/apache/tomcat/util/net/AprEndpoint.java webapps/docs/changelog.xml

2013-09-30 Thread markt
Author: markt Date: Mon Sep 30 07:43:19 2013 New Revision: 1527470 URL: http://svn.apache.org/r1527470 Log: Narrow scope of localAddList Modified: tomcat/tc7.0.x/trunk/ (props changed) tomcat/tc7.0.x/trunk/java/org/apache/tomcat/util/net/AprEndpoint.java tomcat/tc7.0.x/trunk/webapps

svn commit: r1527468 - /tomcat/trunk/webapps/docs/config/cluster-valve.xml

2013-09-30 Thread kfujino
Author: kfujino Date: Mon Sep 30 07:37:53 2013 New Revision: 1527468 URL: http://svn.apache.org/r1527468 Log: Add document for sessionIdAttribute attribute in JvmRouteBinderValve. Modified: tomcat/trunk/webapps/docs/config/cluster-valve.xml Modified: tomcat/trunk/webapps/docs/config/cluster-

Re: Recent tcnative null-dereference with 8.0.0-RC3 and 7.0.45 [tcnative-1.dll+0x7e23]

2013-09-30 Thread Mark Thomas
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 th