diff options
author | Alexandra Hájková <alexandra.khirnova@gmail.com> | 2015-07-17 14:17:03 +0200 |
---|---|---|
committer | Anton Khirnov <anton@khirnov.net> | 2015-07-19 08:59:48 +0200 |
commit | 406627287e015ce381795e85e2557b12bf60ca35 (patch) | |
tree | 6e5f906905fdf59aac6e1bdf53011f51e47a4371 /libavformat/asfdec.c | |
parent | c9f8cfb6d9b34f3c51f1b7152c4dc3f2f8724dc4 (diff) | |
download | ffmpeg-406627287e015ce381795e85e2557b12bf60ca35.tar.gz |
asfdec: do not read replicated data when their length is 0
Signed-off-by: Anton Khirnov <anton@khirnov.net>
Diffstat (limited to 'libavformat/asfdec.c')
-rw-r--r-- | libavformat/asfdec.c | 24 |
1 files changed, 13 insertions, 11 deletions
diff --git a/libavformat/asfdec.c b/libavformat/asfdec.c index 7af372805a..371d4807ff 100644 --- a/libavformat/asfdec.c +++ b/libavformat/asfdec.c @@ -1123,17 +1123,19 @@ static int asf_read_multiple_payload(AVFormatContext *s, AVPacket *pkt, if ((ret = asf_read_subpayload(s, pkt, 1)) < 0) return ret; } else { - if (!asf_pkt->data_size) { - asf_pkt->data_size = asf_pkt->size_left = avio_rl32(pb); // read media object size - if (asf_pkt->data_size <= 0) - return AVERROR_EOF; - if ((ret = av_new_packet(&asf_pkt->avpkt, asf_pkt->data_size)) < 0) - return ret; - } else - avio_skip(pb, 4); // reading of media object size is already done - asf_pkt->dts = avio_rl32(pb); // read presentation time - if ((asf->rep_data_len - 8) > 0) - avio_skip(pb, asf->rep_data_len - 8); // skip replicated data + if (asf->rep_data_len) { + if (!asf_pkt->data_size) { + asf_pkt->data_size = asf_pkt->size_left = avio_rl32(pb); // read media object size + if (asf_pkt->data_size <= 0) + return AVERROR_EOF; + if ((ret = av_new_packet(&asf_pkt->avpkt, asf_pkt->data_size)) < 0) + return ret; + } else + avio_skip(pb, 4); // reading of media object size is already done + asf_pkt->dts = avio_rl32(pb); // read presentation time + if (asf->rep_data_len && ((asf->rep_data_len - 8) > 0)) + avio_skip(pb, asf->rep_data_len - 8); // skip replicated data + } pay_len = avio_rl16(pb); // payload length should be WORD if (pay_len > asf->packet_size) { av_log(s, AV_LOG_ERROR, |