Fix storing and freeing of hyperlink paths
authorMike Gorse <mgorse@novell.com>
Sat, 1 Jan 2011 11:41:10 +0000 (06:41 -0500)
committerMike Gorse <mgorse@novell.com>
Sat, 1 Jan 2011 11:41:10 +0000 (06:41 -0500)
atspi/atspi-misc.c
atspi/atspi-object.c

index 8dca97c..dd014a8 100644 (file)
@@ -254,7 +254,7 @@ ref_hyperlink (const char *app_name, const char *path)
   hyperlink = atspi_hyperlink_new (app, path);
   if (!hyperlink)
     return NULL;
-  g_hash_table_insert (app->hash, hyperlink->parent.path, hyperlink);
+  g_hash_table_insert (app->hash, g_strdup (hyperlink->parent.path), hyperlink);
   /* TODO: This should be a weak ref */
   g_object_ref (hyperlink);    /* for the hash */
   return hyperlink;
index 21a0ce1..3545845 100644 (file)
@@ -49,8 +49,7 @@ atspi_object_finalize (GObject *object)
 {
   AtspiObject *aobj = ATSPI_OBJECT (object);
 
-  /* TODO: Figure out why the next line sometimes crashes */
-  /*g_free (aobj->path); */
+  g_free (aobj->path);
 
   G_OBJECT_CLASS (atspi_object_parent_class)->finalize (object);
 }