You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by cr...@locus.apache.org on 2000/09/14 02:12:42 UTC

cvs commit: jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/util RequestUtil.java

craigmcc    00/09/13 17:12:41

  Modified:    catalina/src/share/org/apache/catalina/util RequestUtil.java
  Log:
  Do *not* stop parsing request parameters when a zero-length parameter value
  (such as an empty text field on a form) is encountered.  Previously, such a
  value would cause all subsequent request parameters on this request to be
  ignored.
  
  Revision  Changes    Path
  1.4       +6 -6      jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/util/RequestUtil.java
  
  Index: RequestUtil.java
  ===================================================================
  RCS file: /home/cvs/jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/util/RequestUtil.java,v
  retrieving revision 1.3
  retrieving revision 1.4
  diff -u -r1.3 -r1.4
  --- RequestUtil.java	2000/09/01 18:17:16	1.3
  +++ RequestUtil.java	2000/09/14 00:12:41	1.4
  @@ -1,7 +1,7 @@
   /*
  - * $Header: /home/cvs/jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/util/RequestUtil.java,v 1.3 2000/09/01 18:17:16 craigmcc Exp $
  - * $Revision: 1.3 $
  - * $Date: 2000/09/01 18:17:16 $
  + * $Header: /home/cvs/jakarta-tomcat-4.0/catalina/src/share/org/apache/catalina/util/RequestUtil.java,v 1.4 2000/09/14 00:12:41 craigmcc Exp $
  + * $Revision: 1.4 $
  + * $Date: 2000/09/14 00:12:41 $
    *
    * ====================================================================
    *
  @@ -78,7 +78,7 @@
    * General purpose request parsing and encoding utility methods.
    *
    * @author Craig R. McClanahan
  - * @version $Revision: 1.3 $ $Date: 2000/09/01 18:17:16 $
  + * @version $Revision: 1.4 $ $Date: 2000/09/14 00:12:41 $
    */
   
   public final class RequestUtil {
  @@ -246,8 +246,8 @@
   	    name = parser.extract(nameStart, nameEnd);
   	    value = parser.extract(valueStart, valueEnd);
   
  -	    // A zero-length name or value means we are done
  -	    if ((name.length() < 1) || (value.length() < 1))
  +	    // A zero-length name means we are done
  +	    if (name.length() < 1)
   	        break;
   
   	    // Create or update the array of values for this name