get_buffer(): do not initialize the data.

There may be more decoders that rely on this. Those should be found and
fixed.

diff --git a/cmdutils.c b/cmdutils.c
index 0b3724e8a6a45b4cfaa7b7ab07abaf67e9760f4f..5b853fd9101b4d22b24c57c43781afcbaa38029b 100644 (file)
--- 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 86e154621b056000db921d2700bbf20b3f5d490d..4148264eb1206c63ed5468312e7ebfcf558e771b 100644 (file)
--- a/libavcodec/utils.c
+++ b/libavcodec/utils.c
@@ -421,7 +421,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])