Re: [spice-gtk v1 2/2] spice-widget: fix leak of boxed type cursor-shape

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> 
> From: Victor Toso <me@xxxxxxxxxxxxxx>
> 
> No valgrind on this one but a leak similar to previous patch.
> This one was introduced in e691954be459087
> 
> Signed-off-by: Victor Toso <victortoso@xxxxxxxxxx>
> ---
>  src/spice-widget.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/src/spice-widget.c b/src/spice-widget.c
> index b60ca2b..1b79ad3 100644
> --- a/src/spice-widget.c
> +++ b/src/spice-widget.c
> @@ -2993,6 +2993,7 @@ static void channel_new(SpiceSession *s, SpiceChannel
> *channel, gpointer data)
>          g_object_get(G_OBJECT(channel), "cursor", &cursor_shape, NULL);
>          if (cursor_shape != NULL) {
>              cursor_set(d->cursor, NULL, display);
> +            g_boxed_free(spice_cursor_shape_get_type(), cursor_shape);
>          }
>          return;
>      }

Why not using SPICE_TYPE_CURSOR_SHAPE ?
You can release cursor_shape even before calling cursor_set.

Frediano
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]