BUG/MEDIUM: url32 does not take the path part into account in the returned hash.

The url32 sample fetch does not take the path part of the URL into
account. This is because in smp_fetch_url32() we erroneously modify
path.len and path.ptr before testing their value and building the
path based part of the hash.

This fixes issue #235

This must be backported as far as 1.9, when HTX was introduced.

diff --git a/src/http_fetch.c b/src/http_fetch.c
index 922e25cc1adaff4e1b11fc34285b5f03121450d4..126c1a2a8d2f5a3c1341e4e5f7c8ad47ec3cf0c9 100644 (file)
--- a/src/http_fetch.c
+++ b/src/http_fetch.c
@@ -1786,10 +1786,6 @@ static int smp_fetch_url32(const struct arg *args, struct sample *smp, const cha
        /* now retrieve the path */
        sl = http_get_stline(htx);
        path = http_get_path(htx_sl_req_uri(sl));
-       while (path.len > 0 && *(path.ptr) != '?') {
-               path.ptr++;
-               path.len--;
-       }
        if (path.len && *(path.ptr) == '/') {
                while (path.len--)
                        hash = *(path.ptr++) + (hash << 6) + (hash << 16) - hash;