From e7035a9f2cbdd6b630b7816ae01dc337ae9ece66 Mon Sep 17 00:00:00 2001 From: Ben Pfaff Date: Tue, 25 Sep 2012 17:00:34 -0700 Subject: [PATCH] gui: Check return value of gtk_tree_model_get_iter(). In these two cases the return value was assigned to a variable but never used. This commit adds a check for the return value. --- src/ui/gui/autorecode-dialog.c | 1 + src/ui/gui/recode-dialog.c | 1 + 2 files changed, 2 insertions(+) diff --git a/src/ui/gui/autorecode-dialog.c b/src/ui/gui/autorecode-dialog.c index f303e18d0f..011d5c219a 100644 --- a/src/ui/gui/autorecode-dialog.c +++ b/src/ui/gui/autorecode-dialog.c @@ -216,6 +216,7 @@ on_selection_change (GtkTreeSelection *selection, gpointer data) ok = gtk_tree_model_get_iter (model, &iter, (GtkTreePath*) rows->data); + g_return_if_fail (ok); gtk_tree_model_get (model, &iter, 0, &var, -1); diff --git a/src/ui/gui/recode-dialog.c b/src/ui/gui/recode-dialog.c index 012b2f26c1..5bdee0bb9e 100644 --- a/src/ui/gui/recode-dialog.c +++ b/src/ui/gui/recode-dialog.c @@ -452,6 +452,7 @@ on_selection_change (GtkTreeSelection *selection, gpointer data) gtk_widget_set_sensitive (rd->new_label_entry, TRUE); ok = gtk_tree_model_get_iter (model, &iter, (GtkTreePath*) rows->data); + g_return_if_fail (ok); gtk_tree_model_get (model, &iter, 0, &var, -- 2.30.2