ljpeg: use the correct number of components in yuv

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

mjpeg: Validate sampling factors

They must be non-zero.

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

mpegvideo: unref old current_picture before allocating a new one

Allows decoding with only three frame buffers with frame reference
counting. Fixes VAAPI based decoding in vlc which uses only three
frame buffers for the mpegvideo-based codecs.

vorbis: fallback to normal division instead of crashing

The use of ff_inverse speeds up slightly arches + compilers that
do not provide a division faster than the whole machinery, such
as ppc32 + gcc4.7, but has operational limits.

Drop the always-enable assert and provide a fallback.

vorbis: simplify the inner loop in setup_classifs

vorbis: use normal integer values for iterators

No need to use fixed-size integers.

vorbis: refactor vorbis_residue_decode_internal

win32: Do not use GetProcAddress when compiled for Vista+

Instead link directly to the condition variable functions.

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

configure: msvc: Set the target windows version to XP if no target is set

MSVC 2010 (or more precisely, Windows SDK 7.0 which comes with MSVC
2010) sets _WIN32_WINNT to the constant for Windows 7 if nothing is
set. This could lead to the libav configure script detecting and
using functions only present in Windows 7 or newer, which in most
cases isn't desired. If the caller explicitly wants this, the caller
can add the _WIN32_WINNT define via --extra-cflags, setting the desired
version.

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

Escape 130 FATE test

prores: update FATE test to account for alpha plane present in the test sample

configure: Add basic valgrind-massif support

With the parameter --toolchain valgrind-massif, the configure
script sets reasonable defaults that can be overridden as explained
in the documentation.

wavpack: validate samples size parsed in wavpack_decode_block

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

wavpack: return meaningful errors

And forward those that were already meaningful.

wavpack: K&R formatting cosmetics

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

Escape 130 (RPL) decoder

Some fixes provided by Paul B Mahol <onemda@gmail.com>
and Michael Niedermayer <michaelni@gmx.at> and me.

Signed-off-by: Diego Biurrun <diego@biurrun.de>
Signed-off-by: Kostya Shishkov <kostya.shishkov@gmail.com>

prores: decode alpha plane when it's present

avfilter: check a malloc in avfilter_link().

Additionally change sizeof(type) into sizeof(var)

avfilter.c: cosmetics, reformat

lavfi: factorize freeing a link

vf_format: do not use the AVFilterFormats internals.

Use the formats.h API instead.

af_aformat: cosmetics, fix indentation

lavfi: rename AVFilterFormats.format_count to nb_formats

This is more consistent with naming in the rest of Libav.

doxy: don't sort struct members alphabetically

The declaration order is more meanigful in most cases.

lavr doxy: add version.h to the lavr group

lavc doxy: add avcodec.h to the libavcodec group.

doxy: add a section about versioning.

lavfi doxy: add a page for lavfi.

jpegls: check the scan offset

Prevent an out of array bound write.

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

jpegls: factorize return paths

jpegls: return meaningful errors

sparc: VIS mnemonics

Currently, if VIS is enabled by configure, it will also be enabled at
run-time regardless of its support in the hardware. Thus, masking VIS
usage as it is done in vis.h by constructing binary instructions is
pointless. Using normal VIS mnemonics in inline assembly allows to take
advantage of automatic register allocation, gets rid of register
variables, which are unsupported by suncc for SPARC, and improves code
readability.

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

mpegvideo: allocate sufficiently large scratch buffer for interlaced vid

MPV_decode_mb_internal needs 3 * 16 * linesize bytes of scratch buffer

For interlaced content, linesize is multiplied by two after the allocation
of the scratch buffer, and the dest_cr pointer ends past the buffer.

This patch makes ff_mpv_frame_size_alloc allocate a total of
(aligned line_size) * 2 * 16 * 3 bytes, which suffices even for the
interlaced case.

CC:libav-stable@libav.org

Signed-off-by: Jindrich Makovicka <makovick@gmail.com>
Signed-off-by: Anton Khirnov <anton@khirnov.net>

pthread: reindent

lavc/pthread: remove obsolete checks

Those were useful when avcodec_thread_init() was a public functions. It
was deprecated and removed some time ago, so those checks are not needed
anymore.

vf_pixdesctest: make config_props work properly when called multiple times.

vf_hqdn3d: make config_props work properly when called multiple times.

Do not leak all the temp buffers.

vf_drawtext: don't leak the expressions.

vf_crop: make config_props work properly when called multiple times.

Do not leak the x/y expressions.

vf_setdar: make config_props work properly when called multiple times.

Do not overwrite the variable set through AVOptions.

vf_drawbox: make config_props work properly when called multiple times.

Do not overwrite the variables set through AVOptions.

vf_drawtext: do not reset the frame number in config_input.

Frame number should be incremented normally even if the link properties
change.

vf_fps: move initializing pts from config_props to init.

It should not be reinitialized if the link properties change.

vf_frei0r: make config_props work properly when called multiple times.

Do not leak the initialized filter instance.

vf_gradfun: make config_props work properly when called multiple times.

vf_lut: make config_props work properly when called multiple times.

Do not leak the expressions.

Apple Intermediate Codec decoder

movenc: Don't add base_data_offset in tfhd for PIFF files

According to the PIFF specification[1] the base_data_offset field MUST be
omitteed. See section 5.2.17. Since the ISMV files created by libavformat
state that they are 'piff' compatible via 'ftyp' box, this needs to be
corrected.

[1] http://www.iis.net/learn/media/smooth-streaming/protected-interoperable-file-format

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

vc1dec: Add support for interlaced B-frames

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

sparc: Eliminate dead code in VIS acceleration macros

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

flacdec: drop unnecessary assert

The condition cannot happen anymore.

mjpegdec: properly report unsupported disabled features

When JPEG-LS support is disabled the decoder would feed the
data to the JPEG Lossless decode_*_scan function resulting in
faulty decoding.

CC: libav-stable@libav.org

vf_pad: use the name 's' for the pointer to the private context

This is shorter and consistent across filters.

vf_overlay: use the name 's' for the pointer to the private context

This is shorter and consistent across filters.

vf_lut: use the name 's' for the pointer to the private context

This is shorter and consistent across filters.

vf_libopencv: use the name 's' for the pointer to the private context

This is shorter and consistent across filters.

vf_hqdn3d: use the name 's' for the pointer to the private context

This is shorter and consistent across filters.

vf_hflip: use the name 's' for the pointer to the private context

This is shorter and consistent across filters.

vf_gradfun: use the name 's' for the pointer to the private context

This is shorter and consistent across filters.

vf_frei0r: use the name 's' for the pointer to the private context

This is shorter and consistent across filters.

vf_format: use the name 's' for the pointer to the private context

This is shorter and consistent across filters.

vf_fieldorder: use the name 's' for the pointer to the private context

This is shorter and consistent across filters.

vf_fieldorder: cosmetics, remove an extra linebreak

vf_fade: use the name 's' for the pointer to the private context

This is shorter and consistent across filters.

vf_drawtext: use the name 's' for the pointer to the private context

This is shorter and consistent across filters.

vf_drawbox: use the name 's' for the pointer to the private context

This is shorter and consistent across filters.

vf_delogo: use the name 's' for the pointer to the private context

This is shorter and consistent across filters.

vf_cropdetect: use the name 's' for the pointer to the private context

This is shorter and consistent across filters.

vf_crop: cosmetics, break lines

vf_crop: use the name 's' for the pointer to the private context

This is shorter and consistent across filters.

vf_boxblur: use the name 's' for the pointer to the private context

This is shorter and consistent across filters.

vf_blackframe: use the name 's' for the pointer to the private context

This is shorter and consistent across filters.

vf_aspect: use the name 's' for the pointer to the private context

This is shorter and consistent across filters.

Remove commented-out debug #define cruft

jpegls: K&R formatting cosmetics

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

jpegls: cosmetics: Drop some unnecessary parentheses

mpegvideo: Remove commented-out PARANOID debug cruft

Remove commented-out #includes

h263dec: Remove broken and disabled debug cruft

vc1: Reindent INIT_LUT(), align backslashes

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

configure: Use linker hardening flags on mingw

This makes it consistent with the msvc builds which automatically set
the DEP and ASLR flags by default.  There really is no good reason why
they shouldn't be set.

The fact that binutils does not set them on by default boggles the mind.

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

vc1: Make INIT_LUT() self contained

Move the local variables it uses into the macro, enclosing them
in a do {} while (0) scope.

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

vc1: Simplify code by using INIT_LUT()

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

vc1: Move INIT_LUT() further up to allow using it in more places

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

vc1dec: Remove some now unused variables

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

vc1dec: Do not allow field_mode to change after the first header

This fixes out of array accesses.

Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
Signed-off-by: Martin Storsjö <martin@martin.st>

vc1, mss2: Check for any negative return value from ff_vc1_parse_frame_header

This is required if we return other error codes than explicitly
-1, which so far has been the only other possible return value
besides 0.

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

vc1dec: Fix current ptr selection in vc1_mc_4mv_chroma()

No sample tried shows a difference.

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

vc1dec: Factorize picture pointer selection in vc1_mc_4mv_chroma()

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

vc1dec: Factorize picture pointer selection code

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

vc1dec: Factorize srcU/V offseting

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

vc1dec: Fix tff == 0 handling in init_block_index()

This fixes several files from VLC ticket 5887.

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

vc1dec: Update destination pointers in context for fields

This replaces a large number of checks for the second field by
fixing the pointers when they are setup.

This should also fix I/BI field pictures.

Changes checksums for vc1_sa10143, the file becomes slightly closer
to what the reference decoder outputs.

Based on "vc1dec: the second field is written wrong to the picture"
by Sebastian Sandberg <sebastiand.sandberg@gmail.com>.

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

pixdesc: rename PIX_FMT_* flags to AV_PIX_FMT_FLAG_*

jpeg2000dec: don't use deprecated PIX_FMT values

lavfi: fix compatibility code for old vf_scale options syntax

Currently it would incorrectly trigger on a string that contains a '='
but does not contain a ':', e.g. flags=<flags>.

avfiltergraph: simplify inserting conversion filters.

There is now no need to explicitly pass 0:0 as width/height to scale,
those are the defaults.

Clarify output of av_get_bits_per_pixel

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

fate: update the mmf reference

Using Optional Data chunk introduced changes the hash.

mmf: Write metadata into an Optional Data chunk

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