From 6bd2eb06cbbba09c24e7ab5d5323343c5c2a881f Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ond=C5=99ej=20Sur=C3=BD?= Date: Thu, 5 Sep 2019 18:35:03 +0200 Subject: [PATCH] Add a semantic patch to make refactor the isc_mem_create() --- cocci/isc_mem_create_never_fail.spatch | 95 ++++++++++++++++++++++++++ 1 file changed, 95 insertions(+) create mode 100644 cocci/isc_mem_create_never_fail.spatch diff --git a/cocci/isc_mem_create_never_fail.spatch b/cocci/isc_mem_create_never_fail.spatch new file mode 100644 index 0000000000..e526506ec1 --- /dev/null +++ b/cocci/isc_mem_create_never_fail.spatch @@ -0,0 +1,95 @@ +@@ +statement S; +expression E; +@@ + +- if (isc_mem_create(0, 0, E) != ISC_R_SUCCESS) S; ++ isc_mem_create(E); + +@@ +statement S; +expression V, E; +@@ + +- V = isc_mem_create(0, 0, E); +- if (V == ISC_R_SUCCESS) S ++ isc_mem_create(E); ++ S + + +@@ +statement S; +expression V, E; +@@ + +- V = isc_mem_create(0, 0, E); +- if (V != ISC_R_SUCCESS) S ++ isc_mem_create(E); + +@@ +expression V, E; +@@ + +- V = isc_mem_create(0, 0, E); +- check_result(V, ...); ++ isc_mem_create(E); + +@@ +statement S; +expression V, E, C; +@@ + +- if (C) { V = isc_mem_create(0, 0, E); } +- if (V != ISC_R_SUCCESS) S ++ if (C) { isc_mem_create(E); } + +@@ +expression E; +@@ + +- RUNTIME_CHECK(isc_mem_create(0, 0, E) == ISC_R_SUCCESS); ++ isc_mem_create(E); + +@@ +expression E; +@@ + +- RUNCHECK(isc_mem_create(0, 0, E)); ++ isc_mem_create(E); + +@@ +expression E; +@@ + +- CHECK(isc_mem_create(0, 0, E), ...); ++ isc_mem_create(E); + +@@ +expression E; +@@ + +- CHECK(isc_mem_create(0, 0, E)); ++ isc_mem_create(E); + +@@ +expression V, E; +@@ + +- V = isc_mem_create(0, 0, E); +- assert_int_equal(V, ISC_R_SUCCESS); ++ isc_mem_create(E); + +@@ +expression E; +statement S; +@@ + +- if (isc_mem_create(0, 0, E) != ISC_R_SUCCESS) S ++ isc_mem_create(E); + +@@ +expression E; +@@ + +- DO(..., isc_mem_create(0, 0, E)); ++ isc_mem_create(E);