[4.1] 36f80ab Always close the VFP stack, also on BS_NONE.

Lasse Karstensen lkarsten at varnish-software.com
Fri Mar 4 15:53:47 CET 2016


commit 36f80ab5c2cb743c78d67c6e0771c3ff972568e2
Author: Poul-Henning Kamp <phk at FreeBSD.org>
Date:   Fri Feb 26 17:13:42 2016 +0000

    Always close the VFP stack, also on BS_NONE.
    
    Fixes:	#1860

diff --git a/bin/varnishd/cache/cache_fetch.c b/bin/varnishd/cache/cache_fetch.c
index 1800b71..1a05c8f 100644
--- a/bin/varnishd/cache/cache_fetch.c
+++ b/bin/varnishd/cache/cache_fetch.c
@@ -519,8 +519,6 @@ vbf_fetch_body_helper(struct busyobj *bo)
 		}
 	} while (vfps == VFP_OK);
 
-	VFP_Close(vfc);
-
 	if (vfps == VFP_ERROR) {
 		AN(vfc->failed);
 		(void)VFP_Error(vfc, "Fetch pipeline failed to process");
@@ -674,6 +672,8 @@ vbf_stp_fetch(struct worker *wrk, struct busyobj *bo)
 		vbf_fetch_body_helper(bo);
 	}
 
+	VFP_Close(bo->vfc);
+
 	if (bo->vfc->failed) {
 		VDI_Finish(bo->wrk, bo);
 		if (!bo->do_stream) {



More information about the varnish-commit mailing list