diff options
| author | Aleksander Morgado <aleksander@aleksander.es> | 2021-05-23 12:16:55 +0200 |
|---|---|---|
| committer | Aleksander Morgado <aleksander@aleksander.es> | 2021-05-26 13:14:52 +0000 |
| commit | 01223b2d4dd61d3a3801543981791927b0746a66 (patch) | |
| tree | 2fcdeac7409b67b27b97ea711cef52af74c99e40 | |
| parent | 38740e9d11a52f91e41349a62caf35f8e1a67f48 (diff) | |
base-manager: make sure g_autoptr() variables are initialized to NULL
Otherwise, we may have memory issues if the variable isn't initialized
and the method exits.
| -rw-r--r-- | src/mm-base-manager.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/mm-base-manager.c b/src/mm-base-manager.c index e7df890a..f538469a 100644 --- a/src/mm-base-manager.c +++ b/src/mm-base-manager.c @@ -741,11 +741,11 @@ mm_base_modem_sync_ready (MMBaseModem *self, GAsyncResult *res, gpointer user_data) { - g_autoptr(GError) error; + g_autoptr(GError) error = NULL; mm_base_modem_sync_finish (self, res, &error); if (error) { - mm_obj_warn (self, "synchronization failed"); + mm_obj_warn (self, "synchronization failed: %s", error->message); return; } mm_obj_info (self, "synchronization finished"); |