examples/qsvdec: do not free the surfaces in the frame_free() callback
authorAnton Khirnov <anton@khirnov.net>
Tue, 11 Aug 2015 12:57:07 +0000 (14:57 +0200)
committerAnton Khirnov <anton@khirnov.net>
Mon, 28 Sep 2015 13:45:29 +0000 (15:45 +0200)
Even though libmfx might not need them anymore, avcodec might still
access the surfaces. So free them separately at the end.

doc/examples/qsvdec.c

index 04e5b5f..2f26f41 100644 (file)
@@ -116,15 +116,6 @@ fail:
 
 static mfxStatus frame_free(mfxHDL pthis, mfxFrameAllocResponse *resp)
 {
-    DecodeContext *decode = pthis;
-
-    if (decode->surfaces)
-        vaDestroySurfaces(decode->va_dpy, decode->surfaces, decode->nb_surfaces);
-    av_freep(&decode->surfaces);
-    av_freep(&decode->surface_ids);
-    av_freep(&decode->surface_used);
-    decode->nb_surfaces = 0;
-
     return MFX_ERR_NONE;
 }
 
@@ -144,6 +135,16 @@ static mfxStatus frame_get_hdl(mfxHDL pthis, mfxMemId mid, mfxHDL *hdl)
     return MFX_ERR_NONE;
 }
 
+static void free_surfaces(DecodeContext *decode)
+{
+    if (decode->surfaces)
+        vaDestroySurfaces(decode->va_dpy, decode->surfaces, decode->nb_surfaces);
+    av_freep(&decode->surfaces);
+    av_freep(&decode->surface_ids);
+    av_freep(&decode->surface_used);
+    decode->nb_surfaces = 0;
+}
+
 static void free_buffer(void *opaque, uint8_t *data)
 {
     int *used = opaque;
@@ -467,6 +468,8 @@ finish:
 
     av_frame_free(&frame);
 
+    free_surfaces(&decode);
+
     if (decode.mfx_session)
         MFXClose(decode.mfx_session);
     if (decode.va_dpy)