[FFmpeg-devel] [PATCH 03/12] mdct: remove temporary array in ff_kbd_window_init()
Thu Jun 24 01:37:34 CEST 2010
Michael Niedermayer <michaelni at gmx.at> writes:
> On Wed, Jun 23, 2010 at 06:26:41PM +0100, Mans Rullgard wrote:
>> The intermediate values can be stored in the output array, avoiding
>> the need for a variable-length array.
> this can write into static arrays and thus introduces a race condition
Quite. Which would you prefer then, 1) malloc/free or, 2) always
allocating a  array (largest size used)? Option 2 would use 4k
(float) or 8k (double) of stack space, which is IMO a bit larger than
what's comfortable. It would of course not change anything compared
to current code so should be safe.
mans at mansr.com
More information about the ffmpeg-devel