svn commit: r292036 - in /httpd/apreq/trunk: CHANGES include/apreq_version.h library/parser_header.c library/t/parsers.c

[joes]

Author: joes
Date: Tue Sep 27 13:45:48 2005
New Revision: 292036

URL: http://svn.apache.org/viewcvs?rev=292036&view=rev
Log:
Fix off-by-one bug in the header parser wrt continuation lines.

Modified:
    httpd/apreq/trunk/CHANGES
    httpd/apreq/trunk/include/apreq_version.h
    httpd/apreq/trunk/library/parser_header.c
    httpd/apreq/trunk/library/t/parsers.c

Modified: httpd/apreq/trunk/CHANGES
URL: 
http://svn.apache.org/viewcvs/httpd/apreq/trunk/CHANGES?rev=292036&r1=292035&r2=292036&view=diff
==============================================================================
--- httpd/apreq/trunk/CHANGES (original)
+++ httpd/apreq/trunk/CHANGES Tue Sep 27 13:45:48 2005
@@ -5,6 +5,10 @@
 @section v2_07 Changes with libapreq2-2.07
 
 
+- C API [joes]
+  Fix off-by-one bug in the continuation-lines portion of the
+  header parser.
+
 - Perl API [joes]
   Move APR::Request::upload to APR::Request, where it belongs.
 

Modified: httpd/apreq/trunk/include/apreq_version.h
URL: 
http://svn.apache.org/viewcvs/httpd/apreq/trunk/include/apreq_version.h?rev=292036&r1=292035&r2=292036&view=diff
==============================================================================
--- httpd/apreq/trunk/include/apreq_version.h (original)
+++ httpd/apreq/trunk/include/apreq_version.h Tue Sep 27 13:45:48 2005
@@ -61,7 +61,7 @@
 #define APREQ_MINOR_VERSION       5
 
 /** patch level */
-#define APREQ_PATCH_VERSION       1
+#define APREQ_PATCH_VERSION       2
 
 /**
  *  This symbol is defined for internal, "development" copies of libapreq.

Modified: httpd/apreq/trunk/library/parser_header.c
URL: 
http://svn.apache.org/viewcvs/httpd/apreq/trunk/library/parser_header.c?rev=292036&r1=292035&r2=292036&view=diff
==============================================================================
--- httpd/apreq/trunk/library/parser_header.c (original)
+++ httpd/apreq/trunk/library/parser_header.c Tue Sep 27 13:45:48 2005
@@ -300,7 +300,7 @@
                 case '\t':
                     ctx->status = HDR_CONTINUE;
                     ++off;
-                    vlen += 2;
+                    ++vlen;
                     break;
 
                 default:

Modified: httpd/apreq/trunk/library/t/parsers.c
URL: 
http://svn.apache.org/viewcvs/httpd/apreq/trunk/library/t/parsers.c?rev=292036&r1=292035&r2=292036&view=diff
==============================================================================
--- httpd/apreq/trunk/library/t/parsers.c (original)
+++ httpd/apreq/trunk/library/t/parsers.c Tue Sep 27 13:45:48 2005
@@ -39,7 +39,7 @@
 "... contents of file1.txt ..." CRLF CRLF
 "--AaB03x" CRLF
 "content-disposition: form-data; name=\"\"" CRLF
-"content-type: text/plain;charset=windows-1250" CRLF
+"content-type: text/plain;" CRLF " charset=windows-1250" CRLF
 "content-transfer-encoding: quoted-printable" CRLF CRLF
 "Joe owes =80100." CRLF
 "--AaB03x--" CRLF;