fix ticket: #6204
Signed-off-by: NSteven Liu <lq@chinaffmpeg.org>

Fixes ticket #6206.

Only I-frames are decoded for now.
Signed-off-by: NPaul B Mahol <onemda@gmail.com>

Public fields were added after the private fields (negating the entire
point of this). New private fields go into AVStreamInternal anyway.

The new marker was set by guessing which fields are supposed to be
private and wshich not. recommended_encoder_configuration is accessed by
ffserver_config.c directly, and is supposed to use the public API.

ffmpeg.c accesses AVStream.cur_dts, even though it's a private field,
but that seems to be an older error.

Allow all struct fields to be accessed directly, as long as they're
public.

Before this change, many fields were "public", but could be accessed via
AVOption only. This meant they were effectively not public, but were
present for documentation purposes, which was incredibly confusing at
best.

Fixes: asan_bug_leak
Found-by: NThomas Guilbert <tguilbert@google.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>

Regression since 2c8d876d

MSVC doesn't support the %s time format, and instead of returning an
error the invalid parameter handler is invoked which (by default)
terminates the process.

Reviewed-by：Steven Liu <lq@chinaffmpeg.org>
Signed-off-by: NHendrik Leppkes <h.leppkes@gmail.com>

refer to ticket id: #6170

rename file from temp to origin name after complete current segment
Reviewed-by: NAman Gupta <ffmpeg@tmm1.net>
Signed-off-by: NSteven Liu <lq@chinaffmpeg.org>

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

Value 4 (Unknown) is for the time being part of the Matroska spec but not
supported by WebM

Addresses ticket #6176

Signed-off-by: NNicolas Roy-Renaud <nicolas.roy-renaud.1@ens.etsmtl.ca>
Reviewed-by: NPaul B Mahol <onemda@gmail.com>
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

check if proto is null before av_strcasecmp
CID:  1401346
Signed-off-by: NSteven Liu <lq@chinaffmpeg.org>

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

Silence a valgrind warning.

Fixes ticket #6160.

lavf/mov.c: Correct keyframe search in edit list to return the very first keyframe/frame with matching timestamp. Fixes ticket#5904
Signed-off-by: NSasi Inguva <isasi@google.com>
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Fixes Coverity CID 1396277.
Signed-off-by: NMarton Balint <cus@passwd.hu>

This dts value can end up in the list in the absence of durations and is in that
case semantically identical to AV_NOPTS_VALUE. We can alternatively prevent
storing RELATIVE_TS_BASE if there is no duration.

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

Fixes Ticket 6018

This fixes a regression, and allows playback of files containing mpeg4video that are otherwise
not supported
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

When the http method is not set, the method will use POST for ts,
PUT for m3u8, it is not unify, now set it unify.
This ticket id: #5315
Reviewed-by: NMichael Niedermayer <michael@niedermayer.cc>
Reviewed-by: NMoritz Barsnick <barsnick@gmx.net>
Signed-off-by: NSteven Liu <lq@chinaffmpeg.org>

Fixes ticket #6100.

Skips using temporary files when outputting to a protocol other than
"file", which enables dash to output content over network
protocols. The logic has been copied from the HLS format.
Reviewed-by: Steven Liu <lingjiujianke@gmail.com>
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

This commit optimizes HTTP performance by reducing forward seeks, instead
favoring a read-ahead and discard on the current connection (referred to
as a short seek) for seeks that are within a TCP window's worth of data.
This improves performance because with TCP flow control, a window's worth
of data will be in the local socket buffer already or in-flight from the
sender once congestion control on the sender is fully utilizing the window.

Note: this approach doesn't attempt to differentiate from a newly opened
connection which may not be fully utilizing the window due to congestion
control vs one that is. The receiver can't get at this information, so we
assume worst case; that full window is in use (we did advertise it after all)
and that data could be in-flight

The previous behavior of closing the connection, then opening a new
with a new HTTP range value results in a massive amounts of discarded
and re-sent data when large TCP windows are used.  This has been observed
on MacOS/iOS which starts with an initial window of 256KB and grows up to
1MB depending on the bandwidth-product delay.

When seeking within a window's worth of data and we close the connection,
then open a new one within the same window's worth of data, we discard
from the current offset till the end of the window.  Then on the new
connection the server ends up re-sending the previous data from new
offset till the end of old window.

Example (assumes full window utilization):

TCP window size: 64KB
Position: 32KB
Forward seek position: 40KB

      *                      (Next window)
32KB |--------------| 96KB |---------------| 160KB
        *
  40KB |---------------| 104KB

Re-sent amount: 96KB - 40KB = 56KB

For a real world test example, I have MP4 file of ~25MB, which ffplay
only reads ~16MB and performs 177 seeks. With current ffmpeg, this results
in 177 HTTP GETs and ~73MB worth of TCP data communication.  With this
patch, ffmpeg issues 4 HTTP GETs and 3 seeks for a total of ~22MB of TCP data
communication.

To support this feature, the short seek logic in avio_seek() has been
extended to call a function to get the short seek threshold value.  This
callback has been plumbed to the URLProtocol structure, which now has
infrastructure in HTTP and TCP to get the underlying receiver window size
via SO_RCVBUF.  If the underlying URL and protocol don't support returning
a short seek threshold, the default s->short_seek_threshold is used

This feature has been tested on Windows 7 and MacOS/iOS.  Windows support
is slightly complicated by the fact that when TCP window auto-tuning is
enabled, SO_RCVBUF doesn't report the real window size, but it does if
SO_RCVBUF was manually set (disabling auto-tuning). So we can only use
this optimization on Windows in the later case
Signed-off-by: NJoel Cunningham <joel.cunningham@me.com>
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Reported-by: NSleepProgger <security@gnutp.com>
Reviewed-by: Steven Liu <lingjiujianke@gmail.com>
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

hls-encoder currenlty does not provide stream level metadata to mpegts
muxer. This patch fixes track #3848 bug.
Signed-off-by: NBela Bodecs <bodecsb@vivanet.hu>
Signed-off-by: NSteven Liu <lq@chinaffmpeg.org>

This enables having multiple tracks of the same type which would
be treated as different things by the media server (as opposed to
different bit rate versions of the same track). According to the
smooth streaming specification, just setting the systemLanguage
tag is not enough to note that a track with the same attributes
differs from another one.

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

Silences an "assignment discards ‘const’ qualifier" warning.

Reviewed-by: Marton Balint

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

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

Silences an "assignment discards ‘const’ qualifier" warning.

Only lossy part is decoded for now.
Signed-off-by: NPaul B Mahol <onemda@gmail.com>

When user use the hls_wrap, there have many problem:
1. some platform refersh the old but usefull segment
2. CDN(Content Delivery Network) Deliver HLS not friendly

The hls_wrap is used to wrap segments for use little space,
now user can use hls_list_size and hls_flags delete_segments
instead it.
Reviewed-by: NMichael Niedermayer <michael@niedermayer.cc>
Reviewed-by: NCarl Eugen Hoyos <ceffmpeg@gmail.com>
Signed-off-by: NSteven Liu <lq@chinaffmpeg.org>

In case parsers care about the version that started writing
correct flags.
Signed-off-by: NJames Almer <jamrial@gmail.com>

This way it's clear the size field accounts for the footer length plus every
tag entry, but not the header.
Reviewed-by: NPaul B Mahol <onemda@gmail.com>
Signed-off-by: NJames Almer <jamrial@gmail.com>

The size field in the header/footer accounts for the entire APE tag
structure except the 32 bytes from header, for compatibility with
APEv1.
Reviewed-by: NPaul B Mahol <onemda@gmail.com>
Signed-off-by: NJames Almer <jamrial@gmail.com>