--------------------- PatchSet 6037 Date: 2003/11/14 20:10:48 Author: dwsquid Branch: icap-2_5 Tag: (none) Log: bugfix for the case when we don't get the entire HTTP response in the first read(). Instead of giving this to storeAppend() we can now hold onto the partial data in our own 'chunk_buf' and wait for the rest to arrive. Members: src/icap_respmod.c:1.1.2.25->1.1.2.26 Index: squid/src/icap_respmod.c =================================================================== RCS file: /cvsroot/squid-sf//squid/src/Attic/icap_respmod.c,v retrieving revision 1.1.2.25 retrieving revision 1.1.2.26 diff -u -r1.1.2.25 -r1.1.2.26 --- squid/src/icap_respmod.c 14 Nov 2003 17:28:46 -0000 1.1.2.25 +++ squid/src/icap_respmod.c 14 Nov 2003 20:10:48 -0000 1.1.2.26 @@ -1,5 +1,5 @@ /* - * $Id: icap_respmod.c,v 1.1.2.25 2003/11/14 17:28:46 dwsquid Exp $ + * $Id: icap_respmod.c,v 1.1.2.26 2003/11/14 20:10:48 dwsquid Exp $ * * DEBUG: section 81 Internet Content Adaptation Protocol (ICAP) Client * AUTHOR: Geetha Manjunath, Hewlett Packard Company @@ -833,8 +833,11 @@ assert(icapHttpReplyHdrState(icap) == 2); icap->chunk_size = 0; } else { - storeAppend(entry, icap->chunk_buf.buf, icap->chunk_buf.size); - so_far += icap->chunk_buf.size; + /* + * We don't have the full HTTP reply headers yet, so keep + * the partial reply buffered in 'chunk_buf' and wait + * for more. + */ assert(icapHttpReplyHdrState(icap) == 0); } }