You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by je...@apache.org on 2002/02/03 18:43:36 UTC
cvs commit: httpd-2.0/modules/http http_protocol.c
jerenkrantz 02/02/03 09:43:36
Modified: modules/http http_protocol.c
Log:
Oh, no. It's legal to have 0 bytes read. Just not -1. If *that* happens,
I want to see a core. 0 bytes reads are fine. Grr.
(I'm so tempted to throw this assert out, but if we get -1 length
brigades back, we're gonna be screwed.)
Revision Changes Path
1.391 +1 -1 httpd-2.0/modules/http/http_protocol.c
Index: http_protocol.c
===================================================================
RCS file: /home/cvs/httpd-2.0/modules/http/http_protocol.c,v
retrieving revision 1.390
retrieving revision 1.391
diff -u -r1.390 -r1.391
--- http_protocol.c 25 Jan 2002 01:11:46 -0000 1.390
+++ http_protocol.c 3 Feb 2002 17:43:35 -0000 1.391
@@ -661,7 +661,7 @@
/* If this happens, we have a bucket of unknown length. Die because
* it means our assumptions have changed. */
- AP_DEBUG_ASSERT(totalread > 0);
+ AP_DEBUG_ASSERT(totalread >= 0);
if (ctx->state != BODY_NONE) {
ctx->remaining -= totalread;