On 26.06.2015 01:11, Michael Niedermayer wrote: > On Thu, Jun 25, 2015 at 10:54:17PM +0200, Andreas Cadhalpun wrote: >> get_bits should not be used with more than 25 bits. >> >> Signed-off-by: Andreas Cadhalpun <andreas.cadhal...@googlemail.com> >> --- >> libavcodec/vc1dec.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/libavcodec/vc1dec.c b/libavcodec/vc1dec.c >> index 3fa39a5..d7a0cef 100644 >> --- a/libavcodec/vc1dec.c >> +++ b/libavcodec/vc1dec.c >> @@ -469,7 +469,7 @@ static av_cold int vc1_decode_init(AVCodecContext *avctx) >> count = avctx->extradata_size*8 - get_bits_count(&gb); >> if (count > 0) { > >> av_log(avctx, AV_LOG_INFO, "Extra data: %i bits left, value: >> %X\n", > > maybe this should use "value32:" or something
value32 isn't really informative either. >> - count, get_bits(&gb, count)); >> + count, get_bits_long(&gb, FFMIN(count, 32))); > > LGTM Pushed. Best regards, Andreas _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel