summaryrefslogtreecommitdiffstats
path: root/dive.c
diff options
context:
space:
mode:
authorGravatar Tomaz Canabrava <tomaz.canabrava@intel.com>2014-05-12 14:58:15 -0300
committerGravatar Dirk Hohndel <dirk@hohndel.org>2014-05-13 09:26:50 +0900
commitc86d055db77cdeef6049449ac79fe404c40d28cc (patch)
treef0e53f8d42567a670567e7600de287dcdde5b627 /dive.c
parente0c0ac5d5c7c218011278fbe57da291a7e10c8eb (diff)
downloadsubsurface-c86d055db77cdeef6049449ac79fe404c40d28cc.tar.gz
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 <tomaz.canabrava@intel.com> Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
Diffstat (limited to 'dive.c')
-rw-r--r--dive.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/dive.c b/dive.c
index 0c4b91fb3..68f619bdd 100644
--- a/dive.c
+++ b/dive.c
@@ -1758,8 +1758,7 @@ static void free_events(struct event *ev)
static void free_dc(struct divecomputer *dc)
{
free(dc->sample);
- if (dc->model)
- free((void *)dc->model);
+ free((void *)dc->model);
free_events(dc->events);
free(dc);
}