This patch fixes an issue in disconnect_set_ready(). If
mbim_message_connect_response_parse(), `session_id' and `nw_error' are
not set to a valid value, and thus shouldn't be used.
---
src/mm-bearer-mbim.c | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/src/mm-bearer-mbim.c b/src/mm-bearer-mbim.c
index 63bb0579..b6f4bd72 100644
--- a/src/mm-bearer-mbim.c
+++ b/src/mm-bearer-mbim.c
@@ -1119,11 +1119,15 @@ disconnect_set_ready (MbimDevice *device,
} else if (g_error_matches (error,
MBIM_STATUS_ERROR,
MBIM_STATUS_ERROR_CONTEXT_NOT_ACTIVATED)) {
- mm_dbg ("Session ID '%u' already disconnected.", session_id);
+ if (parsed_result)
+ mm_dbg ("Session ID '%u' already disconnected.", session_id);
+ else
+ mm_dbg ("Session ID '<unknown>' already disconnected.");
+
g_clear_error (&error);
g_clear_error (&inner_error);
} else if (g_error_matches (error, MBIM_STATUS_ERROR,
MBIM_STATUS_ERROR_FAILURE)) {
- if (nw_error) {
+ if (parsed_result && nw_error != 0) {
g_error_free (error);
error = mm_mobile_equipment_error_from_mbim_nw_error
(nw_error);
}
--
2.14.0.rc1.383.gd1ce394fe2-goog
_______________________________________________
ModemManager-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/modemmanager-devel
