tiff: fix handling of metadata with refcounted frames

Since the conversion to refcounted frames, the tiff decoder
only wrote the metadata into its internal "picture" in its own context,
never exposing the metadata to the user, and eventually leaking the
metadata.

Instead, properly store the metadata directly into the frame being decoded into.

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>

yadif: restore speed of the C filtering code

Always use the special filter for the first and last 3 columns (only).

Changes made in 64ed397 slowed the filter to just under 3/4 of what it
was.  This commit restores the speed while maintaining identical output.

For reference, on my Athlon64:
1733222 decicycles in old
2358563 decicycles in new
1727558 decicycles in this

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>

lavfi/cropdetect: add support for named options

lavu/frame: free frame metadata when unrefing a frame.

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>

avcodec: remove AVCodecContext->metadata

This field was only ever set and freed from avcodec, and not otherwise
used. However, because frames are refcounted now, avcodec cannot make any
assumptions about the lifetime of the frame metadata, which can result in
double-frees or leaked memory.

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>

lavfi/delogo: remove sscanf and rely on av_opt_set_from_string() only.

lavc/crystalhd: remove now unecessary buffer_hints.

lavc: factorize ff_{thread_,re,}get_buffer error messages.

Coccinelle profile used:

  @@
  expression r, ctx, f, loglevel, str, flags;
  @@

  -if ((r = ff_get_buffer(ctx, f, flags)) < 0) {
  -    av_log(ctx, loglevel, str);
  -    return r;
  -}
  +if ((r = ff_get_buffer(ctx, f, flags)) < 0)
  +    return r;

  @@
  expression r, ctx, f, loglevel, str;
  @@

  -if ((r = ff_reget_buffer(ctx, f)) < 0) {
  -    av_log(ctx, loglevel, str);
  -    return r;
  -}
  +if ((r = ff_reget_buffer(ctx, f)) < 0)
  +    return r;

  @@
  expression r, ctx, f, loglevel, str, flags;
  @@

  -if ((r = ff_thread_get_buffer(ctx, f, flags)) < 0) {
  -    av_log(ctx, loglevel, str);
  -    return r;
  -}
  +if ((r = ff_thread_get_buffer(ctx, f, flags)) < 0)
  +    return r;

...along with some manual patches for the remaining ones.

lavfi/thumbnail: remove unecessary poll_frame() callback.

lavfi/mptestsrc: raise filter_frame() error.

lavfi/mandelbrot: raise filter_frame() error.

lavfi/life: raise filter_frame() error.

lavfi/cellauto: raise filter_frame() error.

lavfi/alphamerge: raise filter_frame() error.

lavfi/movie: raise filter_frame() error.

lavfi/buffersrc: raise filter_frame() error.

lavfi/showspectrum: raise filter_frame() error.

lavfi/avfilter: raise filter_frame() error.

lavfi/concat: raise filter_frame() error.

lavfi/anullsrc: raise filter_frame() error.

lavfi/aevalsrc: raise filter_frame() error.

lavfi/atempo: raise filter_frame() error.

lavfi/asetnsamples: raise filter_frame() error.

lavfi/aresample: raise filter_frame() error.

lavu/frame: av_frame_make_writable: set the channels on the new frame.

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>

lavu/frame: copy all frame properties in av_frame_copy_props

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>

dsputil: remove deprecated dsp_mask usage

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>

dsputil_mmx: remove unused variables

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>

exr: silence warning "libavcodec/exr.c:351:9: warning: variable ret set but not used"

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>

Merge remote-tracking branch 'qatar/master'

* qatar/master:
  hqdn3d: Fix out of array read in LOWPASS
  cabac: remove unused argument of ff_init_cabac_states()
  rawdec: fix a typo -- || instead of |

Conflicts:
libavcodec/cabac.c
libavcodec/h264.c
libavfilter/vf_hqdn3d.c

Merged-by: Michael Niedermayer <michaelni@gmx.at>

Merge commit '37cb3b180a1dc3d6f123f68e0806585ebc2578b6'

* commit '37cb3b180a1dc3d6f123f68e0806585ebc2578b6':
  matroskadec: request a read buffer for the wav header

See: d0b450457b3aeb8c234b0b0a987db55d3485326b

Merged-by: Michael Niedermayer <michaelni@gmx.at>

vc1: fix int/ptrdiff_t mismatches

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>

Merge commit 'a8b6015823e628047a45916404c00044c5e80415'

* commit 'a8b6015823e628047a45916404c00044c5e80415':
  dsputil: convert remaining functions to use ptrdiff_t strides

Conflicts:
libavcodec/dsputil.h
libavcodec/dsputil_template.c
libavcodec/h264qpel_template.c

Merged-by: Michael Niedermayer <michaelni@gmx.at>

Merge commit 'a4472ac01e86f9fae5adb9034f2777b86a9c5480'

* commit 'a4472ac01e86f9fae5adb9034f2777b86a9c5480':
  Add informative messages to av_log_ask_for_sample calls lacking them
  anm: Get rid of some very silly goto statements

Conflicts:
libavformat/anm.c

Merged-by: Michael Niedermayer <michaelni@gmx.at>

Merge commit '202c2acc40a6de8758b44ab3f5c85ab250079734'

* commit '202c2acc40a6de8758b44ab3f5c85ab250079734':
  vda: remove async decoder leftovers

Conflicts:
libavcodec/vda_h264.c

Merged-by: Michael Niedermayer <michaelni@gmx.at>

Merge commit 'a2816230c5c0a8fc72bc0163b7d21a96b194d87a'

* commit 'a2816230c5c0a8fc72bc0163b7d21a96b194d87a':
  avframe: call release_buffer only if it is set

Conflicts:
libavcodec/utils.c

Merged-by: Michael Niedermayer <michaelni@gmx.at>

Merge commit 'fce68c93554803801c32c1b20509bfa8d496b02a'

* commit 'fce68c93554803801c32c1b20509bfa8d496b02a':
  pthread: unref the decoded but not returned frames on close.

Merged-by: Michael Niedermayer <michaelni@gmx.at>

Merge commit '555000c7d5c1e13043a948ebc48d2939b0ba6536'

* commit '555000c7d5c1e13043a948ebc48d2939b0ba6536':
  h264: check that DPB is allocated before accessing it in flush_dpb()
  vf_hqdn3d: fix uninitialized variable use
  vf_gradfun: fix uninitialized variable use

Merged-by: Michael Niedermayer <michaelni@gmx.at>

Merge commit 'c2597c5a0a08526493abb9bc9544b61a290fe20f'

* commit 'c2597c5a0a08526493abb9bc9544b61a290fe20f':
  h264_refs: cosmetics, reformat

Conflicts:
libavcodec/h264_refs.c

Merged-by: Michael Niedermayer <michaelni@gmx.at>

Merge commit 'f08fefc4d099f2a1f2e3a6db3d340537e601f762'

* commit 'f08fefc4d099f2a1f2e3a6db3d340537e601f762':
  h264: remove a useless svq3 condition
  mpegvideo: remove FMT_H264

Merged-by: Michael Niedermayer <michaelni@gmx.at>

Merge remote-tracking branch 'cehoyos/master'

* cehoyos/master:
  avframe: Copy buffer type in copy_props.

Merged-by: Michael Niedermayer <michaelni@gmx.at>

Merge commit 'ee8704916de79158da8a48a9ea5be819e83d23ba'

* commit 'ee8704916de79158da8a48a9ea5be819e83d23ba':
  mpegvideo: reindent

Conflicts:
libavcodec/mpegvideo.c

Merged-by: Michael Niedermayer <michaelni@gmx.at>

avframe: Copy buffer type in copy_props.

Fixes VDPAU decoding with MPlayer.

Merge commit '19cac8e301419dcde31526d15196a952ddf2c4c7'

* commit '19cac8e301419dcde31526d15196a952ddf2c4c7':
  mpegvideo: remove useless references to h264 and svq3

Conflicts:
libavcodec/mpegvideo.c

Merged-by: Michael Niedermayer <michaelni@gmx.at>

Merge commit '85a5bc054c1290699ccbf5799ba6c4e2fbcc3530'

* commit '85a5bc054c1290699ccbf5799ba6c4e2fbcc3530':
  lavf: remove disabled FF_API_R_FRAME_RATE cruft

Conflicts:
libavformat/avformat.h
libavformat/electronicarts.c
libavformat/rawdec.c
libavformat/utils.c
libavformat/version.h

The field is in use and no semantically equivalent field is available,
thus not removed

Merged-by: Michael Niedermayer <michaelni@gmx.at>

avformat: postpone API removials where the functions are trivial wrapers

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>

Merge commit '7b486ab13bfcfa88a7cd92586de50e49966ec292'

* commit '7b486ab13bfcfa88a7cd92586de50e49966ec292':
  lavf: remove disabled FF_API_AV_GETTIME cruft
  lavf: remove disabled FF_API_INTERLEAVE_PACKET cruft

Conflicts:
libavformat/version.h

Merged-by: Michael Niedermayer <michaelni@gmx.at>

Merge commit '435c2a31ad5eead20eda1152097f60c3bfa22847'

* commit '435c2a31ad5eead20eda1152097f60c3bfa22847':
  lavf: remove disabled FF_API_READ_PACKET cruft

Conflicts:
libavformat/version.h

Removial postponed as wraper is trivial

Merged-by: Michael Niedermayer <michaelni@gmx.at>

Merge commit 'c7e044c61bb08b3a6e1e8063e8f4449c88b01201'

* commit 'c7e044c61bb08b3a6e1e8063e8f4449c88b01201':
  lavf: remove disabled FF_API_APPLEHTTP_PROTO cruft

Conflicts:
libavformat/allformats.c
libavformat/version.h

Merged-by: Michael Niedermayer <michaelni@gmx.at>

lavc: remove disabled 8SVX_RAW cruft

Signed-off-by: Paul B Mahol <onemda@gmail.com>

Merge commit '0a7c4daf469d4ac447fb822fe78337f62f4c04e6'

* commit '0a7c4daf469d4ac447fb822fe78337f62f4c04e6':
  lavf: remove disabled FF_API_CLOSE_INPUT_FILE cruft

Conflicts:
libavformat/version.h

removial postponed as the wraper code is trivial

Merged-by: Michael Niedermayer <michaelni@gmx.at>

Merge commit 'd6ed604cf4ca86ed345125b26eb5bdeea6d6bf68'

* commit 'd6ed604cf4ca86ed345125b26eb5bdeea6d6bf68':
  lavc: remove disabled FF_API_IDCT cruft

Conflicts:
libavcodec/version.h

Merged-by: Michael Niedermayer <michaelni@gmx.at>

Merge commit '3bcdf8dcb9891ffe49b6398d0e2c02f1712d8f00'

* commit '3bcdf8dcb9891ffe49b6398d0e2c02f1712d8f00':
  lavc: remove disabled FF_API_SNOW cruft

Conflicts:
libavcodec/avcodec.h
libavcodec/options_table.h
libavcodec/version.h

Merged-by: Michael Niedermayer <michaelni@gmx.at>

bktr: add missing #include for av_gettime()

Signed-off-by: Paul B Mahol <onemda@gmail.com>

Merge commit 'fcb07e8b332bbd6f9558bff98ff5102c5f2d8252'

* commit 'fcb07e8b332bbd6f9558bff98ff5102c5f2d8252':
  lavc: remove disabled FF_API_MMI cruft

Conflicts:
libavcodec/version.h

Merged-by: Michael Niedermayer <michaelni@gmx.at>

Revert "Merge commit '0517c9e098092709397cc522c59fa63c83cc81be'" bring the old audio resampling API back

This reverts commit d3edc65dd1e5b5d4246fcb8bcd216eb558bab7d4, reversing
changes made to 150de78d7c9cee65b4095832b25ae353e0d7c7af.

Conflicts:

libavcodec/version.h

It seems there are several applications still using it

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>

vc1dec: remove unused variables

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>

CrystalHD: Port to ref-counted frame API.

Signed-off-by: Philip Langdale <philipl@overt.org>
Signed-off-by: Michael Niedermayer <michaelni@gmx.at>

hqdn3d: Fix out of array read in LOWPASS

CC:libav-stable@libav.org
Signed-off-by: Anton Khirnov <anton@khirnov.net>

cabac: remove unused argument of ff_init_cabac_states()

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

rawdec: fix a typo -- || instead of |

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

opt: re-enable memleak fix for duplicated options.

The memleak fix in a1bcc76e was causing problems when the original
string wasn't NULL or av_malloc'ed, thus was disabled. It seems the only
OPT_STRING occurence where it needed to be fixed is in ffserver. This
commit fixes that specific problem and so re-enable the original code.

vp56: remove unused variable

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>

dsputil: remove hpel functions (moved to hpeldsp).

Move alpha half-pel assembly from dsputil to hpeldsp.

Move sparc/vis half-pel assembly from dsputil to hpeldsp.

Move sh4 half-pel assembly from dsputil to hpeldsp.

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>

Move bfin half-pel assembly from dsputil to hpeldsp.

Move arm half-pel assembly from dsputil to hpeldsp.

Move ppc half-pel assembly from dsputil to hpeldsp.

Move x86 half-pel assembly from dsputil to hpeldsp.

mjpeg: use hpeldsp instead of dsputil for half-pel functions.

svq1enc: use hpeldsp instead of dsputil for half-pel functions.

snow: use hpeldsp instead of dsputil for half-pel functions.

svq3: use hpeldsp instead of dsputil for half-pel functions.

mpegvideo: use hpeldsp instead of dsputil for half-pel functions.

This also converts vc1, since that is mpegvideo-based.

svq1: use hpeldsp instead of dsputil for half-pel functions.

This makes svq1 independent of dsputil.

mimic: use hpeldsp instead of dsputil for half-pel functions.

interplayvideo: use hpeldsp instead of dsputil for half-pel functions.

This makes interplayvideo independent of dsputil.

bink: use hpeldsp instead of dsputil for half-pel functions.

indeo3: use hpeldsp instead of dsputil for half-pel functions.

This makes indeo3 independent of dsputil.

vp56: use hpeldsp instead of dsputil for half-pel functions.

This makes vp5 and vp6 independent of dsputil.

vp3: use hpeldsp instead of dsputil for half-pel functions.

This makes vp3 independent of dsputil.

hpeldsp: add half-pel functions (currently copies of dsputil).

Merge commit 'de27d2b92fa97deb2856d18e9f5f19586ce45a0f'

* commit 'de27d2b92fa97deb2856d18e9f5f19586ce45a0f':
  lavc: remove disabled FF_API_LIBMPEG2 cruft

Conflicts:
libavcodec/avcodec.h
libavcodec/version.h

Merged-by: Michael Niedermayer <michaelni@gmx.at>

Merge commit '0517c9e098092709397cc522c59fa63c83cc81be'

* commit '0517c9e098092709397cc522c59fa63c83cc81be':
  lavc: remove disabled FF_API_AVCODEC_RESAMPLE cruft

Conflicts:
libavcodec/Makefile
libavcodec/avcodec.h
libavcodec/resample.c
libavcodec/resample2.c
libavcodec/version.h

Merged-by: Michael Niedermayer <michaelni@gmx.at>

Merge commit 'd6d369bf1370999896500ae7eb5b9447ab635a3d'

* commit 'd6d369bf1370999896500ae7eb5b9447ab635a3d':
  atomic: prefer gcc builtins over win32 atomics, if available.
  avframe: copy reordered_opaque in copy_props

Conflicts:
libavutil/frame.c

Merged-by: Michael Niedermayer <michaelni@gmx.at>

Merge commit '684e3d2e1ce96625eeef63f2564aab66f6715d05'

* commit '684e3d2e1ce96625eeef63f2564aab66f6715d05':
  ra144: check buffer size before requesting a buffer
  wmapro: unref skipped frames

Merged-by: Michael Niedermayer <michaelni@gmx.at>

Merge commit 'e3232f34312f8187094c875445683277ed0c209d'

* commit 'e3232f34312f8187094c875445683277ed0c209d':
  svq1: use av_frame_free to free refcounted frame
  lavc: fix get_buffer() compatibility layer for audio.

Merged-by: Michael Niedermayer <michaelni@gmx.at>

Merge commit '98cec5c84feff34e04428de4a86836a83657ae5e'

* commit '98cec5c84feff34e04428de4a86836a83657ae5e':
  ratecontrol: remove an unused variable
  vorbisdec: do not leak the first frame.
  vf_pad: fix a & instead of && typo

Conflicts:
libavfilter/vf_pad.c

Merged-by: Michael Niedermayer <michaelni@gmx.at>

avpacket: do not copy data when buf ref is available.

This at least fixes issues with lavf/subtitles. The behaviour of
av_dup_packet() is unchanged, only av_copy_packet() is affected.

Merge commit '2eaa3663fda750dac66d41fe8541a8744d5563a4'

* commit '2eaa3663fda750dac66d41fe8541a8744d5563a4':
  avplay: enable only when SDL 1.2 is found

Conflicts:
configure

Merged-by: Michael Niedermayer <michaelni@gmx.at>

Merge commit 'adfa53d67c7a3318157ea9d95e8bdcfb77139452'

* commit 'adfa53d67c7a3318157ea9d95e8bdcfb77139452':
  lavc: remove disabled FF_API_VDA_ASYNC cruft

Conflicts:
libavcodec/vda.h
libavcodec/vda_h264.c
libavcodec/version.h

Merged-by: Michael Niedermayer <michaelni@gmx.at>

avcodec: assert that old codec ids match new

Signed-off-by: Michael Niedermayer <michaelni@gmx.at>

Merge commit 'bdd1567c355a8092e7746ef99e831d579e34fa6a'

* commit 'bdd1567c355a8092e7746ef99e831d579e34fa6a':
  lavc: remove disabled FF_API_CODEC_ID cruft

Conflicts:
libavcodec/old_codec_ids.h
libavcodec/version.h

Only the #define is removed
The CODEC_IDs itself are left until theres some reason to remove
them.

Merged-by: Michael Niedermayer <michaelni@gmx.at>

lavfi/fieldorder: add support to named options

lavfi/fieldorder: remove unused headers and commented define

lavc: extend documentation for skip_idct, skip_loop_filter, skip_frame options

ffprobe: support codec options

Make ffprobe honour codec options, and support stream specifiers like it
is done in ffplay.

In particular, address Trac ticket #1859.

cmdutils: improve feedback in case of not found option