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;