On 28/01/15 15:41, Vittorio Giovara wrote:
This prevents several (theoretical) uninitialized variable uses that take
place towards the end of encoding.

CC: libav-sta...@libav.org
Bug-Id: CID 700760
---
  libavcodec/mpegvideo_enc.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavcodec/mpegvideo_enc.c b/libavcodec/mpegvideo_enc.c
index 96cbac0..d9a666a 100644
--- a/libavcodec/mpegvideo_enc.c
+++ b/libavcodec/mpegvideo_enc.c
@@ -2540,7 +2540,7 @@ static int encode_thread(AVCodecContext *c, void *arg){
      int mb_x, mb_y, pdif = 0;
      int chr_h= 16>>s->chroma_y_shift;
      int i, j;
-    MpegEncContext best_s, backup_s;
+    MpegEncContext best_s = { 0 }, backup_s;
      uint8_t bit_buf[2][MAX_MB_BYTES];
      uint8_t bit_buf2[2][MAX_MB_BYTES];
      uint8_t bit_buf_tex[2][MAX_MB_BYTES];


I'm sure some can be practical, we just had been lucky =)

lu
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to