From 29c655c0a6affc0359e499162e8308663eb4d04f Mon Sep 17 00:00:00 2001 From: Markus Elfring Date: Wed, 11 Mar 2015 17:41:01 +0100 Subject: Bug #149: Deletion of unnecessary checks before calls of the function "free" The function "free" is documented in the way that no action shall occur for a passed null pointer. It is therefore not needed that a function caller repeats a corresponding check. http://stackoverflow.com/questions/18775608/free-a-null-pointer-anyway-or-check-first This issue was fixed by using the software Coccinelle 1.0.0-rc24. Signed-off-by: Markus Elfring --- tests/unit/unit1305.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) (limited to 'tests/unit/unit1305.c') diff --git a/tests/unit/unit1305.c b/tests/unit/unit1305.c index 91e3b0b27..93815e5f8 100644 --- a/tests/unit/unit1305.c +++ b/tests/unit/unit1305.c @@ -65,9 +65,7 @@ static void unit_stop( void ) Curl_freeaddrinfo(data_node->addr); free(data_node); } - if (data_key) - free(data_key); - + free(data_key); Curl_hash_destroy(hp); curl_easy_cleanup(data); -- cgit v1.2.3