On Wed, Feb 1, 2012 at 2:18 PM, Ronald S. Bultje <[email protected]> wrote:
> Hi,
>
> On Wed, Feb 1, 2012 at 1:26 PM, Alex Converse <[email protected]> wrote:
>> 2012/2/1 Måns Rullgård <[email protected]>:
>>> "Ronald S. Bultje" <[email protected]> writes:
>>>
>>>> Prevent values in floor1[] from wrapping over uint16_t boundaries (in
>>>> crafted bitstreams, they can go < 0), which causes them to wrap to
>>>> MAXUINT16, causing huge jumps in the dB LUT indexes. Likewise, clip
>>>> (rather than wrap) dB LUT indexes, to prevent jumping of indexes at
>>>> uint8_t wrapping boundaries.
>>>
>>> Why does it matter how invalid streams are decoded, as long as it
>>> doesn't crash or overrun buffers?
>>>
>>
>> Are large floor1 values actually forbidden? I've noticed that in both
>> Vorbis and AAC a lot of very stupid behavior is legal despite
>> increased decoder complexity to handle the behavior properly.
>
> The vorbis devs say my patch is correct.
>

That's not what I asked.
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to