in get_default_pattern_localtime_fmt the default pattern contains
%Y%m%d%H%I%S but the original intention was %Y%m%d%H%M%S
Signed-off-by: NBela Bodecs <bodecsb@vivanet.hu>
Signed-off-by: NSteven Liu <lq@chinaffmpeg.org>

fix CID: 1398364 Resource leak
refine the code of the new options
Reviewed-by: NBodecs Bela <bodecsb@vivanet.hu>
Signed-off-by: NSteven Liu <lq@chinaffmpeg.org>

A wrong, unitialized variable is used for testing. This patch fixes this
typo.
Signed-off-by: NBela Bodecs <bodecsb@vivanet.hu>
Signed-off-by: NSteven Liu <lq@chinaffmpeg.org>

When the mapping was originally added AV_CODEC_ID_MP1 did not exist.

1st:
This patch makes it possible to put actual segment file size (measured
in bytes) and/or duration (calculated in microseconds) into segment
filenames. This feature is useful when post-processing live streaming
access log files. New behaviour works only when -use_localtime option
is set and second_level_segment_size or/and
second_level_segment_duration new hls_flags are specified. %%s is the
placeholder for size and %%t for duration in hls_segment_filename
option. Fix sized trailing zeropadding also works eg. %%09s or %%023t.

A command to test new features:
./ffmpeg -loglevel info -y -f lavfi -i color=c=red:size=640x480:r=25 -f
lavfi -i sine=f=440:b=4:r=44100 -c:v mpeg2video -g 25 -acodec aac
-cutoff 20000 -ac 2 -ar 44100 -ab 192k -f hls -hls_time 3 -hls_list_size
5 -hls_flags
second_level_segment_index+second_level_segment_size+second_level_segment_duration
-use_localtime 1 -use_localtime_mkdir 1 -hls_segment_filename
"segment_%Y%m%d%H%M%S_%%04d_%%08s_%%013t.ts" stream.m3u8

2nd:
doc/muxers: beside second_level_segment_duration and second_level_segment_size,
added some more details and example to hls_segment_filename,
use_localtime, use_localtime_mkdir, hls_flags. hls_flags option list
reformatted to table
Signed-off-by: NBela Bodecs <bodecsb@vivanet.hu>
Signed-off-by: NSteven Liu <lq@chinaffmpeg.org>

Signed-off-by: NTobias Rapp <t.rapp@noa-archive.com>
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

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

if the http server don't response the http command,
then the thread will be blocked and never be interrupted.
Reported-by: Nyinyunjiang <yinyunjiang1991@qq.com>
Signed-off-by: NSteven Liu <lq@chinaffmpeg.org>
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Also increases the score for large jpeg files.
Fixes autodetection for the file from mpv issue 3973.

Current implementation of finding duplicate segment filenames may fail
if use_localtime_mkdir and use_localtime are in effect and
segment_filename option expression contains subdirectories with
date/time specifiers. This patch fixes this false behaviour.
Signed-off-by: NBela Bodecs <bodecsb@vivanet.hu>
Signed-off-by: NSteven Liu <lq@chinaffmpeg.org>

in hlcenc.c, in the hls_write_header() function the default format
string for strftime() function contains %s specifier when use_localtime
is true. This %s specifier will insert the seconds since EPOCH. But %s
is not available on all system/environment. This patch check %s
availabilty at runtine and alter the default format string if necessary.
Signed-off-by: NBela Bodecs <bodecsb@vivanet.hu>
Signed-off-by: NSteven Liu <lq@chinaffmpeg.org>

In ff_index_search_timestamp(), if b == num_entries,
m == num_entries - 1, and entries[m].flags & AVINDEX_DISCARD_FRAME is
true, then the search for the next non-discarded packet could access
entries[nb_entries], exceeding its bounds. This change adds a protection
against that scenario. Reference: https://crbug.com/666770Reviewed-by: NSasi Inguva <isasi@google.com>
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

When delete_segments hls_flag is specified, deleting old segments may
fail in certain cases when use_localtime_mkdir is in effect and
hls_segment_filename expression contains subdirs. This patch fixes this
behaviour.
Signed-off-by: NBela Bodecs <bodecsb@vivanet.hu>
Signed-off-by: NSteven Liu <lq@chinaffmpeg.org>

Signed-off-by: NJan Sebechlebsky <sebechlebskyjan@gmail.com>

 in filenames

Putting date/time values into segment filenames is very usefull.
But to produce non-conflicting segment filenames with -use_localtime
option with date/time
values in hls_segment_filename option, sometimes is not enough.
Like in cases when multiple segments produced in the same second.
But hlsenc currently does not make possible to use segment index (%d) at
the

same time whe use_localtime is in effect, due to identifier conflict.
This patch makes possible to use strftime identifiers and still put
segment index (%d) at same time in segment filenames by introducing
second_level_segment_index flag. When -use_localtime is active,
identifier %d is for month day index, so %%d is the segment index
placeholder. This enhanced behaviour only exists when new
second_level_segment_index flag is specified.
For instance putting 'segment_%Y%m%d%H%M%S_%%05d.ts' value into
-hls_segment_filename option and specifing -hls_flags
second_level_segment_index and -use_localtime 1, may produce segment
filename as 'segment_20161230235758_00002.ts'

An example:

ffmpeg -loglevel info -y -f lavfi -i color=c=red:size=640x480:r=25 -f
lavfi -i anullsrc=r=44100:cl=stereo -c:v mpeg2video -g 25 -acodec aac
-cutoff 20000 -ac 2 -ar 44100 -ab 192k -f hls -hls_time 3 -hls_list_size
5 -hls_flags delete_segments+second_level_segment_index -use_localtime 1
-hls_segment_filename "segment_%Y%m%d%H%M%S_%%05d.ts" stream.m3u8

will produce segments filenames:

....
segment_20161227005902_00013.ts
segment_20161227005902_00014.ts
segment_20161227005902_00015.ts
segment_20161227005903_00016.ts
segment_20161227005903_00017.ts
segment_20161227005903_00018.ts
segment_20161227005903_00019.ts
segment_20161227005903_00020.ts
....
Signed-off-by: NBela Bodecs <bodecsb@vivanet.hu>

initial_prog_date_time shouldn't be adjusted when deleting segments
from disk, but rather when segments are removed from the playlist.
Signed-off-by: NJesper Ek <deadbeef84@gmail.com>
Signed-off-by: NSteven Liu <lq@chinaffmpeg.org>

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

  ffmpeg-devel

with use_localtime parameter hlsenc may produce identical filenames for
different but still existing segments. It happens when
hls_segment_filename contains
syntacticaly correct but inadequate format parameters. Currently there
is no any log message when such a situaton occurs but these cases should
be avoided in most times. This patch generate warning log messages in
these cases.

ticketID: #6043
Signed-off-by: NBela Bodecs <bodecsb@vivanet.hu>
Signed-off-by: Steven Liu <lingjiujianke@gmail.com>

Fixes pts gaps when reading AVI files > 256GiB generated by FFmpeg.
Signed-off-by: NTobias Rapp <t.rapp@noa-archive.com>
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

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

Fixes #5602 and #5603.
Signed-off-by: NPaul B Mahol <onemda@gmail.com>

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

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

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

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

This makes it possible to decode motion jpeg 2000
encoded in a transport stream without a correct PMT/PAT.
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Support url_get_file_handle on TLS streams.
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Clang is not able to eliminate the reference to ff_spdif_probe() when
there is a goto target in the same block and optimization is disabled.

This fixes the following build failure on OS X:
  ./configure --disable-everything --disable-doc \
    --enable-decoder=pcm_s16le --enable-demuxer=wav \
    --enable-protocol=file --disable-optimizations --cc=clang
  make
  ...
  Undefined symbols for architecture x86_64:
    "_ff_spdif_probe", referenced from:
        _set_spdif in libavformat.a(wavdec.o)
  ld: symbol(s) not found for architecture x86_64
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

MPEG Audio frame header must be 4 bytes. If we fail to read
4 bytes bail early to avoid Use-of-uninitialized-value msan error.
Reference https://crbug.com/666874.
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

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>

(cherry picked from commit f912fd76)
Signed-off-by: NJames Almer <jamrial@gmail.com>

Found-by: NPaul B Mahol <onemda@gmail.com>
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

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

The bit_rate field has type int64_t since commit
7404f3bd.
Reviewed-by: NPaul B Mahol <onemda@gmail.com>
Signed-off-by: NAndreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>

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

Fixes CVE-2016-9561, Note the security relevance of this is disputed as
running out of memory can happen with valid files
Suggested-by: NAndreas Cadhalpun <andreas.cadhalpun@googlemail.com>
Reviewed-by: NAndreas Cadhalpun <andreas.cadhalpun@googlemail.com>
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Fixes ticket #5982.

Fixes ticket #6014.