Author: markt Date: Mon Aug 27 21:39:51 2012 New Revision: 1377883 URL: http://svn.apache.org/viewvc?rev=1377883&view=rev Log: Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=53607 To avoid NPE, set TCP PING data to ChannelMessage.
Modified: tomcat/tc6.0.x/trunk/STATUS.txt tomcat/tc6.0.x/trunk/java/org/apache/catalina/tribes/group/interceptors/TcpPingInterceptor.java tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml Modified: tomcat/tc6.0.x/trunk/STATUS.txt URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=1377883&r1=1377882&r2=1377883&view=diff ============================================================================== --- tomcat/tc6.0.x/trunk/STATUS.txt (original) +++ tomcat/tc6.0.x/trunk/STATUS.txt Mon Aug 27 21:39:51 2012 @@ -124,12 +124,6 @@ PATCHES PROPOSED TO BACKPORT: +1: kfujino, kkolinko, markt -1: -* Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=53607 - To avoid NPE, set TCP PING data to ChannelMessage. - http://svn.apache.org/viewvc?view=revision&revision=1367456 - +1: kfujino, kkolinko, markt - -1: - * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=53481 Add support for SSLHonorCipherOrder http://svn.apache.org/viewvc?view=revision&revision=1371298 Modified: tomcat/tc6.0.x/trunk/java/org/apache/catalina/tribes/group/interceptors/TcpPingInterceptor.java URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/catalina/tribes/group/interceptors/TcpPingInterceptor.java?rev=1377883&r1=1377882&r2=1377883&view=diff ============================================================================== --- tomcat/tc6.0.x/trunk/java/org/apache/catalina/tribes/group/interceptors/TcpPingInterceptor.java (original) +++ tomcat/tc6.0.x/trunk/java/org/apache/catalina/tribes/group/interceptors/TcpPingInterceptor.java Mon Aug 27 21:39:51 2012 @@ -27,6 +27,7 @@ import org.apache.catalina.tribes.Channe import org.apache.catalina.tribes.Member; import org.apache.catalina.tribes.group.ChannelInterceptorBase; import org.apache.catalina.tribes.io.ChannelData; +import org.apache.catalina.tribes.io.XByteBuffer; /** * @@ -141,6 +142,7 @@ public class TcpPingInterceptor extends data.setAddress(getLocalMember(false)); data.setTimestamp(System.currentTimeMillis()); data.setOptions(getOptionFlag()); + data.setMessage(new XByteBuffer(TCP_PING_DATA, false)); try { super.sendMessage(members, data, null); }catch (ChannelException x) { Modified: tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml?rev=1377883&r1=1377882&r2=1377883&view=diff ============================================================================== --- tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml (original) +++ tomcat/tc6.0.x/trunk/webapps/docs/changelog.xml Mon Aug 27 21:39:51 2012 @@ -269,6 +269,10 @@ <bug>53606</bug>: Fix potential NPE in <code>TcpPingInterceptor</code>. Based on a patch by F. Arnoud. (markt) </fix> + <fix> + <bug>53607</bug>: To avoid NPE, set TCP PING data to ChannelMessage. + Patch provided by F.Arnoud (kfujino) + </fix> </changelog> </subsection> <subsection name="Web applications"> --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org