]> git.ipfire.org Git - thirdparty/bind9.git/commitdiff
Add semantic patch to remove checking for isc_mem_get() return value
authorOndřej Surý <ondrej@sury.org>
Tue, 16 Jul 2019 13:50:45 +0000 (15:50 +0200)
committerOndřej Surý <ondrej@sury.org>
Tue, 23 Jul 2019 19:32:35 +0000 (15:32 -0400)
The isc_mem_get() cannot fail gracefully now, it either gets memory of
assert()s.  The added semantic patch cleans all the blocks checking whether
the return value of isc_mem_get() was NULL.

cocci/isc_mem_get_never_fail.spatch [new file with mode: 0644]

diff --git a/cocci/isc_mem_get_never_fail.spatch b/cocci/isc_mem_get_never_fail.spatch
new file mode 100644 (file)
index 0000000..e79a040
--- /dev/null
@@ -0,0 +1,41 @@
+@@
+statement S;
+expression V;
+@@
+
+V = isc_mem_get(...);
+- if (V == NULL) S
+
+@@
+type T;
+statement S;
+expression V;
+@@
+
+V = (T *)isc_mem_get(...);
+- if (V == NULL) S
+
+@@
+statement S;
+expression V;
+@@
+
+if (V == NULL) V = isc_mem_get(...);
+- if (V == NULL) S
+
+@@
+statement S1, S2;
+expression V;
+@@
+
+V = isc_mem_get(...);
+- if (V == NULL) S1 else { S2 }
++ S2
+
+@@
+type T;
+expression V, E1, E2;
+@@
+
+- V = (T)isc_mem_get(E1, E2);
++ V = isc_mem_get(E1, E2);