AVFilterLink.frame_count is supposed to count the number of frames
that were passed on the link, but with min_samples, that number is
not always the same for the source and destination filters.
With the addition of a FIFO on the link, the difference will become
more significant.

Split the variable in two: frame_count_in counts the number of
frames that entered the link, frame_count_out counts the number
of frames that were sent to the destination filter.

The test is not supposed to cover audio.
Also, using -vframes along with an audio stream depends on
the exact order the frames are processed by filters, it is
too much constraint to guarantee.

This should fix build with Solaris CC.
Reviewed-by: NMichael Niedermayer <michael@niedermayer.cc>
Signed-off-by: NMarton Balint <cus@passwd.hu>

Reviewed-by: NJan Sebechlebsky <sebechlebskyjan@gmail.com>
Signed-off-by: NMarton Balint <cus@passwd.hu>

Reviewed-by: NMichael Niedermayer <michael@niedermayer.cc>
Signed-off-by: NMarton Balint <cus@passwd.hu>

Fixes valgrind warning about "Conditional jump or move depends on uninitialised value(s)"
Reviewed-by: NAndreas Cadhalpun <andreas.cadhalpun@googlemail.com>
Signed-off-by: NJames Almer <jamrial@gmail.com>

No need to run the whole fate-lavf set. fate-lavf-flv_fmt is enough to
create the required source file.
Signed-off-by: NJames Almer <jamrial@gmail.com>

Fixes https://trac.ffmpeg.org/ticket/5940#ticketReviewed-by: NMichael Niedermayer <&lt;michael@niedermayer.cc&gt;>
Signed-off-by: NRobert Nagy <ronag89@gmail.com>
Signed-off-by: Steven Liu <lingjiujianke@gmail.com>

Fixes: fuzz-2-ffmpeg_SUBTITLE_AV_CODEC_ID_SUBRIP_fuzzer

Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/targets/ffmpegSigned-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Otherwise put_bits can be called with a value that doesn't fit in the
sample_len, causing an assertion failure.
Reviewed-by: NMichael Niedermayer <michael@niedermayer.cc>
Signed-off-by: NAndreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>

From 'man ppm': The maximum color value (Maxval), again in ASCII decimal.
                Must be less than 65536.
Reviewed-by: NMichael Niedermayer <michael@niedermayer.cc>
Signed-off-by: NAndreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>

This fixes a heap-buffer-overflow detected by AddressSanitizer.
Reviewed-by: NMichael Niedermayer <michael@niedermayer.cc>
Signed-off-by: NAndreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>

Also contains the following changes to the library:
- add ff_ prefix to functions
- remove cplusplus defines.
- add FF_ prefix to contants and some structs
- remove true peak calculation feature, since it uses its own resampler, and
  af_loudnorm does not need it.
- remove version info and some fprintf(stderr) functions
- convert to use av_malloc
- always use histogram mode for LRA calculation, otherwise LRA data is slowly
  consuming memory making af_loudnorm unfit for 24/7 operation. It also uses a
  BSD style linked list implementation which is probably not available on all
  platforms. So let's just remove the classic mode which not uses histogram.
- add ff_thread_once for calculating static histogram tables
- convert some functions to void which cannot fail
- remove intrinsics and some unused headers
- add support for planar audio
- remove channel / sample rate changer function, in ffmpeg usually we simply
  alloc a new context
- convert some static variables to defines
- declare static histogram variables as aligned
- convert some initalizations to mallocz
- add window size parameter to init function and remove window size setter
  function
- convert return codes to AVERROR
- fix indentation
Signed-off-by: NMarton Balint <cus@passwd.hu>

Based on a patch by Kyle Swanson <k@ylo.ph>.
Signed-off-by: NMarton Balint <cus@passwd.hu>

Signed-off-by: NLou Logan <lou@lrcd.com>

Documents options and behaviour, noting when 'chunks' option will
not be honoured.
Signed-off-by: NTom Butterworth <bangnoise@gmail.com>
Signed-off-by: NMartin Vignali <martin.vignali@gmail.com>

It can read less than the requested amount, in which case buf contains
uninitialized data, causing problems like segmentation faults later on.

Also make sure that image->size is positive, so that it can't match a
negative error code.
Reviewed-by: NMichael Niedermayer <michael@niedermayer.cc>
Signed-off-by: NAndreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>

This problem was introduced in commit
4b90dcb8.
Reviewed-by: NMichael Niedermayer <michael@niedermayer.cc>
Signed-off-by: NAndreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>

The implicit checks via v_data_size and a_data_size don't work in the case
'(hdr_size > 7) && !ctx->alpha_info'.

This fixes segmentation faults due to invalid reads.

This problem was introduced in commit
547c2f00.
Reviewed-by: NMichael Niedermayer <michael@niedermayer.cc>
Signed-off-by: NAndreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>

Signed-off-by: NSasi Inguva <isasi@google.com>
Reviewed-by: NDerek Buitenhuis <derek.buitenhuis@gmail.com>
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Signed-off-by: NSasi Inguva <isasi@google.com>
Reviewed-by: NDerek Buitenhuis <derek.buitenhuis@gmail.com>
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

The secondary compression in Hap is optional, this change exposes that option to
the user as some use-cases favour higher bitrate files to reduce workload
decoding.
Adds "none" or "snappy" as options for "compressor". Selecting "none" disregards
"chunks" option: chunking is only of benefit decompressing Snappy.
Reviewed-by: NMartin Vignali <martin.vignali@gmail.com>
Signed-off-by: NTom Butterworth <bangnoise@gmail.com>

Fixes part of ticket #5544.

Add keyframe index metadata
Used to facilitate seeking; particularly for HTTP pseudo streaming.
 1. read live streaming or file by sequence
 2. if use add_keyframe_index option, add a mark flag at the position,
    use to insert new context at the last step.
 3. add the keyframes *offset* and *timestamp* into a list
 4. if use add_keyframe_index option, shift the metadata data from
    mark flag offset
 5. insert the keyframes *offset* and *timestamp* from the list by
    sequence
 6. free the list
 7. end.

Add FATE test case;
Reviewed-by: NLou Logan <lou@lrcd.com>
Signed-off-by: NSteven Liu <liuqi@gosun.com>
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Found-by: Luca Barbato
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

It causes a cb_depth of 32, leading to assertion failures in get_bits.
Reviewed-by: NMichael Niedermayer <michael@niedermayer.cc>
Signed-off-by: NAndreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>

Reviewed-by: NMichael Niedermayer <michael@niedermayer.cc>
Signed-off-by: NAndreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>

Reviewed-by: NMichael Niedermayer <michael@niedermayer.cc>
Signed-off-by: NAndreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>

Signed-off-by: NSteven Liu <liuqi@gosun.com>
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Reviewed-by: NMichael Niedermayer <michael@niedermayer.cc>
Signed-off-by: NAndreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>

The code assumes that s->streams[0] is valid.
Reviewed-by: NMichael Niedermayer <michael@niedermayer.cc>
Signed-off-by: NAndreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>

This allows a subsequent change to compress directly into the output packet when possible.
Reviewed-by: NMichael Niedermayer <michael@niedermayer.cc>
Signed-off-by: NTom Butterworth <bangnoise@gmail.com>