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