On Mon, Sep 04, 2023 at 11:48:11AM -0300, Fabiano Rosas wrote:
> @@ -189,11 +193,14 @@ static void iothread_init(EventLoopBase *base, Error 
> **errp)
>          return;
>      }
>  
> +    thread_name = g_strdup_printf("IO %s",
> +                        object_get_canonical_path_component(OBJECT(base)));
> +
>      /*
>       * Init one GMainContext for the iothread unconditionally, even if
>       * it's not used
>       */
> -    iothread_init_gcontext(iothread);
> +    iothread_init_gcontext(iothread, thread_name);
>  
>      iothread_set_aio_context_params(base, &local_error);
>      if (local_error) {

I think thread_name might be leaked if error here.  Thanks,

-- 
Peter Xu


Reply via email to