[FFmpeg-devel] [PATCH] fix for roundup issue 2471

Aurelien Jacobs aurel
Sun Jan 2 23:55:25 CET 2011

On Sat, Jan 01, 2011 at 03:26:15PM -0500, Daniel Kang wrote:
> I am a Google Code-In student and as part of a task, I have used zzuf to
> generate a file that crashes ffmpeg. The issue is caused by invalid mkv
> timestamps.
> The patch attached fixes roundup issue 2471. It adds a sanity check
> before incrementing the timestamp.

I don't think it's the proper place to fix this issue.
Attached patch should fix the root of this issue.

Currently, the mpeg4video decoder corretly detects when time_base.den is
null and error out, but it leaves time_base.num with it's non-null
value. My patch just reset time_base.num to 0 before erroring out.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: m4vdec_issue2471.diff
Type: text/x-diff
Size: 469 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20110102/1d29e70b/attachment.diff>

More information about the ffmpeg-devel mailing list