[FFmpeg-devel] [PATCH] Description of avcodec_encode_audio()

John Fletcher john.fletcher
Thu Apr 2 12:33:43 CEST 2009


> patch rejected, the buffer MUST be FF_MIN_BUFFER_SIZE bytes large
> and "is no longer applied" is incorrect it was never applied, the user
> can if he either knows by some means that the buffer will be 
> large enough
> or works in a secure environment where the input is known not 
> to contain
> an exploit use a smaller buffer and deal with the segfaults somehow
> The PCM case falls in the category where the user might know how much
> space will be needed

OK, here is a revised patch.

I was misled by this comment so I think further explanation
is helpful.

When encoding a video frame's worth of audio samples to PCM,
you typically need to set buf_size to a value lower than
FF_MIN_BUFFER_SIZE.

John
-------------- next part --------------
A non-text attachment was scrubbed...
Name: avcodec.h.diff
Type: application/octet-stream
Size: 586 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090402/e72bf665/attachment.obj>



More information about the ffmpeg-devel mailing list