From b362314efb9858e5bcad156a9bbac97f61f0e71e Mon Sep 17 00:00:00 2001 From: Alberto Bertogli Date: Fri, 26 Jun 2009 03:01:29 -0300 Subject: [PATCH 6/9] Unconditionally free() some pointers free(NULL) is perfectly valid, so there is no need to check for it. Signed-off-by: Alberto Bertogli --- libjio/journal.c | 6 ++---- 1 files changed, 2 insertions(+), 4 deletions(-) diff --git a/libjio/journal.c b/libjio/journal.c index be9e04b..b059eee 100644 --- a/libjio/journal.c +++ b/libjio/journal.c @@ -196,10 +196,8 @@ unlink_error: close(fd); error: - if (name) - free(name); - if (jop) - free(jop); + free(name); + free(jop); return NULL; } -- 1.6.2.2.646.gb214