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

An AVIO offset is int64_t.

Bug-Id: 921

This fixes infinite loops due to seeking back.
Signed-off-by: NAlexandra Hájková <alexandra@khirnov.net>
Signed-off-by: NLuca Barbato <lu_zero@gentoo.org>

This fixes infinite loops due to seeking back.
Signed-off-by: NAlexandra Hájková <alexandra@khirnov.net>
Signed-off-by: NLuca Barbato <lu_zero@gentoo.org>

The loop can be very long, even though the file is very short.
Signed-off-by: NAndreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>
Signed-off-by: NAlexandra Hájková <alexandra@khirnov.net>
Signed-off-by: NLuca Barbato <lu_zero@gentoo.org>

asf_read_payload can unset eof_reached, so check it also before calling
that function.

This fixes infinite loops.
Signed-off-by: NAndreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>
Signed-off-by: NAlexandra Hájková <alexandra@khirnov.net>
Signed-off-by: NLuca Barbato <lu_zero@gentoo.org>

Now all IO should go through the IO callbacks and be interceptable by
the caller.

Some (de)muxers open additional files beyond the main IO context.
Currently, they call avio_open() directly, which prevents the caller
from using custom IO for such streams.

This commit adds callbacks to AVFormatContext that default to
avio_open2()/avio_close(), but can be overridden by the caller. All
muxers and demuxers using AVIO are switched to using those callbacks
instead of calling avio_open()/avio_close() directly.

(de)muxers that use the URLProtocol layer directly instead of AVIO
remain unconverted for now. This should be fixed in later commits.

This feature is mostly only used by NLE software, and is
both of dubious value being enabled by default, and a
possible security risk.
Signed-off-by: NDerek Buitenhuis <derek.buitenhuis@gmail.com>
Signed-off-by: NLuca Barbato <lu_zero@gentoo.org>

For http, this avoids spurious warnings about failed requests (e.g.
HTTP error 416 Requested Range Not Satisfiable), if the last packet
is truncated and the size read is bogus.
Signed-off-by: NMartin Storsjö <martin@martin.st>

When loading a truncated flv file, it would previously try to do a seek to
the end of every packet read. For some input protocols (such as http), such
repeated seek attempts are cripple the reading performance.
Signed-off-by: NMartin Storsjö <martin@martin.st>

Fixes runtime error: null pointer passed as argument 2, which is
declared to never be null
Signed-off-by: NAndreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>
Signed-off-by: NAnton Khirnov <anton@khirnov.net>

Signed-off-by: NAndreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>
Signed-off-by: NAnton Khirnov <anton@khirnov.net>

This test does no encoding, setting the flag was done just to silence
the warning removed in the previous commit.

The AVStream codec context is often not (and should not be) the actual
encoding context, so this warning will be spurious in many cases.

Signed-off-by: NVittorio Giovara <vittorio.giovara@gmail.com>

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

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

Both avio_skip and detect_unknown_subobject use int64_t for the size
parameter.

This fixes a segmentation fault due to infinite recursion.
Signed-off-by: NAndreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>
Signed-off-by: NAlexandra Hájková <alexandra.khirnova@gmail.com>
Signed-off-by: NAnton Khirnov <anton@khirnov.net>

Otherwise invalid values are used unchecked in the next run.
This can cause NULL pointer dereferencing.
Signed-off-by: NAndreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>
Signed-off-by: NAlexandra Hájková <alexandra.khirnova@gmail.com>
Signed-off-by: NAnton Khirnov <anton@khirnov.net>

So far an AC-3 elementary stream is refered to in the PMT according to
System A (ATSC). However System B (DVB) has a different way to signal an AC-3
ES within the PMT. This different way can be enabled by a new flag. The flag is
more generally named 'system_b' as there are further differences between ATSC
and DVB (e.g. the signalling of E-AC-3) which should then also be covered by it
in the future.

Bug-Id: 73
Signed-off-by: NLuca Barbato <lu_zero@gentoo.org>

This matches what librtmp does. This fixes automatic url parsing of
crunchyroll urls.
Signed-off-by: NMartin Storsjö <martin@martin.st>

The stream parameters are not known until we read a packet, so postpone
creating it until then.

The current muxer behaviour is to create streams in read_header() based
on the audio/video presence flags, but fill in the stream parameters
later when we actually get some packets for them. This is rather shady,
since other demuxers set the stream parameters immediately when the
stream is created and do not touch the stream codec context after that.

Change the flv demuxer to behave in the same way as other similar
demuxers -- create the streams only when we get a packet for them.

The latter does not require a full AVCodecContext and still provides all
the functionality needed here.

Almost all the places from which this function is called already check
the header manually and in the two that don't (the mp3 muxer) the check
should not cause any problems.

It will not be set unless the muxing codec context is also the encoding
context, which is discouraged. When the frame size is not known from
av_get_audio_frame_duration(), the fallback should still be good enough.

It will not be set if the stream codec context is not the encoding
context. Use av_get_audio_frame_duration() instead, it should work for
all audio codecs supported by the muxer.

matroskaenc applies divisors to the display width/height when generating
stereo content. This patch adds the corresponding multipliers to matroskadec
so that the original sample aspect ratio can be recovered.
Signed-off-by: Nwm4 <nfxjfg@googlemail.com>
Signed-off-by: NVittorio Giovara <vittorio.giovara@gmail.com>

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