You are viewing a plain text version of this content. The canonical link for it is here.
Posted to apreq-cvs@httpd.apache.org by jo...@apache.org on 2012/04/29 03:52:20 UTC

svn commit: r1331861 - in /httpd/apreq/trunk: include/apreq_version.h library/parser.c

Author: joes
Date: Sun Apr 29 01:52:20 2012
New Revision: 1331861

URL: http://svn.apache.org/viewvc?rev=1331861&view=rev
Log:
bug here in that the hook ptr fu is busted so we need
to ensure only the first matching param sets ctx->param

Modified:
    httpd/apreq/trunk/include/apreq_version.h
    httpd/apreq/trunk/library/parser.c

Modified: httpd/apreq/trunk/include/apreq_version.h
URL: http://svn.apache.org/viewvc/httpd/apreq/trunk/include/apreq_version.h?rev=1331861&r1=1331860&r2=1331861&view=diff
==============================================================================
--- httpd/apreq/trunk/include/apreq_version.h (original)
+++ httpd/apreq/trunk/include/apreq_version.h Sun Apr 29 01:52:20 2012
@@ -62,7 +62,7 @@ extern "C" {
 #define APREQ_MINOR_VERSION       8
 
 /** patch level */
-#define APREQ_PATCH_VERSION       0
+#define APREQ_PATCH_VERSION       1
 
 /**
  *  This symbol is defined for internal, "development" copies of libapreq.

Modified: httpd/apreq/trunk/library/parser.c
URL: http://svn.apache.org/viewvc/httpd/apreq/trunk/library/parser.c?rev=1331861&r1=1331860&r2=1331861&view=diff
==============================================================================
--- httpd/apreq/trunk/library/parser.c (original)
+++ httpd/apreq/trunk/library/parser.c Sun Apr 29 01:52:20 2012
@@ -347,7 +347,7 @@ APREQ_DECLARE_HOOK(apreq_hook_find_param
     apr_status_t s = (hook->next == NULL)
         ? APR_SUCCESS : apreq_hook_run(hook->next, param, bb);
 
-    if (is_final && s == APR_SUCCESS
+    if (is_final && s == APR_SUCCESS && ctx->param == NULL
         && strcasecmp(ctx->name, param->v.name) == 0) {
         ctx->param = param;
         ctx->prev->next = hook->next;