SymbX/ModemManager
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

modem-helpers: handle no space after "+CESQ:"

Browse Source 
See https://gitlab.freedesktop.org/mobile-broadband/ModemManager/-/merge_requests/1277
for more details.

Signed-off-by: Dan Williams <dan@ioncontrol.co>
This commit is contained in:
Dan Williams
2025-09-11 08:24:42 -05:00
parent 672a1e29c8
commit ee75d23f00
2 changed files with 8 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/mm-modem-helpers.c
+1 -1
View File
@@ -2755,7 +2755,7 @@ mm_3gpp_parse_cesq_response (const gchar *response,
/* Response may be e.g.:
* +CESQ: 99,99,255,255,20,80
*/
r = g_regex_new ("\\+CESQ: (\\d+),(\\d+),(\\d+),(\\d+),(\\d+),(\\d+)(?:\\r\\n)?", 0, 0, NULL);
r = g_regex_new ("\\+CESQ:\\s*(\\d+),(\\d+),(\\d+),(\\d+),(\\d+),(\\d+)(?:\\r\\n)?", 0, 0, NULL);
g_assert (r != NULL);
g_regex_match_full (r, response, strlen (response), 0, 0, &match_info, &inner_error);
src/tests/test-modem-helpers.c
+7
View File
@@ -4171,6 +4171,13 @@ static const CesqResponseTest cesq_response_tests[] = {
.gsm_info = TRUE, .rxlev = 10, .rssi = -101.0, .ber = 6,
.umts_info = FALSE, .rscp_level = 255, .ecn0_level = 255,
.lte_info = FALSE, .rsrq_level = 255, .rsrp_level = 255,
},
{
/* No space after ':' */
.str = "+CESQ:10,6,255,255,255,255",
.gsm_info = TRUE, .rxlev = 10, .rssi = -101.0, .ber = 6,
.umts_info = FALSE, .rscp_level = 255, .ecn0_level = 255,
.lte_info = FALSE, .rsrq_level = 255, .rsrp_level = 255,
}
};