2018-02-14 1:54 GMT+01:00 James Almer <jamr...@gmail.com>: > Calling ffio_ensure_seekback() right after initializing an AVIOContext > with ffio_init_context() would result in a use of uninitialised value. > > Fixes fate-adts-id3v2-demux when using valgrind. > > Signed-off-by: James Almer <jamr...@gmail.com> > --- > Maybe we should do a memset(s, 0, sizeof(*s)) at the beginning of > ffio_init_context() instead, to effectively initialize every field to > zero. Which is prefered?
Imo, a memset is more readable (and I suspect it makes it less likely a compiler does something unreasonable). Carl Eugen _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel