libs: decoder: release VA buffers after vaEndPicture

This change is due a problem decoding JPEGs with Intel's media-driver:
no image was generated.

This patch relases the VA buffers after vaEndPicture() is called,
and not before (after vaRenderPicture()).

https://bugzilla.gnome.org/show_bug.cgi?id=796505

diff --git a/gst-libs/gst/vaapi/gstvaapidecoder_objects.c b/gst-libs/gst/vaapi/gstvaapidecoder_objects.c
index 20d4f55..2dd4c27 100644 (file)
--- a/gst-libs/gst/vaapi/gstvaapidecoder_objects.c
+++ b/gst-libs/gst/vaapi/gstvaapidecoder_objects.c
@@ -304,12 +304,17 @@ gst_vaapi_picture_decode (GstVaapiPicture * picture)
     status = vaRenderPicture (va_display, va_context, va_buffers, 2);
     if (!vaapi_check_status (status, "vaRenderPicture()"))
       return FALSE;
+  }
+
+  status = vaEndPicture (va_display, va_context);
+
+  for (i = 0; i < picture->slices->len; i++) {
+    GstVaapiSlice *const slice = g_ptr_array_index (picture->slices, i);
 
     vaapi_destroy_buffer (va_display, &slice->param_id);
     vaapi_destroy_buffer (va_display, &slice->data_id);
   }
 
-  status = vaEndPicture (va_display, va_context);
   if (!vaapi_check_status (status, "vaEndPicture()"))
     return FALSE;
   return TRUE;