There may be more decoders that rely on this. Those should be found and
fixed.
---
 cmdutils.c         |    5 -----
 libavcodec/utils.c |    1 -
 2 files changed, 6 deletions(-)

diff --git a/cmdutils.c b/cmdutils.c
index 0b3724e..5b853fd 100644
--- a/cmdutils.c
+++ b/cmdutils.c
@@ -1563,11 +1563,6 @@ static int alloc_buffer(FrameBuffer **pool, 
AVCodecContext *s, FrameBuffer **pbu
         av_freep(&buf);
         return ret;
     }
-    /* XXX this shouldn't be needed, but some tests break without this line
-     * those decoders are buggy and need to be fixed.
-     * the following tests fail:
-     */
-    memset(buf->base[0], 128, ret);
 
     av_pix_fmt_get_chroma_sub_sample(s->pix_fmt,
                                      &h_chroma_shift, &v_chroma_shift);
diff --git a/libavcodec/utils.c b/libavcodec/utils.c
index 19c8a99..5abe492 100644
--- a/libavcodec/utils.c
+++ b/libavcodec/utils.c
@@ -414,7 +414,6 @@ static int video_get_buffer(AVCodecContext *s, AVFrame *pic)
             buf->base[i] = av_malloc(size[i] + 16); //FIXME 16
             if (buf->base[i] == NULL)
                 return -1;
-            memset(buf->base[i], 128, size[i]);
 
             // no edge if EDGE EMU or not planar YUV
             if ((s->flags & CODEC_FLAG_EMU_EDGE) || !size[2])
-- 
1.7.10.4

_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to