[master] 832d6da2d Guard all the VDP_Push calls

Dridi Boukelmoune dridi.boukelmoune at gmail.com
Fri Feb 8 09:23:08 UTC 2019


commit 832d6da2d732636b42a78920980a84b3e33bda9b
Author: Dridi Boukelmoune <dridi.boukelmoune at gmail.com>
Date:   Fri Feb 8 10:21:41 2019 +0100

    Guard all the VDP_Push calls
    
    Two of them have no init step, and the third one is fail-safe.

diff --git a/bin/varnishd/cache/cache_esi_deliver.c b/bin/varnishd/cache/cache_esi_deliver.c
index eeb4d1e79..9279baaa7 100644
--- a/bin/varnishd/cache/cache_esi_deliver.c
+++ b/bin/varnishd/cache/cache_esi_deliver.c
@@ -833,14 +833,14 @@ ved_deliver(struct req *req, struct boc *boc, int wantbody)
 
 		INIT_OBJ(foo, VED_FOO_MAGIC);
 		foo->ecx = ecx;
-		(void)VDP_Push(req, &ved_gzgz, foo);
+		AZ(VDP_Push(req, &ved_gzgz, foo));
 
 	} else if (ecx->isgzip && !i) {
 		/* Non-Gzip'ed include in gzip'ed parent */
-		(void)VDP_Push(req, &ved_pretend_gz, ecx);
+		AZ(VDP_Push(req, &ved_pretend_gz, ecx));
 	} else {
 		/* Anything else goes straight through */
-		(void)VDP_Push(req, &ved_ved, ecx);
+		AZ(VDP_Push(req, &ved_ved, ecx));
 	}
 	(void)VDP_DeliverObj(req);
 	(void)VDP_bytes(req, VDP_FLUSH, NULL, 0);


More information about the varnish-commit mailing list