Bug 4735: Truncated chunked responses cached as whole (#528)

Mark responses received without the last chunk as responses that have
bad (and, hence, unknown) message body length (i.e. ENTRY_BAD_LENGTH).
If they were being cached, such responses will be released and will stop
being shareable.

diff --git a/src/http.cc b/src/http.cc
index 742cf65d434b66a24cf7ee44d24214d2e2aac59c..9c0601b0a645fbfb39c0e87fdc604172a55f97c8 100644 (file)
--- a/src/http.cc
+++ b/src/http.cc
@@ -1505,6 +1505,9 @@ HttpStateData::processReplyBody()
 
         case COMPLETE_NONPERSISTENT_MSG:
             debugs(11, 5, "processReplyBody: COMPLETE_NONPERSISTENT_MSG from " << serverConnection);
+            if (flags.chunked && !lastChunk)
+                entry->lengthWentBad("missing last-chunk");
+
             serverComplete();
             return;
         }