On Wed, Apr 23, 2014 at 12:20:01PM -0400, Derek Buitenhuis wrote: > Signed-off-by: Derek Buitenhuis <[email protected]> > --- > libavcodec/fic.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/fic.c b/libavcodec/fic.c > index 45f56b9..d389f7b 100644 > --- a/libavcodec/fic.c > +++ b/libavcodec/fic.c > @@ -296,7 +296,7 @@ static int fic_decode_frame(AVCodecContext *avctx, void > *data, > /* Skip cursor data. */ > tsize = AV_RB24(src + 24); > if (tsize > avpkt->size - FIC_HEADER_SIZE) { > - av_log(avctx, AV_LOG_ERROR, "Invalid cursor data size.\n"); > + av_log(avctx, AV_LOG_ERROR, "Invalid cursor data size: %d.\n", > tsize); > return AVERROR_INVALIDDATA; > } > > --
would be even better if you recognize truncated packet because cursor data size 4xxx would look legit to me _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
