[PATCH] avcodec/jpeg2000dec: Zero code and quant style
This prevents future frames from using half initialized/inconsistent structures Fixes assertion failure Fixes Ticket2897 Signed-off-by: Michael Niedermayer <michaelni@gmx.at> --- libavcodec/jpeg2000dec.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/jpeg2000dec.c b/libavcodec/jpeg2000dec.c index 3f22ead..6e8f0d6 100644 --- a/libavcodec/jpeg2000dec.c +++ b/libavcodec/jpeg2000dec.c @@ -1369,6 +1369,8 @@ static void jpeg2000_dec_cleanup(Jpeg2000DecoderContext *s) } } av_freep(&s->tile); + memset(s->codsty, 0, sizeof(s->codsty)); + memset(s->qntsty, 0, sizeof(s->qntsty)); s->numXtiles = s->numYtiles = 0; } -- 1.7.9.5
On 8/31/13, Michael Niedermayer <michaelni@gmx.at> wrote:
This prevents future frames from using half initialized/inconsistent structures Fixes assertion failure Fixes Ticket2897
Signed-off-by: Michael Niedermayer <michaelni@gmx.at> --- libavcodec/jpeg2000dec.c | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/libavcodec/jpeg2000dec.c b/libavcodec/jpeg2000dec.c index 3f22ead..6e8f0d6 100644 --- a/libavcodec/jpeg2000dec.c +++ b/libavcodec/jpeg2000dec.c @@ -1369,6 +1369,8 @@ static void jpeg2000_dec_cleanup(Jpeg2000DecoderContext *s) } } av_freep(&s->tile); + memset(s->codsty, 0, sizeof(s->codsty)); + memset(s->qntsty, 0, sizeof(s->qntsty)); s->numXtiles = s->numYtiles = 0; }
lgtm
On Wed, Oct 02, 2013 at 04:53:41PM +0000, Paul B Mahol wrote:
On 8/31/13, Michael Niedermayer <michaelni@gmx.at> wrote:
This prevents future frames from using half initialized/inconsistent structures Fixes assertion failure Fixes Ticket2897
Signed-off-by: Michael Niedermayer <michaelni@gmx.at> --- libavcodec/jpeg2000dec.c | 2 ++ 1 file changed, 2 insertions(+)
diff --git a/libavcodec/jpeg2000dec.c b/libavcodec/jpeg2000dec.c index 3f22ead..6e8f0d6 100644 --- a/libavcodec/jpeg2000dec.c +++ b/libavcodec/jpeg2000dec.c @@ -1369,6 +1369,8 @@ static void jpeg2000_dec_cleanup(Jpeg2000DecoderContext *s) } } av_freep(&s->tile); + memset(s->codsty, 0, sizeof(s->codsty)); + memset(s->qntsty, 0, sizeof(s->qntsty)); s->numXtiles = s->numYtiles = 0; }
lgtm
applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Observe your enemies, for they first find out your faults. -- Antisthenes
participants (2)
-
Michael Niedermayer -
Paul B Mahol