Locking the decoder against channel config changes in
parse_adts_frame_header() seems to be unnecessary and
streams with channel config changes are reported.

The sample in http://roundup.libav.org/issue999 still works.
Signed-off-by: Janne Grunau <janne-libav@jannau.net>

This avoids having to convert all input data from int16 to float, which is used
internally for encoding.

instead just print a warning

avoids silently truncating the output

Fix segfault on emu edge, to reproduce

make fate-vsynth1-snow
avplay -flags emu_edge tests/data/vsynth1/snow.avi
Signed-off-by: Luca Barbato <lu_zero@gentoo.org>

this allows for checking against the actual output size instead of max size.

Also fix some undefined unsigned/signed conversions.

buffer during entropy decoding.

The pointer address could overflow, which would likely segfault. Instead set
the context error flag to indicate that the decoder tried to read past the
end of the packet data.

avoids implementation-defined unsigned-to-signed conversion and simplifies
the bounds checking.

The variable is not used anymore at that point.

This prevents errors and/or invalid writes in the next decode call due to
s->samples still being negative.

break some excessively long lines and remove space after '*'

This is already checked by avcodec_decode_audio3().