From ae4ace01f8bf25e32062fab75199845147a44b6a Mon Sep 17 00:00:00 2001 From: Ray Strode Date: Wed, 16 May 2018 14:04:51 -0400 Subject: display-access-file: fix early free The unused function gdm_display_access_file_remove_display frees some variables and then tries to use them. For completeness, this commit moves the free() calls to the appropriate place in the code. A subsequent commit will drop the function. https://bugzilla.gnome.org/show_bug.cgi?id=796176 --- daemon/gdm-display-access-file.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'daemon/gdm-display-access-file.c') diff --git a/daemon/gdm-display-access-file.c b/daemon/gdm-display-access-file.c index 9bf818d2..c6d01351 100644 --- a/daemon/gdm-display-access-file.c +++ b/daemon/gdm-display-access-file.c @@ -598,9 +598,6 @@ gdm_display_access_file_remove_display (GdmDisplayAccessFile *file, number, name_length, name); - g_free (address); - g_free (number); - g_free (name); if (auth_entry != NULL) { XauDisposeAuth (auth_entry); @@ -625,6 +622,9 @@ gdm_display_access_file_remove_display (GdmDisplayAccessFile *file, } } + g_free (address); + g_free (number); + g_free (name); if (result == FALSE) { g_set_error (error, -- cgit v1.2.1