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>

Signed-off-by: NLuca Barbato <lu_zero@gentoo.org>

Libgsm header can reside either in the base include dir or in
the gsm subdir.
Signed-off-by: NLuca Barbato <lu_zero@gentoo.org>

This saves one instruction in the x86-64 assembly.

This allows us to get rid of them on the next major bump. All of the
above are functionally irrelevant, and most of them are unused, except
the vp3 one, which is used wrongly in the bfin arch optimizations.

This allows us to remove FF_IDCT_WMV2, which serves no practical purpose
other than to be able to select the WMV2 IDCT for MPEG (or vice versa)
and get corrupt output.

Fate tests for all wmv2-related tests change, because (for some obscure
reason) they forced use of the MPEG IDCT. You would get the same changes
previously by not using -idct simple in the fate test (or replacing it
with -idct auto).

The function is unused.

This simplifies the code slightly.
Signed-off-by: NMartin Storsjö <martin@martin.st>

Signed-off-by: NMartin Storsjö <martin@martin.st>

Signed-off-by: NMartin Storsjö <martin@martin.st>

The theoretical minimum for a (not totally well formed) RTCP packet
is 8 bytes, so we shouldn't require 12 bytes as minimum input.

Also return AVERROR_INVALIDDATA instead of 0 if something that is
not a proper packet is given.
Signed-off-by: NMartin Storsjö <martin@martin.st>

This clarifies where the limit number comes from, and only
requires exactly as much padding space as will be needed.
Signed-off-by: NMartin Storsjö <martin@martin.st>

The main difference to the existing suites from RFC 4568 is
that the version with a 32 bit HMAC still uses 80 bit HMAC
for RTCP packets.
Signed-off-by: NMartin Storsjö <martin@martin.st>

This is untested in practice, but follows the spec.
Signed-off-by: NMartin Storsjö <martin@martin.st>

Signed-off-by: NMartin Storsjö <martin@martin.st>

Use a noise range of -0.707 to 0.707 instead of -0.5 to 0.5
Signed-off-by: NJustin Ruggles <justin.ruggles@gmail.com>

Also fixes compilation with AltiVec disabled.

Signed-off-by: NMartin Storsjö <martin@martin.st>

This gets rid of almost all the codec specific details from the
generic rtpdec code.
Signed-off-by: NMartin Storsjö <martin@martin.st>

This also adds checking of mallocs.
Signed-off-by: NMartin Storsjö <martin@martin.st>

This gets rid of a number of special cases from the common rtpdec
code.
Signed-off-by: NMartin Storsjö <martin@martin.st>

Signed-off-by: NMartin Storsjö <martin@martin.st>

The lavf-internal parser functions are used when receiving
mpegts over RTP. This fixes memory leaks in this setup.

The normal mpegts demuxer close function was updated in ec7d0d2e in
2004 to fix leaks, but the parsing function used for RTP wasn't
updated and has been leaking ever since.
Signed-off-by: NMartin Storsjö <martin@martin.st>

This makes the returned data valid to stream copy into other
containers as well, not only for decoding straight away.
Signed-off-by: NMartin Storsjö <martin@martin.st>

This fixes build failures on ppc machines with a compiler that
supports -Werror=implicit-function-declaration.
Signed-off-by: NMartin Storsjö <martin@martin.st>

libavutil/arm/asm.S sets '.arch' depending on HAVE_ARMV5TE so that
assembling armv5te code will always succeed even if the default -march
flag does not support it. HAVE_ARMV5TE_EXTERNAL tests assembling code
with the default arch.
Fixes the missing symbol ff_prefetch_arm with --cpu= not including
armv5te.

CC: libav-stable@libav.org

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

Too much code relies in having init_get_bits fed with a valid
buffer and set its dimension to 0.

Check for NULL buffer instead.

The function is never used.

Conveniently (together with Justin's earlier patches), this makes
our vorbis decoder entirely independent of dsputil.

This is identical to what e.g. vp8 does, and prevents the function call
overhead (plus dependency on dsputil for this particular function).

Arm asm updated by Janne Grunau <janne-libav@jannau.net>.
Signed-off-by: NJanne Grunau <janne-libav@jannau.net>

The loop condition `x = ex' is incorrect.  It should be `x <= ex'.

This bug was introduced in commit c65dfac4 "mpegvideo.c: K&R formatting
and cosmetics."

CC:libav-stable@libav.org
Signed-off-by: NLuca Barbato <lu_zero@gentoo.org>

Signed-off-by: NAlexandra Khirnova <alexandra.khirnova@gmail.com>
Signed-off-by: NLuca Barbato <lu_zero@gentoo.org>

This provides a fallback when building with Yasm enabled, but neither
inline assembly, nor the _mm_empty intrinsic are available or enabled.
Signed-off-by: NDiego Biurrun <diego@biurrun.de>

This allows defining externally visible library symbols.
Signed-off-by: NDiego Biurrun <diego@biurrun.de>

The new name is more descriptive and will allow defining a separate
public prefix for externally visible library symbols.
Signed-off-by: NDiego Biurrun <diego@biurrun.de>

These flags are as linker-specific as other LDFLAGS and thus
need to be translated to the correct linker syntax.
Signed-off-by: NDiego Biurrun <diego@biurrun.de>

ref_list is constructed from other fields per slice when needed, so do
not copy it for both frame and slice threading.
default_ref_list is constructed per frame and still needs to be copied
to per-slice contexts for slice threading, but a copy is not needed for
frame threading.