]> Pileus Git - ~andy/gtk/commitdiff
a11y: fix a typo leading to a crash
authorCosimo Cecchi <cosimoc@gnome.org>
Wed, 6 Jun 2012 13:22:29 +0000 (09:22 -0400)
committerCosimo Cecchi <cosimoc@gnome.org>
Thu, 7 Jun 2012 20:51:02 +0000 (16:51 -0400)
text can't be NULL here, or we would have failed way earlier.
The original code from gail returned if the renderer text was NULL, and
we have to do that, or we'll end up calling e.g.
g_utf8_offset_to_pointer() on a NULL pointer.

https://bugzilla.redhat.com/show_bug.cgi?id=827930

https://bugzilla.gnome.org/show_bug.cgi?id=677551

gtk/a11y/gtktextcellaccessible.c

index dbc9978d68e7cd0df610dc1407c324dafdd6162a..cf24b493ac82c49ae5fe1436ffd6e7a4318f8b3d 100644 (file)
@@ -542,11 +542,8 @@ gtk_text_cell_accessible_get_character_extents (AtkText      *text,
   gtk_renderer = GTK_CELL_RENDERER_TEXT (gail_renderer->renderer);
 
   g_object_get (gtk_renderer, "text", &renderer_text, NULL);
-  if (text == NULL)
-    {
-      g_free (renderer_text);
-      return;
-    }
+  if (renderer_text == NULL)
+    return;
 
   parent = atk_object_get_parent (ATK_OBJECT (text));
   if (GTK_IS_CONTAINER_CELL_ACCESSIBLE (parent))