[PATCH] broadband-bearer: fix crm_range_ready

Ben Chan benchan at chromium.org
Tue Jul 18 22:28:51 UTC 2017


crm_range_ready() misses a return statement after it invokes
interface_initialization_step(), which will lead to an invalid access of
the already freed InitAsyncContext.
---
 src/mm-broadband-bearer.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/mm-broadband-bearer.c b/src/mm-broadband-bearer.c
index 696f9bfd..a2bcb3a4 100644
--- a/src/mm-broadband-bearer.c
+++ b/src/mm-broadband-bearer.c
@@ -2210,6 +2210,7 @@ crm_range_ready (MMBaseModem *modem,
                 /* Fine, go on with next step */
                 ctx->step++;
                 interface_initialization_step (ctx);
+                return;
             }
 
             g_assert (error == NULL);
-- 
2.13.2.932.g7449e964c-goog



More information about the ModemManager-devel mailing list