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

Uri Lublin uril at redhat.com
Sun May 27 14:50:40 UTC 2018


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>
---

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;
     }
 
-- 
2.17.0



More information about the Spice-devel mailing list