From: hno <>
Date: Sat, 12 Oct 2002 16:50:55 +0000 (+0000)
Subject: Removed offending copying of reply data, messing up reply headers.
X-Git-Tag: SQUID_3_0_PRE1~678
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=e34201d83bb090bf95c3468d350f86a8fc9d9cb5;p=thirdparty%2Fsquid.git

Removed offending copying of reply data, messing up reply headers.
Code left commented out awaiting comment from Robert.
---

diff --git a/src/client_side_reply.cc b/src/client_side_reply.cc
index 8d28cba7ae..6b2de7d0c4 100644
--- a/src/client_side_reply.cc
+++ b/src/client_side_reply.cc
@@ -1,6 +1,6 @@
 
 /*
- * $Id: client_side_reply.cc,v 1.13 2002/10/03 09:47:58 hno Exp $
+ * $Id: client_side_reply.cc,v 1.14 2002/10/12 10:50:55 hno Exp $
  *
  * DEBUG: section 88    Client-side Reply Routines
  * AUTHOR: Robert Collins (Originally Duane Wessels in client_side.c)
@@ -1416,12 +1416,18 @@ clientSendMoreData(void *data, StoreIOBuffer result)
      */
     assert(context->reqofs == 0 || context->flags.storelogiccomplete);
 
+#if THIS_DOES_NOT_WORK
+    /* XXX What is the purpose of this code? It overwrites the
+     * previous "packet" completely messing up header processing
+     * when headers are received in more than one chunk..
+     */
     if (buf != result.data) {
 	/* we've got to copy some data */
 	assert(result.length <= next->readBuffer.length);
 	xmemcpy(buf, result.data, result.length);
 	body_buf = buf;
     }
+#endif
     /* We've got the final data to start pushing... */
     context->flags.storelogiccomplete = 1;