diff options
| author | Aleksander Morgado <aleksander@aleksander.es> | 2020-01-03 22:25:44 +0100 |
|---|---|---|
| committer | Aleksander Morgado <aleksander@aleksander.es> | 2020-01-03 22:25:44 +0100 |
| commit | 0d8a5e2a43e6baff0c282a3d2b15d03063c48199 (patch) | |
| tree | e452e033b41287c2bc99edbd549468960ea9066f /src | |
| parent | b774cbe658c6e7e0eba169d72c2cbee9764eddc1 (diff) | |
iface-modem-voice: fix segfault when voice support check fails
The error returned in support_check_finish() should be treated as
optional, as in all the other optional feature interfaces.
Fixes https://gitlab.freedesktop.org/mobile-broadband/ModemManager/issues/172
Diffstat (limited to 'src')
| -rw-r--r-- | src/mm-iface-modem-voice.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/mm-iface-modem-voice.c b/src/mm-iface-modem-voice.c index 9c3b84c5..592afc41 100644 --- a/src/mm-iface-modem-voice.c +++ b/src/mm-iface-modem-voice.c @@ -2813,8 +2813,10 @@ check_support_ready (MMIfaceModemVoice *self, GError *error = NULL; if (!MM_IFACE_MODEM_VOICE_GET_INTERFACE (self)->check_support_finish (self, res, &error)) { - mm_dbg ("Voice support check failed: '%s'", error->message); - g_error_free (error); + if (error) { + mm_dbg ("Voice support check failed: '%s'", error->message); + g_error_free (error); + } g_task_return_new_error (task, MM_CORE_ERROR, MM_CORE_ERROR_UNSUPPORTED, "Voice not supported"); g_object_unref (task); return; |