This uses the RIFF header stored size to figure out the expected AVI
file size, instead of the actual file. To work fully it requires handling
failed avio_seek() instead of assuming they always succeed.

Some fate file has been cut off and contains half a frame at the end which
previously was not output during demuxing. This frame is now output to
encoder, thus the fate diff update.

Bug-Id: 261
Signed-off-by: NVittorio Giovara <vittorio.giovara@gmail.com>

Additional fixes by Nigel Touati-Evans <nigel.touatievans@gmail.com>.

Check the index for streams with a time drift of 2s or a buffer drift
of 64MB.

Bug-Id: 666
CC: libav-stable@libav.org
Sample-Id: yet-another-broken-interleaved-avi.avi
Signed-off-by: NVittorio Giovara <vittorio.giovara@gmail.com>
Signed-off-by: NLuca Barbato <lu_zero@gentoo.org>
Signed-off-by: NDiego Biurrun <diego@biurrun.de>

Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC:libav-stable@libav.org

Reported-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-stable@libav.org

Reported-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-stable@libav.org

CC: libav-stable@libav.org

Reported-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-stable@libav.org
Signed-off-by: NMartin Storsjö <martin@martin.st>

When av_reallocp fails, the associated variables that keep track of
the number of elements in the array (and in some cases, the
separate number of allocated elements) need to be reset.

Not all of these might technically be needed, but it's better to
reset them if in doubt, to make sure variables don't end up
conflicting.
Signed-off-by: NMartin Storsjö <martin@martin.st>

Signed-off-by: NDiego Biurrun <diego@biurrun.de>

CC: libav-stable@libav.org

Signed-off-by: NDiego Biurrun <diego@biurrun.de>
Signed-off-by: NLuca Barbato <lu_zero@gentoo.org>

This will allow us to avoid copying the packets in many cases.

This breaks ABI.

Use AVERROR_INVALIDDATA on invalid inputs, and AVERROR_EOF when no more
frames are available in an interleaved AVI.
Signed-off-by: NJindrich Makovicka <makovick@gmail.com>
Signed-off-by: NDiego Biurrun <diego@biurrun.de>

Fixes CVE-2012-2788

The specs say they are unsigned 32bit integers.

The caller can achieve the same effect (i.e. getting raw unparsed/mangled
packets) with av_read_frame() and AVFMT_FLAG_NOPARSE |
AVFMT_FLAG_NOFILLIN

Prefix the functions/tables brktimegm, pcm_read_seek,
dv_offset_reset, voc_get_packet, codec_movaudio_tags,
codec_movvideo_tags.

After this, lavf has no global symbols without the proper prefix.
Signed-off-by: NMartin Storsjö <martin@martin.st>

Signed-off-by: NAnton Khirnov <anton@khirnov.net>

It's supposed to be called only from (de)muxers.

Signed-off-by: NRonald S. Bultje <rsbultje@gmail.com>

All current usages of it are incompatible with localization.
For example strcasecmp("i", "I") != 0 is possible, but would
break many of the places where it is used.

Instead use our own implementations that always treat the data
as ASCII.
Signed-off-by: NMartin Storsjö <martin@martin.st>

They are used in libavdevice.

Manual replacements are done in this commit.

In many cases, the id is some constant made up number (e.g. 0 for video
and 1 for audio), which is then not used in the demuxer for anything.
Those ids are removed.

Keeping byte values read from the file as unsigned is consistent
with how they are subsequently used and avoids an undefined left
shift by 24 when bit 7 is set.
Signed-off-by: NMans Rullgard <mans@mansr.com>

Signed-off-by: NAnton Khirnov <anton@khirnov.net>

It's more readable and less prone to breakage.

Signed-off-by: NMans Rullgard <mans@mansr.com>