[PATCH] bearer-mbim: avoid accessing invalid session_id and nw_error

[Ben Chan]

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