I have not been able to reproduce this exact circumstance, but the

fix is readily obvious, so: Tighten requirement for range delivery. Fixes #1007

I have not been able to reproduce this exact circumstance, but the
fix is readily obvious, so: Tighten requirement for range delivery. Fixes #1007
85482168 · Poul-Henning Kamp · db1a21ff · 85482168
Commit 85482168 authored Sep 06, 2011 by Poul-Henning Kamp
Hide whitespace changes
Inline Side-by-side

Showing with 7 additions and 3 deletions

cache_response.c bin/varnishd/cache_response.c +7 -3

No files found.
--- a/bin/varnishd/cache_response.c
+++ b/bin/varnishd/cache_response.c
@@ -276,9 +276,13 @@ RES_WriteObj(struct sess *sp)
 	 */
 	low = 0;
 	high = sp->obj->len - 1;
-	if (!(sp->wrk->res_mode & (RES_ESI|RES_ESI_CHILD|RES_GUNZIP)) &&
+	if (
-	    params->http_range_support && sp->obj->response == 200 &&
+	    sp->wantbody &&
-	    sp->wantbody && http_GetHdr(sp->http, H_Range, &r))
+	    (sp->wrk->res_mode & RES_LEN) &&
+	    !(sp->wrk->res_mode & (RES_ESI|RES_ESI_CHILD|RES_GUNZIP)) &&
+	    params->http_range_support &&
+	    sp->obj->response == 200 &&
+	    http_GetHdr(sp->http, H_Range, &r))
 		res_dorange(sp, r, &low, &high);
 	/*