[master] a2a5cf1ba VDP_Push itself is not fail-safe

Dridi Boukelmoune dridi.boukelmoune at gmail.com
Sat Feb 16 09:12:08 UTC 2019


commit a2a5cf1ba9c21d510699852d99a6864d8b68b9d6
Author: Dridi Boukelmoune <dridi.boukelmoune at gmail.com>
Date:   Sat Feb 16 10:10:07 2019 +0100

    VDP_Push itself is not fail-safe
    
    Refs 832d6da2d732636b42a78920980a84b3e33bda9b

diff --git a/bin/varnishd/cache/cache_esi_deliver.c b/bin/varnishd/cache/cache_esi_deliver.c
index 9279baaa7..df30f0e8c 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;
-		AZ(VDP_Push(req, &ved_gzgz, foo));
+		XXXAZ(VDP_Push(req, &ved_gzgz, foo));
 
 	} else if (ecx->isgzip && !i) {
 		/* Non-Gzip'ed include in gzip'ed parent */
-		AZ(VDP_Push(req, &ved_pretend_gz, ecx));
+		XXXAZ(VDP_Push(req, &ved_pretend_gz, ecx));
 	} else {
 		/* Anything else goes straight through */
-		AZ(VDP_Push(req, &ved_ved, ecx));
+		XXXAZ(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