Author: billbarker Date: Sun May 20 15:28:47 2007 New Revision: 539971 URL: http://svn.apache.org/viewvc?view=rev&rev=539971 Log: Always reset the MB when doing getBytes
Fix for bug #36155 1) an unconditional reset is cheap if I'm going to call MB.setBytes 2) the JK connector doesn't support any charset except iso-latin-1 anyway 3) This particular connector is on the fast track to deprecated Modified: tomcat/tc6.0.x/trunk/java/org/apache/jk/common/MsgAjp.java Modified: tomcat/tc6.0.x/trunk/java/org/apache/jk/common/MsgAjp.java URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/java/org/apache/jk/common/MsgAjp.java?view=diff&rev=539971&r1=539970&r2=539971 ============================================================================== --- tomcat/tc6.0.x/trunk/java/org/apache/jk/common/MsgAjp.java (original) +++ tomcat/tc6.0.x/trunk/java/org/apache/jk/common/MsgAjp.java Sun May 20 15:28:47 2007 @@ -237,8 +237,8 @@ public void getBytes(MessageBytes mb) { int length = getInt(); + mb.recycle(); if( (length == 0xFFFF) || (length == -1) ) { - mb.recycle(); return; } mb.setBytes( buf, pos, length ); --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]