summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorwm4 <wm4@nowhere>2018-09-15 22:37:14 +0200
committerJan Ekström <jeebjp@gmail.com>2018-10-01 18:55:06 +0300
commit36eeeecff665c66a3079f72b3f9469a83d3df79d (patch)
tree8fd1955795eba3a538085f269cebbd48446b77ec
parente74ffab36ea3d8fce4408d9d2c75142187c594c3 (diff)
downloadmpv-36eeeecff665c66a3079f72b3f9469a83d3df79d.tar.bz2
mpv-36eeeecff665c66a3079f72b3f9469a83d3df79d.tar.xz
demux_lavf: hack-fix EDL mp4 DASH hack
I encountered a stream that fails with "Could not demux init fragment.". It turns out this is a regression from the recent change to that code. The assumption was that demux_lavf.c would treat this as concatenated stream - which it does, but not for probing. Doing this transparently is hard without doing it properly. Doing it properly would mean creating some sort of stream_concat (reminiscent of that FFmpeg security bug). I probably don't want to go there, and I think libavformat should just support this directly, so whatever. Hack-fix this with the knowledge that the init segment will always contain the headers. (cherry picked from commit 315004a38a1bf9394507d8ca308da1ce86ca2204)
-rw-r--r--demux/demux_lavf.c4
1 files changed, 4 insertions, 0 deletions
diff --git a/demux/demux_lavf.c b/demux/demux_lavf.c
index ee3918318a..8603a6a634 100644
--- a/demux/demux_lavf.c
+++ b/demux/demux_lavf.c
@@ -430,6 +430,10 @@ static int lavf_check_file(demuxer_t *demuxer, enum demux_check check)
int nsize = av_clip(avpd.buf_size * 2, INITIAL_PROBE_SIZE,
PROBE_BUF_SIZE);
bstr buf = stream_peek(s, nsize);
+ if (demuxer->params && demuxer->params->init_fragment.len) {
+ buf = demuxer->params->init_fragment;
+ buf.len = MPMIN(buf.len, nsize);
+ }
if (buf.len <= avpd.buf_size)
final_probe = true;
memcpy(avpd.buf, buf.start, buf.len);