aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAnton Khirnov <anton@khirnov.net>2013-01-30 16:41:08 +0100
committerAnton Khirnov <anton@khirnov.net>2013-02-15 16:08:44 +0100
commitd2a25c4032ce6ceabb0f51b5c1e6ca865395a793 (patch)
tree9b37d46aa8ff7f9e2a24f623c72e8f50d57a95ff
parent64ed397635ef2666b0ca0c8d8c60a8bc44581d82 (diff)
downloadffmpeg-d2a25c4032ce6ceabb0f51b5c1e6ca865395a793.tar.gz
get_buffer(): do not initialize the data.
There may be more decoders that rely on this. Those should be found and fixed.
-rw-r--r--cmdutils.c5
-rw-r--r--libavcodec/utils.c1
2 files changed, 0 insertions, 6 deletions
diff --git a/cmdutils.c b/cmdutils.c
index 0b3724e8a6..5b853fd910 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 86e154621b..4148264eb1 100644
--- 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])