Am 01.02.2017 um 13:59 schrieb Nayan Deshmukh:
we anyway allow for multiple slices

Signed-off-by: Nayan Deshmukh <nayan26deshm...@gmail.com>
---
  src/gallium/state_trackers/va/picture_mpeg12.c | 1 -
  1 file changed, 1 deletion(-)

diff --git a/src/gallium/state_trackers/va/picture_mpeg12.c 
b/src/gallium/state_trackers/va/picture_mpeg12.c
index 812e9e5..4d9c45f 100644
--- a/src/gallium/state_trackers/va/picture_mpeg12.c
+++ b/src/gallium/state_trackers/va/picture_mpeg12.c
@@ -81,6 +81,5 @@ void vlVaHandleIQMatrixBufferMPEG12(vlVaContext *context, 
vlVaBuffer *buf)
void vlVaHandleSliceParameterBufferMPEG12(vlVaContext *context, vlVaBuffer *buf)
  {
-   assert(buf->size >= sizeof(VASliceParameterBufferMPEG2) && 
buf->num_elements == 1);

NAK, we can lower the requirements here, but we shouldn't completely remove the assert.

Especially since we ignore all of the fields from the VASliceParameterBufferMPEG2 structure and so won't catch errors.

Please test something like the following:

assert(buf->size >= (sizeof(VASliceParameterBufferMPEG2) * buf->num_elements));

Regards,
Christian.

     context->desc.mpeg12.num_slices += buf->num_elements;
  }


_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to