[Spice-devel] [spice-gtk PATCH v2 1/2] mjpeg_decoder_queue_frame: free frame when dropping the frame

[Frediano Ziglio]

> 
> The frame is dropped and is not inserted into a queue to be
> freed later.
> mjpeg_decoder_queue_frame owns the frame and must free it.
> 
> Signed-off-by: Uri Lublin <uril at redhat.com>

For the series:

Acked-by: Frediano Ziglio <fziglio at redhat.com>

> ---
> 
> changes since v1: information added to commit log
> 
> ---
>  src/channel-display-mjpeg.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/src/channel-display-mjpeg.c b/src/channel-display-mjpeg.c
> index 8c4c0aa..7b2f775 100644
> --- a/src/channel-display-mjpeg.c
> +++ b/src/channel-display-mjpeg.c
> @@ -262,6 +262,7 @@ static gboolean mjpeg_decoder_queue_frame(VideoDecoder
> *video_decoder,
>       * So drop late frames as early as possible to save on processing time.
>       */
>      if (latency < 0) {
> +        frame->free(frame);
>          return TRUE;
>      }
>  

Frediano