[PATCH 3/3] sms-mbim: fix peek_device to use g_task_report_new_error

Aleksander Morgado aleksander at aleksander.es
Tue Jul 18 08:34:57 UTC 2017


On 18/07/17 03:11, Ben Chan wrote:
> When porting MMSmsMbim to use GTask, peek_device wasn't modified to use
> g_task_report_new_error, which could potentially lead to an incorrect
> G_TASK cast.
> ---

Pushed to git master, thanks!

>  src/mm-sms-mbim.c | 13 +++++++------
>  1 file changed, 7 insertions(+), 6 deletions(-)
> 
> diff --git a/src/mm-sms-mbim.c b/src/mm-sms-mbim.c
> index 99e0de74..e20a83d2 100644
> --- a/src/mm-sms-mbim.c
> +++ b/src/mm-sms-mbim.c
> @@ -53,12 +53,13 @@ peek_device (gpointer self,
>  
>          port = mm_base_modem_peek_port_mbim (modem);
>          if (!port) {
> -            g_simple_async_report_error_in_idle (G_OBJECT (self),
> -                                                 callback,
> -                                                 user_data,
> -                                                 MM_CORE_ERROR,
> -                                                 MM_CORE_ERROR_FAILED,
> -                                                 "Couldn't peek MBIM port");
> +            g_task_report_new_error (self,
> +                                     callback,
> +                                     user_data,
> +                                     peek_device,
> +                                     MM_CORE_ERROR,
> +                                     MM_CORE_ERROR_FAILED,
> +                                     "Couldn't peek MBIM port");
>              g_object_unref (modem);
>              return FALSE;
>          }
> 


-- 
Aleksander
https://aleksander.es


More information about the ModemManager-devel mailing list