[FFmpeg-cvslog] avformat/thp: Check av_get_packet() for failure not only for partial output
Michael Niedermayer
git at videolan.org
Thu Mar 12 18:05:34 CET 2015
ffmpeg | branch: release/0.10 | Michael Niedermayer <michaelni at gmx.at> | Thu Feb 5 03:45:21 2015 +0100| [3b90117a1f9a29dfe9588ad5de8c3f4b9640cc38] | committer: Michael Niedermayer
avformat/thp: Check av_get_packet() for failure not only for partial output
Fixes null pointer dereference
Fixes: signal_sigsegv_db2c1f_3108_cov_163322880_pikmin2_opening1_partial.thp
Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
(cherry picked from commit f2579dbb4b31e6ae731e7f5555680528ef3020ab)
Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=3b90117a1f9a29dfe9588ad5de8c3f4b9640cc38
---
libavformat/thp.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/libavformat/thp.c b/libavformat/thp.c
index 75f7941..bed3f59 100644
--- a/libavformat/thp.c
+++ b/libavformat/thp.c
@@ -179,6 +179,8 @@ static int thp_read_packet(AVFormatContext *s,
pkt->stream_index = thp->video_stream_index;
} else {
ret = av_get_packet(pb, pkt, thp->audiosize);
+ if (ret < 0)
+ return ret;
if (ret != thp->audiosize) {
av_free_packet(pkt);
return AVERROR(EIO);
More information about the ffmpeg-cvslog
mailing list