[master] c108e6031 Re-apply tools/coccinelle/check_obj.cocci
Nils Goroll
nils.goroll at uplex.de
Tue Feb 9 16:15:09 UTC 2021
commit c108e603141bd8270090a91fec880d1a2805d56c
Author: Nils Goroll <nils.goroll at uplex.de>
Date: Tue Feb 9 16:51:27 2021 +0100
Re-apply tools/coccinelle/check_obj.cocci
After edb0ca9863431dc8ccc6c37e779c179fc29f53cf, the ladybug understands
our code better.
diff --git a/bin/varnishd/cache/cache_mempool.c b/bin/varnishd/cache/cache_mempool.c
index abd5d5daa..b6d69bd61 100644
--- a/bin/varnishd/cache/cache_mempool.c
+++ b/bin/varnishd/cache/cache_mempool.c
@@ -191,7 +191,7 @@ mpl_guard(void *priv)
if (mi == NULL) {
mi = VTAILQ_FIRST(&mpl->surplus);
if (mi != NULL) {
- CHECK_OBJ_NOTNULL(mi, MEMITEM_MAGIC);
+ CHECK_OBJ(mi, MEMITEM_MAGIC);
VTAILQ_REMOVE(&mpl->surplus, mi, list);
mpl_slp = .01; // random
}
diff --git a/bin/varnishd/cache/cache_vrt_priv.c b/bin/varnishd/cache/cache_vrt_priv.c
index 3aa166514..02ba46bdd 100644
--- a/bin/varnishd/cache/cache_vrt_priv.c
+++ b/bin/varnishd/cache/cache_vrt_priv.c
@@ -166,11 +166,11 @@ vrt_priv_task_context(VRT_CTX)
ctx->method == VCL_MET_PIPE || ctx->method == 0);
if (ctx->req) {
- CHECK_OBJ_NOTNULL(ctx->req, REQ_MAGIC);
+ CHECK_OBJ(ctx->req, REQ_MAGIC);
return (ctx->req->privs);
}
if (ctx->bo) {
- CHECK_OBJ_NOTNULL(ctx->bo, BUSYOBJ_MAGIC);
+ CHECK_OBJ(ctx->bo, BUSYOBJ_MAGIC);
return (ctx->bo->privs);
}
ASSERT_CLI();
diff --git a/bin/varnishd/cache/cache_vrt_vcl.c b/bin/varnishd/cache/cache_vrt_vcl.c
index cda736eaa..b7ecdd7ff 100644
--- a/bin/varnishd/cache/cache_vrt_vcl.c
+++ b/bin/varnishd/cache/cache_vrt_vcl.c
@@ -457,7 +457,7 @@ vcl_call_method(struct worker *wrk, struct req *req, struct busyobj *bo,
CHECK_OBJ_NOTNULL(wrk, WORKER_MAGIC);
INIT_OBJ(&ctx, VRT_CTX_MAGIC);
if (bo != NULL) {
- CHECK_OBJ_NOTNULL(bo, BUSYOBJ_MAGIC);
+ CHECK_OBJ(bo, BUSYOBJ_MAGIC);
CHECK_OBJ_NOTNULL(bo->vcl, VCL_MAGIC);
VCL_Bo2Ctx(&ctx, bo);
}
diff --git a/vmod/vmod_std.c b/vmod/vmod_std.c
index 1cf60162c..bb24e04ce 100644
--- a/vmod/vmod_std.c
+++ b/vmod/vmod_std.c
@@ -224,7 +224,7 @@ vmod_timestamp(VRT_CTX, VCL_STRING label)
VSLb_ts_busyobj(ctx->bo, label, VTIM_real());
} else if (ctx->req != NULL) {
/* Called from request vcl methods */
- CHECK_OBJ_NOTNULL(ctx->req, REQ_MAGIC);
+ CHECK_OBJ(ctx->req, REQ_MAGIC);
VSLb_ts_req(ctx->req, label, VTIM_real());
}
}
More information about the varnish-commit
mailing list