diff options
| author | Markus Elfring <elfring@users.sourceforge.net> | 2015-03-11 17:41:01 +0100 |
|---|---|---|
| committer | Daniel Stenberg <daniel@haxx.se> | 2015-03-16 12:13:56 +0100 |
| commit | 29c655c0a6affc0359e499162e8308663eb4d04f (patch) | |
| tree | ed6b1fc761dee6623ec1b312cc53dc3b661c1550 /tests/libtest/lib582.c | |
| parent | 059b3a5770075315dbc843b9285a1cdec82c12d5 (diff) | |
| download | curl-29c655c0a6affc0359e499162e8308663eb4d04f.tar.gz | |
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 <elfring@users.sourceforge.net>
Diffstat (limited to 'tests/libtest/lib582.c')
| -rw-r--r-- | tests/libtest/lib582.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/tests/libtest/lib582.c b/tests/libtest/lib582.c index 952efb4ed..4c3ea6f60 100644 --- a/tests/libtest/lib582.c +++ b/tests/libtest/lib582.c @@ -357,10 +357,8 @@ test_cleanup: fclose(hd_src); /* free local memory */ - if(sockets.read.sockets) - free(sockets.read.sockets); - if(sockets.write.sockets) - free(sockets.write.sockets); + free(sockets.read.sockets); + free(sockets.write.sockets); return res; } |