diff options
| author | Benjamin Berg <bberg@redhat.com> | 2020-08-21 12:15:47 +0200 |
|---|---|---|
| committer | Ignacio Casal Quinteiro <qignacio@amazon.com> | 2020-10-30 11:13:21 +0100 |
| commit | 26870c5fa59303cb6ecb2492ccab3a61d976c5be (patch) | |
| tree | dbd3328445599e6357d8a08b78363e427427fda3 | |
| parent | 0c6e50b77e10093ab6cfe8de0d7295c3501fbf64 (diff) | |
| download | gdm-wip/nacho/crash-backport-3-36.tar.gz | |
manager: Always write out parameterwip/nacho/crash-backport-3-36
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.
| -rw-r--r-- | daemon/gdm-manager.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/daemon/gdm-manager.c b/daemon/gdm-manager.c index 25beecb5..6f6c7c90 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); } |