On 1 April 2017 at 18:18, Michael Niedermayer <[email protected]> wrote:
> Fixes: undefined behavior > Fixes: 702974 > > Found-by: Thomas Guilbert <[email protected]> > Signed-off-by: Michael Niedermayer <[email protected]> > --- > libavformat/oggparsedaala.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/libavformat/oggparsedaala.c b/libavformat/oggparsedaala.c > index ce65b2bd7a..a373b41b4c 100644 > --- a/libavformat/oggparsedaala.c > +++ b/libavformat/oggparsedaala.c > @@ -126,6 +126,7 @@ static int daala_header(AVFormatContext *s, int idx) > if (hdr->gpshift >= 32) { > av_log(s, AV_LOG_ERROR, "Too large gpshift %d (>= 32).\n", > hdr->gpshift); > + hdr->gpshift = 0; > return AVERROR_INVALIDDATA; > } > hdr->gpmask = (1U << hdr->gpshift) - 1; > -- > 2.11.0 > > _______________________________________________ > ffmpeg-devel mailing list > [email protected] > http://ffmpeg.org/mailman/listinfo/ffmpeg-devel > LGTM, thanks _______________________________________________ ffmpeg-devel mailing list [email protected] http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
