[master] 9804794a0 side effect of staring at code not worth a ticket

Nils Goroll nils.goroll at uplex.de
Mon Sep 3 16:16:09 UTC 2018


commit 9804794a0e876800e4e3ca10e965b568bd37c38e
Author: Nils Goroll <nils.goroll at uplex.de>
Date:   Mon Sep 3 18:11:42 2018 +0200

    side effect of staring at code not worth a ticket

diff --git a/bin/varnishd/cache/cache_hash.c b/bin/varnishd/cache/cache_hash.c
index c3b652bbe..1b63d906d 100644
--- a/bin/varnishd/cache/cache_hash.c
+++ b/bin/varnishd/cache/cache_hash.c
@@ -407,6 +407,8 @@ HSH_Lookup(struct req *req, struct objcore **ocp, struct objcore **bocp,
 			continue;
 
 		if (oc->boc != NULL && oc->boc->state < BOS_STREAM) {
+			// XXX does it make sense to have the CHECK_OBJ
+			//     if we access boc->state before it?
 			CHECK_OBJ_ORNULL(oc->boc, BOC_MAGIC);
 
 			if (req->hash_ignore_busy)


More information about the varnish-commit mailing list