diff options
author | Limin Wang <lance.lmwang@gmail.com> | 2019-11-08 00:16:22 +0800 |
---|---|---|
committer | Michael Niedermayer <michael@niedermayer.cc> | 2019-11-16 23:40:39 +0100 |
commit | c22665ae691ca313793330bfa7c9a2b70d53cbdf (patch) | |
tree | 90066f26ce23f73e0dce58d203557ebcebb18170 | |
parent | e9a335150a62bb377a26ce096187b4476145d02b (diff) | |
download | ffmpeg-c22665ae691ca313793330bfa7c9a2b70d53cbdf.tar.gz |
avformat/mp3dec: Check for occurances of headers within frames during probing
Fixes misdetection of zYLx.wav
Reviewed-by: Paul B Mahol <onemda@gmail.com>
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
-rw-r--r-- | libavformat/mp3dec.c | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/libavformat/mp3dec.c b/libavformat/mp3dec.c index 4fde920caf..71a4ed706d 100644 --- a/libavformat/mp3dec.c +++ b/libavformat/mp3dec.c @@ -71,7 +71,7 @@ static int mp3_read_probe(const AVProbeData *p) int frames, ret; int framesizes, max_framesizes; uint32_t header; - const uint8_t *buf, *buf0, *buf2, *end; + const uint8_t *buf, *buf0, *buf2, *buf3, *end; buf0 = p->buf; end = p->buf + p->buf_size - sizeof(uint32_t); @@ -86,11 +86,19 @@ static int mp3_read_probe(const AVProbeData *p) buf2 = buf; for (framesizes = frames = 0; buf2 < end; frames++) { MPADecodeHeader h; + int header_emu = 0; header = AV_RB32(buf2); ret = avpriv_mpegaudio_decode_header(&h, header); if (ret != 0 || end - buf2 < h.frame_size) break; + + for (buf3 = buf2 + 4; buf3 < buf2 + h.frame_size; buf3++) { + uint32_t next_sync = AV_RB32(buf3); + header_emu += (next_sync & MP3_MASK) == (header & MP3_MASK); + } + if (header_emu > 2) + break; buf2 += h.frame_size; framesizes += h.frame_size; } |