[FFmpeg-devel] [PATCH] Fix uninitialized reads for fate-vsynth1-asv2 test.

Michael Niedermayer michaelni at gmx.at
Sun Jan 15 14:07:27 CET 2012


On Sun, Jan 15, 2012 at 01:57:07PM +0100, Reimar Döffinger wrote:
> On Sun, Jan 15, 2012 at 01:10:00PM +0100, Michael Niedermayer wrote:
> > On Sun, Jan 15, 2012 at 11:04:23AM +0100, Reimar Döffinger wrote:
> > > This is not a real error and memsetting always even when the
> > > size did not change is overkill, but I'd still like to propose
> > > this.
> > > 
> > > Signed-off-by: Reimar Döffinger <Reimar.Doeffinger at gmx.de>
> > 
> > should do no harm, thus ok
> 
> Pushed.
> Let's see how far a full valgrind run (with uninitialized reads not
> disabled).
> Zlib init function seems broken, so I needed a suppression file for that
> already.
> I see the valgrind fate also has a custom suppression file.

> Maybe we should collect these and add a valgrind suppression file into
> git?

this sounds like a pretty good idea

[...] 

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Complexity theory is the science of finding the exact solution to an
approximation. Benchmarking OTOH is finding an approximation of the exact
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120115/f998a5f2/attachment.asc>


More information about the ffmpeg-devel mailing list