[PATCH 4/5] allow space in response to +CRSM=176,28589,0,0,4 (mnc) and +CRSM=176,28486,0,0,17 (spn)

Aleksander Morgado aleksander at aleksander.es
Thu Feb 11 14:38:23 UTC 2016


On 11/02/16 13:45, Thomas Sailer wrote:
> From: Thomas Sailer <t.sailer at alumni.ethz.ch>
> 
> Signed-off-by: Thomas Sailer <t.sailer at alumni.ethz.ch>
> ---
>  src/mm-base-sim.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/src/mm-base-sim.c b/src/mm-base-sim.c
> index 20b9923..e6bb5ba 100644
> --- a/src/mm-base-sim.c
> +++ b/src/mm-base-sim.c
> @@ -1111,7 +1111,7 @@ parse_mnc_length (const gchar *response,
>          success = TRUE;
>      else {
>          /* May not include quotes... */
> -        if (sscanf (response, "+CRSM:%d,%d,%50c", &sw1, &sw2, (char *) &hex) == 3)
> +        if (sscanf (response, "+CRSM:%d,%d, %50c", &sw1, &sw2, (char *) &hex) == 3)
>              success = TRUE;
>      }
>  
> @@ -1248,7 +1248,7 @@ parse_spn (const gchar *response,
>          success = TRUE;
>      else {
>          /* May not include quotes... */
> -        if (sscanf (response, "+CRSM:%d,%d,%50c", &sw1, &sw2, (char *) &hex) == 3)
> +        if (sscanf (response, "+CRSM:%d,%d, %50c", &sw1, &sw2, (char *) &hex) == 3)
>              success = TRUE;
>      }
>  

Same as the previous patch, a common mm_3gpp_parse_crsm_response() could
help here? (with mm_3gpp_ prefix, no mm_modem_helper_... as I said in
the previous email).

-- 
Aleksander
https://aleksander.es


More information about the ModemManager-devel mailing list