Commit 7ce983ec authored by Nils Goroll's avatar Nils Goroll

more TAKE_OBJ_NOTNULLification

command:

spatch -I include/ -I bin/varnishd/ --dir . --in-place \
	--sp-file tools/coccinelle/take_obj_notnull.cocci
parent 4fe88185
......@@ -980,12 +980,8 @@ HSH_DerefObjCore(struct worker *wrk, struct objcore **ocp, int rushmax)
struct rush rush;
unsigned r;
AN(ocp);
oc = *ocp;
*ocp = NULL;
CHECK_OBJ_NOTNULL(wrk, WORKER_MAGIC);
CHECK_OBJ_NOTNULL(oc, OBJCORE_MAGIC);
TAKE_OBJ_NOTNULL(oc, ocp, OBJCORE_MAGIC);
assert(oc->refcnt > 0);
INIT_OBJ(&rush, RUSH_MAGIC);
......
......@@ -345,11 +345,7 @@ VRT_VCL_Allow_Cold(struct vclref **refp)
struct vcl *vcl;
struct vclref *ref;
AN(refp);
ref = *refp;
*refp = NULL;
CHECK_OBJ_NOTNULL(ref, VCLREF_MAGIC);
TAKE_OBJ_NOTNULL(ref, refp, VCLREF_MAGIC);
vcl = ref->vcl;
CHECK_OBJ_NOTNULL(vcl, VCL_MAGIC);
......@@ -398,11 +394,7 @@ VRT_VCL_Allow_Discard(struct vclref **refp)
struct vcl *vcl;
struct vclref *ref;
AN(refp);
ref = *refp;
*refp = NULL;
CHECK_OBJ_NOTNULL(ref, VCLREF_MAGIC);
TAKE_OBJ_NOTNULL(ref, refp, VCLREF_MAGIC);
vcl = ref->vcl;
CHECK_OBJ_NOTNULL(vcl, VCL_MAGIC);
......
......@@ -12,8 +12,11 @@ expression obj, objp, magic;
@@
- AN(objp);
...
- obj = *objp;
...
- *objp = NULL;
...
- CHECK_OBJ_NOTNULL(obj, magic);
+ TAKE_OBJ_NOTNULL(obj, objp, magic);
......@@ -22,8 +25,11 @@ expression obj, objp, magic;
@@
- AN(objp);
...
- obj = *objp;
...
- CHECK_OBJ_NOTNULL(obj, magic);
...
- *objp = NULL;
+ TAKE_OBJ_NOTNULL(obj, objp, magic);
......@@ -32,7 +38,9 @@ expression obj, objp, magic;
@@
- AN(objp);
...
- obj = *objp;
...
- CHECK_OBJ_NOTNULL(obj, magic);
+ TAKE_OBJ_NOTNULL(obj, objp, magic);
...
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment