On Wed, Nov 22, 2017 at 10:56:07AM -0800, Dale Curtis wrote: > On Tue, Nov 21, 2017 at 7:05 PM, Michael Niedermayer <mich...@niedermayer.cc > > wrote: > > > > I dont think wrap_bits can/should be > 64 or do i miss something ? > > > > Good point, this seems true with the current code. > > > > > > maybe a av_assert* for that would be better. > > > > Done; used av_assert2().
> utils.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > e49bb30e3ddb00bc9f6780c924bfe97e710d7e99 wrap_bits_v5.patch > From 0297fdc52f043592235d93076f91d38c3df40a14 Mon Sep 17 00:00:00 2001 > From: Dale Curtis <dalecur...@chromium.org> > Date: Fri, 17 Nov 2017 13:35:56 -0800 > Subject: [PATCH] [avformat] Prevent undefined shift with wrap_bits > 64. > > 2LL << (wrap_bits=64 - 1) does not fit in int64_t; change the > code to use a uint64_t (2ULL) and add an av_assert2() to > ensure wrap_bits <= 64. > > Signed-off-by: Dale Curtis <dalecur...@chromium.org> > --- > libavformat/utils.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) will apply thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB What does censorship reveal? It reveals fear. -- Julian Assange
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
