diff options
author | Kevin Mitchell <kevmitch@gmail.com> | 2015-12-17 01:40:31 -0800 |
---|---|---|
committer | Kevin Mitchell <kevmitch@gmail.com> | 2015-12-17 12:38:57 -0800 |
commit | 7137afeb2c466135ab0e01ee0f9cbd2abd000563 (patch) | |
tree | 04d3531660fc9f6ed2a0ebee538a95e3b77eae78 /demux/demux.c | |
parent | ad80cc4eeec506fddd11e2a29d0b1868b3ca8170 (diff) | |
download | mpv-7137afeb2c466135ab0e01ee0f9cbd2abd000563.tar.bz2 mpv-7137afeb2c466135ab0e01ee0f9cbd2abd000563.tar.xz |
demux: check embedded cuesheet refers to only one file
If someone was "clever" enough to embed a cuesheet
referencing multiple files, mpv would create a bunch
of nonsense chapter markers.
Diffstat (limited to 'demux/demux.c')
-rw-r--r-- | demux/demux.c | 13 |
1 files changed, 9 insertions, 4 deletions
diff --git a/demux/demux.c b/demux/demux.c index dcad692be5..972bf31bae 100644 --- a/demux/demux.c +++ b/demux/demux.c @@ -926,10 +926,15 @@ static void demux_init_cuesheet(struct demuxer *demuxer) if (cue && !demuxer->num_chapters) { struct cue_file *f = mp_parse_cue(bstr0(cue)); if (f) { - for (int n = 0; n < f->num_tracks; n++) { - struct cue_track *t = &f->tracks[n]; - int idx = demuxer_add_chapter(demuxer, "", t->start, -1); - mp_tags_merge(demuxer->chapters[idx].metadata, t->tags); + if (mp_check_embedded_cue(f) < 0) { + MP_WARN(demuxer, "Embedded cue sheet references more than one file. " + "Ignoring it.\n"); + } else { + for (int n = 0; n < f->num_tracks; n++) { + struct cue_track *t = &f->tracks[n]; + int idx = demuxer_add_chapter(demuxer, "", t->start, -1); + mp_tags_merge(demuxer->chapters[idx].metadata, t->tags); + } } } talloc_free(f); |