[Bug 743311] mpeg2dec does not cope if downstream buffer allocation has stride != width

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=743311
  GStreamer | gst-plugins-ugly | git master

--- Comment #10 from Simon Farnsworth <simon at farnz.org.uk> 2015-01-22 18:00:38 UTC ---
Created an attachment (id=295211)
 View: https://bugzilla.gnome.org/attachment.cgi?id=295211
 Review: https://bugzilla.gnome.org/review?bug=743311&attachment=295211

[PATCH] mpeg2dec: Fix handling of stride


A pipeline like:

gst-launch-1.0 filesrc location=file.ts ! tsdemux ! mpegvideoparse ! mpeg2dec 
! vaapisink

would look bad when file.ts contains 704x576 video, because vaapisink would
give you buffers of stride 768, but libmpeg2 was not told about this and
used a stride of 704.

Tell libmpeg2 about the stride from  downstream; in the process, teach it to
reject buffer pools that don't meet libmpeg2's chroma stride requirements

Signed-off-by: Simon Farnsworth <simon.farnsworth at onelan.co.uk>
---
 ext/mpeg2dec/gstmpeg2dec.c | 41 +++++++++++++++++++++++++++++++++++++++++
 1 file changed, 41 insertions(+)

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.