[gstreamer-bugs] [Bug 571146] mpeg2dec: possibly uses wrong strides for 4:2:2 and 4:4:4 YUV with unusual display width or height

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=571146
  GStreamer | gst-plugins-ugly | 0.10.x

Tim-Philipp Müller <t.i.m> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|[mpeg2dec] Probably uses    |mpeg2dec: possibly uses
                   |wrong strides for 4:2:2 and |wrong strides for 4:2:2 and
                   |4:4:4 YUV                   |4:4:4 YUV with unusual
                   |                            |display width or height

--- Comment #4 from Tim-Philipp Müller <t.i.m at zen.co.uk> 2010-12-27 00:34:08 UTC ---
> Is there any reason why the above code is still correct, like
> size always being an integer multiply of 4?

AIUI mpeg2dec will always decode into a buffer allocated for
decoding_{width,height}, which is guaranteed to be a multiple of 16. In the
case where the display width or height are less than decoding_width/height we
create a new buffer (using new_and_alloc it seems, yuk) and crop the picture
'manually' in the mpeg2dec element by copying from the decoded buffer into the
display buffer with our own strides.

In short: we need to check the crop_copy_i422_buffer() and
crop_copy_i420_buffer() funcs I guess.

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