You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by la...@apache.org on 2001/06/11 22:07:03 UTC

cvs commit: jakarta-tomcat-connectors/util/java/org/apache/tomcat/util/http Parameters.java

larryi      01/06/11 13:07:03

  Modified:    util/java/org/apache/tomcat/util/http Parameters.java
  Log:
  Update processParameters() methods to not allow valStart > valEnd if
  value isn't specified.
  
  Revision  Changes    Path
  1.3       +3 -3      jakarta-tomcat-connectors/util/java/org/apache/tomcat/util/http/Parameters.java
  
  Index: Parameters.java
  ===================================================================
  RCS file: /home/cvs/jakarta-tomcat-connectors/util/java/org/apache/tomcat/util/http/Parameters.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- Parameters.java	2001/05/29 06:22:54	1.2
  +++ Parameters.java	2001/06/11 20:07:02	1.3
  @@ -360,7 +360,7 @@
   	    
   	    int valStart=nameEnd+1;
   	    int valEnd=ByteChunk.indexOf(bytes, valStart, end, '&');
  -	    if( valEnd== -1 ) valEnd=end;
  +	    if( valEnd== -1 ) valEnd = (valStart < end) ? end : valStart;
   	    
   	    pos=valEnd+1;
   	    
  @@ -411,7 +411,7 @@
   
   	    int valStart=nameEnd+1;
   	    int valEnd=CharChunk.indexOf(chars, valStart, end, '&');
  -	    if( valEnd== -1 ) valEnd=end;
  +	    if( valEnd== -1 ) valEnd = (valStart < end) ? end : valStart;
   	    pos=valEnd+1;
   	    
   	    if( nameEnd<=nameStart ) {
  @@ -502,7 +502,7 @@
   
   	    int valStart=nameEnd+1;
   	    int valEnd=str.indexOf('&', valStart);
  -	    if( valEnd== -1 ) valEnd=end;
  +	    if( valEnd== -1 ) valEnd = (valStart < end) ? end : valStart;
   	    pos=valEnd+1;
   	    
   	    if( nameEnd<=nameStart ) {