diff options
| author | Benjamin Berg <bberg@redhat.com> | 2020-08-21 12:15:47 +0200 |
|---|---|---|
| committer | Ray Strode <halfline@gmail.com> | 2020-09-09 13:27:55 +0000 |
| commit | f37d2f01d09c52d1a0df2c34493b59d92f45ae1a (patch) | |
| tree | 3e02eae06460f0cb8010790587c70fb15c58810a /daemon/gdm-manager.c | |
| parent | c836977bc78be5b8c75228c131d26dee8bd09a97 (diff) | |
| download | gdm-f37d2f01d09c52d1a0df2c34493b59d92f45ae1a.tar.gz | |
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.
Diffstat (limited to 'daemon/gdm-manager.c')
| -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 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); } |