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

x22x: ensure error is set when +SYSSEL response parser doesn't match

Browse Source
This commit is contained in:
Aleksander Morgado
2013-04-24 23:12:12 +02:00
parent 77d096bdac
commit a32eef3434
1 changed files with 11 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
plugins/x22x/mm-broadband-modem-x22x.c
View File

@@ -51,6 +51,7 @@ load_allowed_modes_finish (MMIfaceModem *self,
const gchar *response; const gchar *response;
gchar *str; gchar *str;
gint mode = -1; gint mode = -1;
GError *match_error = NULL;
response = mm_base_modem_at_command_finish (MM_BASE_MODEM (self), res, error); response = mm_base_modem_at_command_finish (MM_BASE_MODEM (self), res, error);
if (!response) if (!response)
@@ -59,9 +60,16 @@ load_allowed_modes_finish (MMIfaceModem *self,
r = g_regex_new ("\\+SYSSEL:\\s*(\\d+),(\\d+),(\\d+),(\\d+)", G_REGEX_UNGREEDY, 0, NULL); r = g_regex_new ("\\+SYSSEL:\\s*(\\d+),(\\d+),(\\d+),(\\d+)", G_REGEX_UNGREEDY, 0, NULL);
g_assert (r != NULL); g_assert (r != NULL);
if (!g_regex_match_full (r, response, strlen (response), 0, 0, &match_info, error)) { if (!g_regex_match_full (r, response, strlen (response), 0, 0, &match_info, &match_error)) {
g_prefix_error (error, if (match_error) {
"Failed to parse mode/tech response: "); g_propagate_error (error, match_error);
} else {
g_set_error (error,
MM_CORE_ERROR,
MM_CORE_ERROR_FAILED,
"Couldn't match +SYSSEL reply: %s", response);
}
g_regex_unref (r); g_regex_unref (r);
return FALSE; return FALSE;
} }