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. _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
