diff options
author | Marco Trevisan (TreviƱo) <mail@3v1n0.net> | 2019-07-21 16:49:11 +0200 |
---|---|---|
committer | Ray Strode <rstrode@redhat.com> | 2019-08-13 15:50:03 -0400 |
commit | afb92b9554bef398471f2ec508e53a1f1e965011 (patch) | |
tree | 409412f70ef0f482abf7fa97209886e0dc014cd7 | |
parent | 1e1ad8c170e125bb8d709c1b640abcb7e76334dc (diff) | |
download | gdm-afb92b9554bef398471f2ec508e53a1f1e965011.tar.gz |
sessions: Use auto pointer GPtrArray to allocate session dirs
Gdm leaks session dir names, so use a ptr array with a free function and
auto-pointers to manage the strings lifecycle.
-rw-r--r-- | libgdm/gdm-sessions.c | 30 |
1 files changed, 14 insertions, 16 deletions
diff --git a/libgdm/gdm-sessions.c b/libgdm/gdm-sessions.c index bcfe7658..8ad54576 100644 --- a/libgdm/gdm-sessions.c +++ b/libgdm/gdm-sessions.c @@ -239,8 +239,8 @@ static void collect_sessions (void) { g_autoptr(GHashTable) names_seen_before = NULL; - GArray *xorg_search_array = NULL; - GArray *wayland_search_array = NULL; + g_autoptr(GPtrArray) xorg_search_array = NULL; + g_autoptr(GPtrArray) wayland_search_array = NULL; gchar *session_dir = NULL; int i; const char *xorg_search_dirs[] = { @@ -251,31 +251,34 @@ collect_sessions (void) }; names_seen_before = g_hash_table_new (g_str_hash, g_str_equal); - - xorg_search_array = g_array_new (TRUE, TRUE, sizeof (char *)); + xorg_search_array = g_ptr_array_new_with_free_func (g_free); const gchar * const *system_data_dirs = g_get_system_data_dirs (); for (i = 0; system_data_dirs[i]; i++) { session_dir = g_build_filename (system_data_dirs[i], "xsessions", NULL); - g_array_append_val (xorg_search_array, session_dir); + g_ptr_array_add (xorg_search_array, session_dir); } - g_array_append_vals (xorg_search_array, xorg_search_dirs, G_N_ELEMENTS (xorg_search_dirs)); + for (i = 0; i < G_N_ELEMENTS (xorg_search_dirs); i++) { + g_ptr_array_add (xorg_search_array, g_strdup (xorg_search_dirs[i])); + } #ifdef ENABLE_WAYLAND_SUPPORT const char *wayland_search_dirs[] = { DATADIR "/wayland-sessions/", }; - wayland_search_array = g_array_new (TRUE, TRUE, sizeof (char *)); + wayland_search_array = g_ptr_array_new_with_free_func (g_free); for (i = 0; system_data_dirs[i]; i++) { session_dir = g_build_filename (system_data_dirs[i], "wayland-sessions", NULL); - g_array_append_val (wayland_search_array, session_dir); + g_ptr_array_add (wayland_search_array, session_dir); } - g_array_append_vals (wayland_search_array, wayland_search_dirs, G_N_ELEMENTS (wayland_search_dirs)); + for (i = 0; i < G_N_ELEMENTS (wayland_search_dirs); i++) { + g_ptr_array_add (wayland_search_array, g_strdup (wayland_search_dirs[i])); + } #endif if (gdm_available_sessions_map == NULL) { @@ -284,24 +287,19 @@ collect_sessions (void) } for (i = 0; i < xorg_search_array->len; i++) { - collect_sessions_from_directory (g_array_index (xorg_search_array, gchar*, i)); + collect_sessions_from_directory (g_ptr_array_index (xorg_search_array, i)); } - g_array_free (xorg_search_array, TRUE); - #ifdef ENABLE_WAYLAND_SUPPORT #ifdef ENABLE_USER_DISPLAY_SERVER if (g_getenv ("WAYLAND_DISPLAY") == NULL && g_getenv ("RUNNING_UNDER_GDM") != NULL) { - g_array_free (wayland_search_array, TRUE); return; } #endif for (i = 0; i < wayland_search_array->len; i++) { - collect_sessions_from_directory (g_array_index (wayland_search_array, gchar*, i)); + collect_sessions_from_directory (g_ptr_array_index (wayland_search_array, i)); } - - g_array_free (wayland_search_array, TRUE); #endif g_hash_table_foreach_remove (gdm_available_sessions_map, |