From c86d055db77cdeef6049449ac79fe404c40d28cc Mon Sep 17 00:00:00 2001 From: Tomaz Canabrava Date: Mon, 12 May 2014 14:58:15 -0300 Subject: Do not check for null before free. C specs says that we can safelly free a NULL pointer, so there's no reason to check if it's null before freeing it. Signed-off-by: Tomaz Canabrava Signed-off-by: Dirk Hohndel --- statistics.c | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) (limited to 'statistics.c') diff --git a/statistics.c b/statistics.c index ad5a95d06..2a96efb81 100644 --- a/statistics.c +++ b/statistics.c @@ -114,11 +114,10 @@ void process_all_dives(struct dive *dive, struct dive **prev_dive) * case (one dive per year or all dives during * one month) for yearly and monthly statistics*/ - if (stats_yearly != NULL) { - free(stats_yearly); - free(stats_monthly); - free(stats_by_trip); - } + free(stats_yearly); + free(stats_monthly); + free(stats_by_trip); + size = sizeof(stats_t) * (dive_table.nr + 1); stats_yearly = malloc(size); stats_monthly = malloc(size); -- cgit v1.2.3-70-g09d2