[FFmpeg-devel] [PATCH] avcodec/svq1: zero initialize entries array

Michael Niedermayer michael at niedermayer.cc
Mon Apr 10 14:37:54 EEST 2017


On Mon, Apr 10, 2017 at 08:02:34AM +0200, Clément Bœsch wrote:
> On Sun, Apr 09, 2017 at 11:10:16PM -0300, James Almer wrote:
> [...]
> > > is this a regression ?
> > 
> > Fate didn't always complain about this, so it's either something
> > introduced by a change in our tree, or a valgrind bug introduced in a
> > relatively recent version.
> > 
> 
> > The reports in
> > http://fate.ffmpeg.org/history.cgi?slot=x86_64-archlinux-gcc-valgrindundef
> > are kinda broken and report a nonsense commit as the "last known good
> > ref", so i can't say when it started failing.
> 
> That was because I couldn't upgrade for a long time and bumped both GCC
> and Valgrind (and actually the whole system) at once. Sorry, I couldn't do
> it gradually.

does your fate client have optimizations enabled ?
i dont see them being disabled, and iam not sure valgrind + full
compiler optimizations are a good idea

if they are enabled, please try to disable them to check if that
causes the false positives

my local valgrind shows nothing for svq1

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The misfortune of the wise is better than the prosperity of the fool.
-- Epicurus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170410/1c03d97f/attachment.sig>


More information about the ffmpeg-devel mailing list