[Ffmpeg-devel] [PATCH] a FIXME in lavc/utils.c

Michael Niedermayer michaelni
Thu Jan 18 14:18:00 CET 2007


Hi

On Thu, Jan 18, 2007 at 03:49:38AM -0200, ramiro at lisha.ufsc.br wrote:
> Hello,
> 
> Is this the correct solution for this FIXME?
> Regression tests pass.
> 
> Ramiro Polla
> Index: libavcodec/utils.c
> ===================================================================
> --- libavcodec/utils.c	(revision 7576)
> +++ libavcodec/utils.c	(working copy)
> @@ -301,7 +301,7 @@
>  
>              buf->linesize[i]= picture.linesize[i];
>  
> -            buf->base[i]= av_malloc(size[i]+16); //FIXME 16
> +            buf->base[i]= av_malloc(size[i]+FF_INPUT_BUFFER_PADDING_SIZE);

NO!

16 is the macroblock size here

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Observe your enemies, for they first find out your faults. -- Antisthenes
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20070118/e2b8ef8c/attachment.pgp>



More information about the ffmpeg-devel mailing list