From 06bb1de1c78a3653e197ff3434d550499b556e72 Mon Sep 17 00:00:00 2001 From: Michael Niedermayer Date: Wed, 1 Jan 2014 15:29:20 +0100 Subject: [PATCH] avformat/vqf: check avio_read()s return value more completely Fixes use of uninitialized memory Fixes: msan_uninit-mem_7f180a523a71_5052_esvorbei_extd.vqf Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind Signed-off-by: Michael Niedermayer --- libavformat/vqf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/vqf.c b/libavformat/vqf.c index 526b596146..74c7f5fc17 100644 --- a/libavformat/vqf.c +++ b/libavformat/vqf.c @@ -249,7 +249,7 @@ static int vqf_read_packet(AVFormatContext *s, AVPacket *pkt) pkt->data[1] = c->last_frame_bits; ret = avio_read(s->pb, pkt->data+2, size); - if (ret<=0) { + if (ret != size) { av_free_packet(pkt); return AVERROR(EIO); }