[FFmpeg-devel] [PATCH]lavf/mpeg: Initialize a stack variable

Carl Eugen Hoyos cehoyos at ag.or.at
Sun Feb 19 17:18:16 EET 2017


Hi!

I cannot reproduce ticket #6160 and I wonder how memcmp() must be implemented 
to reproduce it but attached patch is supposed to fix it.

Please comment, Carl Eugen
-------------- next part --------------
From 0c3a72da0780682eb2899aee7ebe05a5e74a26eb Mon Sep 17 00:00:00 2001
From: Carl Eugen Hoyos <cehoyos at ag.or.at>
Date: Sun, 19 Feb 2017 16:15:34 +0100
Subject: [PATCH] lavf/mpeg: Initialize a stack variable used by memcmp().

Fixes ticket #6160.
---
 libavformat/mpeg.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavformat/mpeg.c b/libavformat/mpeg.c
index 423877e..a651cb3 100644
--- a/libavformat/mpeg.c
+++ b/libavformat/mpeg.c
@@ -138,7 +138,7 @@ typedef struct MpegDemuxContext {
 static int mpegps_read_header(AVFormatContext *s)
 {
     MpegDemuxContext *m = s->priv_data;
-    char buffer[7];
+    char buffer[7] = { 0 };
     int64_t last_pos = avio_tell(s->pb);
 
     m->header_state = 0xff;
-- 
1.7.10.4



More information about the ffmpeg-devel mailing list