[FFmpeg-devel] [PATCH] sanity check on over reading buffer (roundup issue 2495)

[Michael Niedermayer]

On Wed, Jan 05, 2011 at 07:47:08PM -0500, Daniel Kang wrote:
> I am a Google Code-In student, and I have zzufed a file. In some cases,
> for mm files, when buf_size is larger than the height, ffmpeg overreads
> the buffer. The patch attached adds a sanity check and a warning for
> this. The roundup issue is 2495. Are there any comments?

>  mmvideo.c |    9 +++++++++
>  1 file changed, 9 insertions(+)
> f4e96983a03e48344d351d9cbdcad2575dae2c08  mm_height_sanity_check.diff
> From cf8fefc71ef79815969fd34d21d4f05247b576a9 Mon Sep 17 00:00:00 2001
> From: Daniel Kang <daniel.d.kang at gmail.com>
> Date: Wed, 5 Jan 2011 19:31:09 -0500
> Subject: [PATCH] Add sanity check on height with mm video.

A modified version of this patch has been applied in r26300
thx for the work

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Frequently ignored awnser#1 FFmpeg bugs should be sent to our bugtracker. User
questions about the command line tools should be sent to the ffmpeg-user ML.
And questions about how to use libav* should be sent to the libav-user ML.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20110518/07b64ee8/attachment.asc>