On Tue, Oct 18, 2016 at 11:15:50PM -0300, James Almer wrote: > On 10/17/2016 9:57 PM, Michael Niedermayer wrote: > > On Sun, Oct 16, 2016 at 09:34:50PM -0300, James Almer wrote: > >> Fixes valgrind warning about "Conditional jump or move depends on > >> uninitialised value(s)" > >> > >> Signed-off-by: James Almer <jamr...@gmail.com> > >> --- > >> libavformat/mov.c | 2 +- > >> 1 file changed, 1 insertion(+), 1 deletion(-) > > > > This should be suppressable by adding it to > > tests/fate-valgrind.supp > > I'll leave that to someone else. No idea how to add stuff to that file.
IIRC, valgrind can print the proper formatted supperssion, so should only require copy and paste from that > > Should i drop this patch? Zero initializing a buffer in stack wouldn't > hurt IMO. iam not objecting to the zero init [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB The real ebay dictionary, page 1 "Used only once" - "Some unspecified defect prevented a second use" "In good condition" - "Can be repaird by experienced expert" "As is" - "You wouldnt want it even if you were payed for it, if you knew ..."
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
