[FFmpeg-devel] [PATCH] ac3enc: fix bug in stereo rematrixing decision.
Justin Ruggles
justin.ruggles
Wed Feb 16 01:47:49 CET 2011
On 02/15/2011 07:39 PM, M?ns Rullg?rd wrote:
> Justin Ruggles <justin.ruggles at gmail.com> writes:
>
>> The rematrixing strategy reuse flags are not reset between frames, so they
>> need to be initialized for all blocks, not just block 0.
>> ---
>> We either have to check for !block->new_rematrixing_strategy or blk. I can't remove both. Here is an alternative patch.
>
> Yes, you can. Writing 1 to something that is already 1 is harmless.
but dereferencing a NULL pointer (block0) is not harmless.
-Justin
More information about the ffmpeg-devel
mailing list