From acc879811fe054397727b75f6885a9049e817669 Mon Sep 17 00:00:00 2001 From: Jeffrey Stedfast Date: Mon, 21 Apr 2003 17:42:40 +0000 Subject: Protect against a NULL strv result from parsing the language list string. 2003-04-18 Jeffrey Stedfast * mail-composer-prefs.c (spell_set_ui): Protect against a NULL strv result from parsing the language list string. svn path=/trunk/; revision=20903 --- mail/ChangeLog | 5 +++++ mail/mail-composer-prefs.c | 12 +++++++----- 2 files changed, 12 insertions(+), 5 deletions(-) (limited to 'mail') diff --git a/mail/ChangeLog b/mail/ChangeLog index 4cd98f177e..89b8a31ff7 100644 --- a/mail/ChangeLog +++ b/mail/ChangeLog @@ -1,3 +1,8 @@ +2003-04-18 Jeffrey Stedfast + + * mail-composer-prefs.c (spell_set_ui): Protect against a NULL + strv result from parsing the language list string. + 2003-04-17 Jeffrey Stedfast Fixes for bug #41142. diff --git a/mail/mail-composer-prefs.c b/mail/mail-composer-prefs.c index 0a01135964..15fbd6fdaf 100644 --- a/mail/mail-composer-prefs.c +++ b/mail/mail-composer-prefs.c @@ -492,16 +492,17 @@ spell_set_ui (MailComposerPrefs *prefs) GtkTreeIter iter; GHashTable *present; gboolean go; - char **strv; + char **strv = NULL; int i; prefs->spell_active = FALSE; /* setup the language list */ - strv = g_strsplit (prefs->language_str, " ", 0); present = g_hash_table_new (g_str_hash, g_str_equal); - for (i = 0; strv[i]; i++) - g_hash_table_insert (present, strv[i], strv[i]); + if (prefs->language_str && (strv = g_strsplit (prefs->language_str, " ", 0))) { + for (i = 0; strv[i]; i++) + g_hash_table_insert (present, strv[i], strv[i]); + } model = (GtkListStore *) gtk_tree_view_get_model (prefs->language); for (go = gtk_tree_model_get_iter_first ((GtkTreeModel *) model, &iter); go; @@ -513,7 +514,8 @@ spell_set_ui (MailComposerPrefs *prefs) } g_hash_table_destroy (present); - g_strfreev (strv); + if (strv != NULL) + g_strfreev (strv); gnome_color_picker_set_i16 (GNOME_COLOR_PICKER (prefs->colour), prefs->spell_error_color.red, -- cgit v1.2.3