Add safety checks to hildon_color_chooser_set_color()

* src/hildon-color-chooser.c (hildon_color_chooser_set_color): Add
safety checks.

diff --git a/ChangeLog b/ChangeLog
index 52b0721..e1057f1 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2009-04-28  Claudio Saavedra  <csaavedra@igalia.com>
+
+       * src/hildon-color-chooser.c (hildon_color_chooser_set_color): Add
+       safety checks.
+
 2009-04-28  Alejandro Pinheiro  <apinheiro@igalia.com>
 
        * src/hildon-time-selector.h:

diff --git a/src/hildon-color-chooser.c b/src/hildon-color-chooser.c
index 14d39d9..81f8704 100644 (file)
--- a/src/hildon-color-chooser.c
+++ b/src/hildon-color-chooser.c
@@ -805,8 +805,12 @@ hildon_color_chooser_set_color                  (HildonColorChooser *chooser,
     unsigned short hue, sat, val;
     unsigned long min, max;
     signed long tmp, diff;
-    HildonColorChooserPrivate *priv = HILDON_COLOR_CHOOSER_GET_PRIVATE (chooser);
+    HildonColorChooserPrivate *priv;
 
+    g_return_if_fail (HILDON_IS_COLOR_CHOOSER (chooser));
+    g_return_if_fail (color != NULL);
+
+    priv = HILDON_COLOR_CHOOSER_GET_PRIVATE (chooser);
     g_assert (priv);
 
     /* ugly nesting */