On Thu, 2013-04-04 at 12:40 -0700, Ben Chan wrote:
> ---
> src/mm-broadband-bearer.c | 8 +++++---
> 1 file changed, 5 insertions(+), 3 deletions(-)
I'll assume then that somebody has a \ or " in their APN then, right?
In any case, patch looks good.
Dan
> diff --git a/src/mm-broadband-bearer.c b/src/mm-broadband-bearer.c
> index c34867d..fe894c9 100644
> --- a/src/mm-broadband-bearer.c
> +++ b/src/mm-broadband-bearer.c
> @@ -753,7 +753,7 @@ find_cid_ready (MMBaseModem *modem,
> DetailedConnectContext *ctx)
> {
> GVariant *result;
> - gchar *command;
> + gchar *apn, *command;
> GError *error = NULL;
> const gchar *pdp_type;
>
> @@ -783,10 +783,12 @@ find_cid_ready (MMBaseModem *modem,
> }
>
> ctx->cid = g_variant_get_uint32 (result);
> - command = g_strdup_printf ("+CGDCONT=%u,\"%s\",\"%s\"",
> + apn = mm_at_serial_port_quote_string (mm_bearer_properties_get_apn
> (mm_bearer_peek_config (MM_BEARER (ctx->self))));
> + command = g_strdup_printf ("+CGDCONT=%u,\"%s\",%s",
> ctx->cid,
> pdp_type,
> - mm_bearer_properties_get_apn
> (mm_bearer_peek_config (MM_BEARER (ctx->self))));
> + apn);
> + g_free (apn);
> mm_base_modem_at_command_full (ctx->modem,
> ctx->primary,
> command,
_______________________________________________
networkmanager-list mailing list
[email protected]
https://mail.gnome.org/mailman/listinfo/networkmanager-list
