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

This makes vp3 independent of dsputil.
Signed-off-by: NMartin Storsjö <martin@martin.st>

This way, the special IDCT permutations are no longer needed. This
is similar to how H264 does it, and removes the dsputil dependency
imposed by the scantable code.

Also remove the unused type == 0 cases from the plain C version
of the idct.
Signed-off-by: NMartin Storsjö <martin@martin.st>

This makes the vp3 decoder less dependent on dsputil, and will aid
in making it (eventually) dsputil-independent.
Signed-off-by: NMartin Storsjö <martin@martin.st>

It does not help as an abstraction and adds dsputil dependencies.
Signed-off-by: NRonald S. Bultje <rsbultje@gmail.com>

The function is only used in VP3 and VP5, so no need to have it in
DSPContext.

The function is never used.

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>

This fixes a regression since d9cf5f51 with theora over RTP
(possibly with other variants of theora as well).

In theora over RTP, the second of the 3 headers turns out to be
0 bytes long, which prior to d9cf5f51 worked just fine. After
d9cf5f51, reading from the bitstream reader fails (since the reader
wasn't initialized but returned an error if initialized with 0 bits).

CC: libav-stable@libav.org
Signed-off-by: NMartin Storsjö <martin@martin.st>

Signed-off-by: NDale Curtis <dalecurtis@chromium.org>
Signed-off-by: NLuca Barbato <lu_zero@gentoo.org>

Move some functions from dsputil. The idea is that videodsp contains
functions that are useful for a large and varied set of video decoders.
Currently, it contains emulated_edge_mc() and prefetch().
Signed-off-by: NLuca Barbato <lu_zero@gentoo.org>

It's got_frame, not data size

Deprecate avcodec_get_chroma_sub_sample.

This moves all VP3-specific function pointers from dsputil to a
new vp3dsp context.  There is no reason to ever use the VP3 IDCT
where an MPEG2 IDCT is expected or vice versa.
Signed-off-by: NMans Rullgard <mans@mansr.com>

Signed-off-by: NDale Curtis <dalecurtis@chromium.org>
Signed-off-by: NRonald S. Bultje <rsbultje@gmail.com>

Also break some long lines, remove codec function placeholder comments
and add spaces in sample/pixel format lists.
Signed-off-by: NMartin Storsjö <martin@martin.st>

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

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

Fixes a regression introduced in 8b94df0f.

Based on a patch by Michael Niedermayer <michaelni@gmx.at>

Fixes NGS00145, CVE-2011-4352

Found-by: Phillip Langlois
Signed-off-by: NReinhard Tartler <siretart@tauware.de>

Add AV_NUM_DATA_POINTERS to simplify the bump transition.
This will allow for supporting more planar audio channels without having to
allocate separate pointer arrays.

A new field, AVCodecContext.internal is used to hold a new struct
AVCodecInternal, which has private fields that are not codec-specific and are
used by general libavcodec functions.

Moved internal_buffer, internal_buffer_count, and is_copy.

It's used in lavf.

Replace our incomplete w32threads implementation with x264's pthreads
w32threads wrapper.
Relicensed to LGPL with kind permission by Pegasys Inc.
Signed-off-by: NJanne Grunau <janne-libav@jannau.net>

It's more readable and less prone to breakage.

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

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

With negative stride, the start of the edge_emu buffer should be pointing to
the last line, not the end of the buffer.
With positive stride, pointing to the end of the buffer was completely wrong.