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;
signature.asc
Description: Digital signature
