diff options
| -rw-r--r-- | plugins/huawei/mm-modem-helpers-huawei.c | 6 | ||||
| -rw-r--r-- | plugins/huawei/tests/test-modem-helpers-huawei.c | 14 |
2 files changed, 20 insertions, 0 deletions
diff --git a/plugins/huawei/mm-modem-helpers-huawei.c b/plugins/huawei/mm-modem-helpers-huawei.c index d3f13c23..fde1c791 100644 --- a/plugins/huawei/mm-modem-helpers-huawei.c +++ b/plugins/huawei/mm-modem-helpers-huawei.c @@ -904,6 +904,12 @@ mm_huawei_parse_syscfg_response (const gchar *response, return NULL; } + /* Fix invalid modes with non-sensical acquisition orders */ + if (mode == 14 && acqorder != 0) /* WCDMA only but acqorder != "Automatic" */ + acqorder = 0; + else if (mode == 13 && acqorder != 0) /* GSM only but acqorder != "Automatic" */ + acqorder = 0; + /* Look for current modes among the supported ones */ for (i = 0; i < supported_mode_combinations->len; i++) { const MMHuaweiSyscfgCombination *combination; diff --git a/plugins/huawei/tests/test-modem-helpers-huawei.c b/plugins/huawei/tests/test-modem-helpers-huawei.c index 13601c47..7044582d 100644 --- a/plugins/huawei/tests/test-modem-helpers-huawei.c +++ b/plugins/huawei/tests/test-modem-helpers-huawei.c @@ -707,6 +707,20 @@ static const SyscfgResponseTest syscfg_response_tests[] = { .format = "^SYSCFG:(2,13,14,16),(0-3),((400000,\"WCDMA2100\")),(0-2),(0-4)\r\n", .allowed = MM_MODEM_MODE_3G, .preferred = MM_MODEM_MODE_NONE + }, + { + /* Non-sensical acquisition order (WCDMA-only but acquire WCDMA-then-GSM */ + .str = "^SYSCFG: 14,2,400000,0,3\r\n", + .format = "^SYSCFG:(2,13,14,16),(0-3),((400000,\"WCDMA2100\")),(0-2),(0-4)\r\n", + .allowed = MM_MODEM_MODE_3G, + .preferred = MM_MODEM_MODE_NONE + }, + { + /* Non-sensical acquisition order (GSM-only but acquire GSM-then-WCDMA */ + .str = "^SYSCFG: 13,1,400000,0,3\r\n", + .format = "^SYSCFG:(2,13,14,16),(0-3),((400000,\"WCDMA2100\")),(0-2),(0-4)\r\n", + .allowed = MM_MODEM_MODE_2G, + .preferred = MM_MODEM_MODE_NONE } }; |