2008-10-17 Emmanuele Bassi <ebassi@linux.intel.com>

* clutter/clutter-color.c (clutter_color_parse): Add checks
	for the arguments, to avoid trying to parse NULL strings.
This commit is contained in:
Emmanuele Bassi 2008-10-17 08:04:03 +00:00
parent 6d4667cb61
commit 9cf5386bb1
2 changed files with 11 additions and 3 deletions

View File

@ -1,3 +1,8 @@
2008-10-17 Emmanuele Bassi <ebassi@linux.intel.com>
* clutter/clutter-color.c (clutter_color_parse): Add checks
for the arguments, to avoid trying to parse NULL strings.
2008-10-16 Emmanuele Bassi <ebassi@linux.intel.com> 2008-10-16 Emmanuele Bassi <ebassi@linux.intel.com>
* clutter/clutter-container.[ch]: Add checks to the Container * clutter/clutter-container.[ch]: Add checks to the Container

View File

@ -482,6 +482,9 @@ clutter_color_parse (const gchar *color,
{ {
PangoColor pango_color; PangoColor pango_color;
g_return_val_if_fail (color != NULL, FALSE);
g_return_val_if_fail (dest != NULL, FALSE);
/* parse ourselves to get alpha */ /* parse ourselves to get alpha */
if (color[0] == '#') if (color[0] == '#')
{ {
@ -491,7 +494,7 @@ clutter_color_parse (const gchar *color,
{ {
if (strlen (color) == 9) if (strlen (color) == 9)
{ {
dest->red = result >> 24 & 0xff; dest->red = (result >> 24) & 0xff;
dest->green = (result >> 16) & 0xff; dest->green = (result >> 16) & 0xff;
dest->blue = (result >> 8) & 0xff; dest->blue = (result >> 8) & 0xff;
dest->alpha = result & 0xff; dest->alpha = result & 0xff;