aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMichael Niedermayer <michael@niedermayer.cc>2021-09-16 23:12:42 +0200
committerMichael Niedermayer <michael@niedermayer.cc>2021-10-12 16:46:06 +0200
commitbbbde5f077d08fae91d1bd06fecf64b436f927cc (patch)
treee1eb425a97c2f24478bb03b5fc376e80929c4d19
parent2e68ac694cbded97d3e4a8b5b52fc4d45e7e1610 (diff)
downloadffmpeg-bbbde5f077d08fae91d1bd06fecf64b436f927cc.tar.gz
avformat/mpegts: use actually read packet size in mpegts_resync special case
Fixes: infinite loop Fixes: 37986/clusterfuzz-testcase-minimized-ffmpeg_dem_MPEGTSRAW_fuzzer-5292311517462528 - Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Reviewed-by: Marton Balint <cus@passwd.hu> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc> (cherry picked from commit 83b2e4c8f15a00f037040131e26e20de83f0d842) Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
-rw-r--r--libavformat/mpegts.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/libavformat/mpegts.c b/libavformat/mpegts.c
index e6c3f9af6f..7d6df608c8 100644
--- a/libavformat/mpegts.c
+++ b/libavformat/mpegts.c
@@ -2768,8 +2768,8 @@ static int mpegts_resync(AVFormatContext *s, int seekback, const uint8_t *curren
int64_t back = FFMIN(seekback, pos);
//Special case for files like 01c56b0dc1.ts
- if (current_packet[0] == 0x80 && current_packet[12] == 0x47) {
- avio_seek(pb, 12 - back, SEEK_CUR);
+ if (current_packet[0] == 0x80 && current_packet[12] == 0x47 && pos >= TS_PACKET_SIZE) {
+ avio_seek(pb, 12 - TS_PACKET_SIZE, SEEK_CUR);
return 0;
}