Anton Khirnov:
> Just noticed
> static int16_t basis[64][64];
> in mpegvideo_enc.c, which will be accessed in a racy way if
> quantizer_noise_shaping is enabled. This applies to all mpegvideo
> encoders as far as I can tell.
> 
The reason I didn't spot this is that this part of the code is not
accessible from any init function. And I am not certain that the code
as-is is even correct: basis depends upon the idct_permutation in use
and so one might need different bases for different callers, i.e. one
would need to make this part of the context.

- Andreas
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to