From faf0947ebbcdbd2088d6349e936a23b24d09153b Mon Sep 17 00:00:00 2001
From: "Daniel P. Berrange" <berrange@redhat.com>
Date: Mon, 12 Dec 2011 17:53:04 +0000
Subject: Remove useless if() before free()

The free() function allows NULL to be passed in, so any
code which puts a if() before free() is wasting time
---
 common/canvas_utils.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

(limited to 'common/canvas_utils.c')

diff --git a/common/canvas_utils.c b/common/canvas_utils.c
index 55b71599..71f6890e 100644
--- a/common/canvas_utils.c
+++ b/common/canvas_utils.c
@@ -51,9 +51,7 @@ static void release_data(pixman_image_t *image, void *release_data)
         gdi_handlers--;
     }
 #endif
-    if (data->data) {
-        free(data->data);
-    }
+    free(data->data);
 
     free(data);
 }
-- 
cgit