The attached patch fixes the problem by eliminating the additional ","
as seperator. I found no sign why this was added in the first place
after some code reconstruction. Also it makes the code reentrant.

Bastian

-- 
We have phasers, I vote we blast 'em!
                -- Bailey, "The Corbomite Maneuver", stardate 1514.2
#! /bin/sh /usr/share/dpatch/dpatch-run
##

@DPATCH@
diff -urN squid-2.7.STABLE3.old/src/HttpHeaderTools.c 
squid-2.7.STABLE3/src/HttpHeaderTools.c
--- squid-2.7.STABLE3.old/src/HttpHeaderTools.c 2008-04-02 03:00:11.000000000 
+0200
+++ squid-2.7.STABLE3/src/HttpHeaderTools.c     2009-07-06 12:16:42.000000000 
+0200
@@ -239,11 +239,11 @@
 strListGetItem(const String * str, char del, const char **item, int *ilen, 
const char **pos)
 {
     size_t len;
-    static char delim[3][8] =
+    char delim[3][8] =
     {
-       "\"?,",
+       "\"?",
        "\"\\",
-       " ?,\t\r\n"
+       " ?\t\r\n"
     };
     int quoted = 0;
     delim[0][1] = del;

Attachment: signature.asc
Description: Digital signature

Reply via email to