From: Emmanuele Bassi <ebassi@linux.intel.com>
Date: Sat, 9 Jun 2012 09:38:09 +0000 (+0100)
Subject: base-types: Don't free zero point and rect
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=5eb07a301060225e2e33d7d8c5a61003dda7714d;p=profile%2Fivi%2Fclutter.git

base-types: Don't free zero point and rect

Calling clutter_point_free(clutter_point_zero()) or calling
clutter_rect_free(clutter_rect_zero()) should be safe, exactly like it's
safe to call those functions with a NULL argument.
---

diff --git a/clutter/clutter-base-types.c b/clutter/clutter-base-types.c
index ac5bc73..32ed407 100644
--- a/clutter/clutter-base-types.c
+++ b/clutter/clutter-base-types.c
@@ -466,7 +466,7 @@ clutter_point_copy (const ClutterPoint *point)
 void
 clutter_point_free (ClutterPoint *point)
 {
-  if (point != NULL)
+  if (point != NULL && point != &_clutter_point_zero)
     g_slice_free (ClutterPoint, point);
 }
 
@@ -840,7 +840,7 @@ clutter_rect_copy (const ClutterRect *rect)
 void
 clutter_rect_free (ClutterRect *rect)
 {
-  if (rect != NULL)
+  if (rect != NULL && rect != &_clutter_rect_zero)
     g_slice_free (ClutterRect, rect);
 }