[gdm/benzea/fix-unknown-session-crash: 15/15] manager: Always write out parameter
- From: Ray Strode <halfline src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gdm/benzea/fix-unknown-session-crash: 15/15] manager: Always write out parameter
- Date: Wed, 9 Sep 2020 13:27:56 +0000 (UTC)
commit f37d2f01d09c52d1a0df2c34493b59d92f45ae1a
Author: Benjamin Berg <bberg redhat com>
Date: Fri Aug 21 12:15:47 2020 +0200
manager: Always write out parameter
The get_display_and_details_for_bus_sender function does not return a
proper error value. Due to this, it makes sense to always write the out
parameters (though, I expect we have still more that we might need to
write).
This is just slightly safer, but the function probably isn't great as
is.
daemon/gdm-manager.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
---
diff --git a/daemon/gdm-manager.c b/daemon/gdm-manager.c
index 22f023fd..459ccf6b 100644
--- a/daemon/gdm-manager.c
+++ b/daemon/gdm-manager.c
@@ -576,10 +576,11 @@ get_display_and_details_for_bus_sender (GdmManager *self,
lookup_by_session_id,
(gpointer) session_id);
+out:
if (out_display != NULL) {
*out_display = display;
}
-out:
+
g_free (session_id);
}
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]