Make this work for list == NULL. (Coverity)
2006-04-12 Matthias Clasen <mclasen@redhat.com> * gdk/x11/gdkselection-x11.c (make_list): Make this work for list == NULL. (Coverity)
This commit is contained in:
parent
6d66ce2fc3
commit
93c2bc828e
@ -1,5 +1,8 @@
|
||||
2006-04-12 Matthias Clasen <mclasen@redhat.com>
|
||||
|
||||
* gdk/x11/gdkselection-x11.c (make_list): Make this work
|
||||
for list == NULL. (Coverity)
|
||||
|
||||
* gdk/x11/gdkkeys-x11.c (gdk_keymap_get_entries_for_keycode):
|
||||
Rearrange the code a bit.
|
||||
|
||||
|
@ -1,5 +1,8 @@
|
||||
2006-04-12 Matthias Clasen <mclasen@redhat.com>
|
||||
|
||||
* gdk/x11/gdkselection-x11.c (make_list): Make this work
|
||||
for list == NULL. (Coverity)
|
||||
|
||||
* gdk/x11/gdkkeys-x11.c (gdk_keymap_get_entries_for_keycode):
|
||||
Rearrange the code a bit.
|
||||
|
||||
|
@ -536,10 +536,11 @@ make_list (const gchar *text,
|
||||
}
|
||||
|
||||
if (list)
|
||||
*list = g_new (gchar *, n_strings + 1);
|
||||
|
||||
(*list)[n_strings] = NULL;
|
||||
|
||||
{
|
||||
*list = g_new (gchar *, n_strings + 1);
|
||||
(*list)[n_strings] = NULL;
|
||||
}
|
||||
|
||||
i = n_strings;
|
||||
tmp_list = strings;
|
||||
while (tmp_list)
|
||||
@ -548,10 +549,10 @@ make_list (const gchar *text,
|
||||
(*list)[--i] = tmp_list->data;
|
||||
else
|
||||
g_free (tmp_list->data);
|
||||
|
||||
|
||||
tmp_list = tmp_list->next;
|
||||
}
|
||||
|
||||
|
||||
g_slist_free (strings);
|
||||
|
||||
return n_strings;
|
||||
|
Loading…
Reference in New Issue
Block a user