From 74308a05b52192d26336488238c4706d1e45f4cd Mon Sep 17 00:00:00 2001 From: Alessandro Bono Date: Wed, 2 Nov 2022 17:42:51 +0100 Subject: gdm-display: Plug a memory leak error was not freed in the second if branch. While at it use g_autoptr. --- daemon/gdm-display.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) (limited to 'daemon/gdm-display.c') diff --git a/daemon/gdm-display.c b/daemon/gdm-display.c index dd617227..e8b45cce 100644 --- a/daemon/gdm-display.c +++ b/daemon/gdm-display.c @@ -1616,7 +1616,7 @@ kernel_cmdline_initial_setup_argument (const gchar *contents, static gboolean kernel_cmdline_initial_setup_force_state (gboolean *force_state) { - GError *error = NULL; + g_autoptr(GError) error = NULL; gchar *contents = NULL; gchar *setup_argument = NULL; @@ -1625,7 +1625,6 @@ kernel_cmdline_initial_setup_force_state (gboolean *force_state) if (!g_file_get_contents ("/proc/cmdline", &contents, NULL, &error)) { g_debug ("GdmDisplay: Could not check kernel parameters, not forcing initial setup: %s", error->message); - g_clear_error (&error); return FALSE; } -- cgit v1.2.1