From a740e32652506eb82fd2a20623dd996ed3bd1ae7 Mon Sep 17 00:00:00 2001 From: Lucas Rocha Date: Thu, 10 Apr 2008 09:57:14 +0000 Subject: use g_value_dup_string when applicable (Fixes bug #527258). Patch from 2008-04-10 Lucas Rocha * src/tree.c (zenity_tree_dialog_get_selected, zenity_tree_dialog_toggle_get_selected): use g_value_dup_string when applicable (Fixes bug #527258). Patch from kraai@ftbfs.org. svn path=/trunk/; revision=1352 --- ChangeLog | 6 ++++++ src/tree.c | 8 ++++---- 2 files changed, 10 insertions(+), 4 deletions(-) diff --git a/ChangeLog b/ChangeLog index cae5fdc2..a3ee4ee5 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,9 @@ +2008-04-10 Lucas Rocha + + * src/tree.c (zenity_tree_dialog_get_selected, + zenity_tree_dialog_toggle_get_selected): use g_value_dup_string when + applicable (Fixes bug #527258). Patch from kraai@ftbfs.org. + 2008-03-30 Lucas Rocha * src/msg.c (zenity_msg_construct_question_dialog): fix memory leak on diff --git a/src/tree.c b/src/tree.c index 6dd3f645..65e9c471 100644 --- a/src/tree.c +++ b/src/tree.c @@ -516,7 +516,7 @@ zenity_tree_dialog_get_selected (GtkTreeModel *model, GtkTreePath *path_buf, Gtk for (i = 0; i < n_columns; i++) { gtk_tree_model_get_value (model, iter, i, &value); - selected = g_slist_append (selected, g_strdup (g_value_get_string (&value))); + selected = g_slist_append (selected, g_value_dup_string (&value)); g_value_unset (&value); } return; @@ -525,7 +525,7 @@ zenity_tree_dialog_get_selected (GtkTreeModel *model, GtkTreePath *path_buf, Gtk for (i = 0; print_columns[i] != 0; i++) { gtk_tree_model_get_value (model, iter, print_columns[i] - 1, &value); - selected = g_slist_append (selected, g_strdup (g_value_get_string (&value))); + selected = g_slist_append (selected, g_value_dup_string (&value)); g_value_unset (&value); } } @@ -547,7 +547,7 @@ zenity_tree_dialog_toggle_get_selected (GtkTreeModel *model, GtkTreePath *path, for (i = 1; i < n_columns; i++) { gtk_tree_model_get_value (model, iter, i, &value); - selected = g_slist_append (selected, g_strdup (g_value_get_string (&value))); + selected = g_slist_append (selected, g_value_dup_string (&value)); g_value_unset (&value); } g_value_unset (&toggle_value); @@ -557,7 +557,7 @@ zenity_tree_dialog_toggle_get_selected (GtkTreeModel *model, GtkTreePath *path, for (i = 0; print_columns[i] != 0; i++) { gtk_tree_model_get_value (model, iter, print_columns[i] - 1, &value); - selected = g_slist_append (selected, g_strdup (g_value_get_string (&value))); + selected = g_slist_append (selected, g_value_dup_string (&value)); g_value_unset (&value); } } -- cgit