diff options
author | Aleksander Morgado <aleksander@lanedo.com> | 2012-09-19 07:35:23 +0200 |
---|---|---|
committer | Aleksander Morgado <aleksander@lanedo.com> | 2012-09-19 07:42:47 +0200 |
commit | a6faae32608833f3d26d57bc6bace4f2a9add8bd (patch) | |
tree | 4dc6bb0e66272ebc9f2e993132105dc1e61ec4d3 /src/mm-broadband-modem.c | |
parent | d2353e01ff4b786da4542f7fd6f4f4e76081f450 (diff) |
broadband-modem: skip +CGMM: prefix when loading device model
Some devices, e.g. ZTE MF820D, seem to prefix the `AT+CGMM?' response with the
`+CGMM:' string, resulting in the following model string being loaded:
model: '+CGMM: "MF820D"'
Avoid this by:
1) Removing the expected prefixes.
2) Unquoting the resulting string.
Reported by: Marius Kotsbak <marius.kotsbak@gmail.com>
Diffstat (limited to 'src/mm-broadband-modem.c')
-rw-r--r-- | src/mm-broadband-modem.c | 12 |
1 files changed, 11 insertions, 1 deletions
diff --git a/src/mm-broadband-modem.c b/src/mm-broadband-modem.c index 77650490..4a2fdb8e 100644 --- a/src/mm-broadband-modem.c +++ b/src/mm-broadband-modem.c @@ -634,13 +634,23 @@ modem_load_model_finish (MMIfaceModem *self, GError **error) { GVariant *result; + const gchar *p; gchar *model; result = mm_base_modem_at_sequence_finish (MM_BASE_MODEM (self), res, NULL, error); if (!result) return NULL; - model = g_strstrip (g_variant_dup_string (result, NULL)); + p = g_variant_get_string (result, NULL); + + /* Some devices (e.g. ZTE MF820D) seem to include the command prefix */ + p = mm_strip_tag (p, "+CGMM:"); + p = mm_strip_tag (p, "+GMM:"); + model = g_strdup (p); + + /* Stripping quotes modifies string in place */ + model = mm_strip_quotes (model); + mm_dbg ("loaded model: %s", model); return model; } |