From: Ruediger Pluem <rpluem@apache.org>
Date: Sat, 3 Dec 2005 20:17:11 +0000 (+0000)
Subject: * Fix PR37753. Off by one bug in get_path_param eats up too much of the string
X-Git-Tag: 2.3.0~2697
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=e7086045557dfd6292ba4fe50ff48cae36de858c;p=thirdparty%2Fapache%2Fhttpd.git

* Fix PR37753. Off by one bug in get_path_param eats up too much of the string
  such that session identifier for sticky sessions is never found in URL.

Submitted by: Kazuhiro Osawa <ko yappo.ne.jp>
Reviewed by: Ruediger Pluem


git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@352010 13f79535-47bb-0310-9956-ffa450edef68
---

diff --git a/CHANGES b/CHANGES
index 06c597bc256..3fec516c175 100644
--- a/CHANGES
+++ b/CHANGES
@@ -2,6 +2,9 @@
 Changes with Apache 2.3.0
   [Remove entries to the current 2.0 and 2.2 section below, when backported]
 
+  *) mod_proxy_balancer: Sticky session identifiers supplied in URL do not work.
+     PR37753. [Ruediger Pluem, Kazuhiro Osawa <ko yappo.ne.jp>]
+
   *) mod_authz_dbd: SQL authz with Login/Session support [Nick Kew]
 
   *) Fix typo in ProxyStatus syntax error message.
diff --git a/modules/proxy/mod_proxy_balancer.c b/modules/proxy/mod_proxy_balancer.c
index 988508adf34..def1919254d 100644
--- a/modules/proxy/mod_proxy_balancer.c
+++ b/modules/proxy/mod_proxy_balancer.c
@@ -113,7 +113,7 @@ static char *get_path_param(apr_pool_t *pool, char *url,
     char *path = NULL;
 
     for (path = strstr(url, name); path; path = strstr(path + 1, name)) {
-        path += (strlen(name) + 1);
+        path += strlen(name);
         if (*path == '=') {
             /*
              * Session path was found, get it's value