[Bug 749428] theoraenc: memory errors reported by valgrind

[GStreamer (GNOME Bugzilla)]

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

Tim-Philipp Müller <t.i.m at zen.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |NEEDINFO
                 CC|                            |t.i.m at zen.co.uk

--- Comment #1 from Tim-Philipp Müller <t.i.m at zen.co.uk> ---
Just filing valgrind bugs where it's not even clear whether there's an issue in
GStreamer or not is only useful if you intend to investigate this further.

Typically these uninitialised values are when padding between rows or planes is
accessed. Encoders often do that for efficiency reasons (so they don't have to
deal with 'edge' cases) if resolutions are not a multiple of their block size.

Check the exact addresses involved. Check the address of the video frame. Check
the row stride and the width and the plane offsets; and calculate where the
padding would be and if this is what it is or not.

Alternative add some kind in gst_video_frame_map() or wherever that initialises
the padding bytes.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.