Re: [libav-devel] [PATCH] configure: add --enable-rpath

2014-08-06 Thread Reinhard Tartler
On Mon, Aug 4, 2014 at 12:48 PM, Luca Barbato wrote: > On 01/08/14 01:31, wm4 wrote: >> --- >> Backported from FFmpeg. FFmpeg had two authors touching these additions; >> I didn't care because the changes are too trivial. Tell me if you want >> me to backport them with accurate author information.

Re: [libav-devel] [PATCH 1/6] Postpone API-incompatible changes until the next bump.

2014-08-06 Thread Luca Barbato
On 06/08/14 22:02, Anton Khirnov wrote: > --- > libavcodec/version.h| 56 > - > libavfilter/version.h | 16 +++--- > libavformat/version.h | 2 +- > libavresample/version.h | 2 +- > libavutil/version.h | 22 +--

Re: [libav-devel] [PATCH 01/10] Add AVFormatContext.event_flags for signaling to the user when events have happened in the stream.

2014-08-06 Thread Anton Khirnov
On Wed, 6 Aug 2014 15:57:55 -0400, Andrew Stone wrote: > On Wed, Aug 6, 2014 at 12:46 PM, wm4 wrote: > > On Wed, 06 Aug 2014 08:18:22 +0200 > > Anton Khirnov wrote: > > > >> > >> On Tue, 5 Aug 2014 12:58:47 -0400, Andrew Stone wrote: > >> > The only flags, for now, indicate if metadata was up

[libav-devel] [PATCH 4/6] Remove obsolete FF_API_PROBE_MIME cruft.

2014-08-06 Thread Anton Khirnov
--- libavformat/avformat.h | 2 -- libavformat/format.c | 6 -- libavformat/version.h | 4 3 files changed, 12 deletions(-) diff --git a/libavformat/avformat.h b/libavformat/avformat.h index ab2081a..2948d40 100644 --- a/libavformat/avformat.h +++ b/libavformat/avformat.h @@ -391,9 +3

[libav-devel] [PATCH 6/6] lavc: remove obsolete and disabled avpriv functions

2014-08-06 Thread Anton Khirnov
--- libavcodec/adx.c| 8 libavcodec/adx.h| 5 - libavcodec/dv_profile.c | 13 - libavcodec/dv_profile.h | 6 -- libavcodec/flac.c | 8 libavcodec/flac.h | 5 - 6 files changed, 45 deletions(-) diff --git a/libavcodec/adx.c

[libav-devel] [PATCH 1/6] Postpone API-incompatible changes until the next bump.

2014-08-06 Thread Anton Khirnov
--- libavcodec/version.h| 56 - libavfilter/version.h | 16 +++--- libavformat/version.h | 2 +- libavresample/version.h | 2 +- libavutil/version.h | 22 +-- libswscale/version.h| 6 +++--- 6 files changed,

[libav-devel] [PATCH 5/6] Remove obsolete FF_API_REFERENCE_DTS cruft.

2014-08-06 Thread Anton Khirnov
--- libavformat/avformat.h | 5 - libavformat/version.h | 3 --- 2 files changed, 8 deletions(-) diff --git a/libavformat/avformat.h b/libavformat/avformat.h index 2948d40..61ae6fa 100644 --- a/libavformat/avformat.h +++ b/libavformat/avformat.h @@ -826,11 +826,6 @@ typedef struct AVStream {

[libav-devel] [PATCH 2/6] Bump major versions of all libraries.

2014-08-06 Thread Anton Khirnov
--- doc/APIchanges | 14 +++--- libavcodec/version.h| 6 +++--- libavdevice/version.h | 2 +- libavfilter/version.h | 4 ++-- libavformat/version.h | 4 ++-- libavresample/version.h | 4 ++-- libavutil/version.h | 4 ++-- libswscale/version.h| 6 +++--- 8

[libav-devel] [PATCH 3/6] Remove obsolete FF_API_AVFRAME_COLORSPACE cruft.

2014-08-06 Thread Anton Khirnov
--- libavcodec/utils.c | 2 -- libavutil/frame.c | 2 -- libavutil/frame.h | 2 -- libavutil/version.h | 3 --- 4 files changed, 9 deletions(-) diff --git a/libavcodec/utils.c b/libavcodec/utils.c index dc7ffc4..b115da1 100644 --- a/libavcodec/utils.c +++ b/libavcodec/utils.c @@ -576,13 +576

Re: [libav-devel] [PATCH 01/10] Add AVFormatContext.event_flags for signaling to the user when events have happened in the stream.

2014-08-06 Thread Andrew Stone
On Wed, Aug 6, 2014 at 12:46 PM, wm4 wrote: > On Wed, 06 Aug 2014 08:18:22 +0200 > Anton Khirnov wrote: > >> >> On Tue, 5 Aug 2014 12:58:47 -0400, Andrew Stone wrote: >> > The only flags, for now, indicate if metadata was updated and are set >> > after each call to >> > av_read_frame(). This c

Re: [libav-devel] [PATCH] [9] [0.8] error_concealment: avoid using the picture if not fully setup

2014-08-06 Thread Luca Barbato
On 06/08/14 19:19, Vittorio Giovara wrote: > From: Michael Niedermayer > > Fixes state becoming inconsistent and a null pointer dereference. > > CC: libav-sta...@libav.org > Bug-Id: CVE-2013-0860 > Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind > Signed-off-by: Vittorio Giovara > --- >

[libav-devel] [PATCH] [9] [0.8] error_concealment: avoid using the picture if not fully setup

2014-08-06 Thread Vittorio Giovara
From: Michael Niedermayer Fixes state becoming inconsistent and a null pointer dereference. CC: libav-sta...@libav.org Bug-Id: CVE-2013-0860 Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind Signed-off-by: Vittorio Giovara --- libavcodec/error_resilience.c | 6 ++ 1 file changed, 6 in

Re: [libav-devel] [PATCH 01/10] Add AVFormatContext.event_flags for signaling to the user when events have happened in the stream.

2014-08-06 Thread wm4
On Wed, 06 Aug 2014 08:18:22 +0200 Anton Khirnov wrote: > > On Tue, 5 Aug 2014 12:58:47 -0400, Andrew Stone wrote: > > The only flags, for now, indicate if metadata was updated and are set after > > each call to > > av_read_frame(). This comes with the caveat that, on stream start, it might

Re: [libav-devel] [PATCH] avcodec: Undeprecate reordered_opaque

2014-08-06 Thread Luca Barbato
On 06/08/14 18:24, Diego Biurrun wrote: > It allows attaching other external, opaque data to the frame and passing it > through the reordering process, for cases when the caller wants other data > than just the plain packet pts. There is no way to cleanly achieve this > without the field. > --- >

[libav-devel] [PATCH] avcodec: Undeprecate reordered_opaque

2014-08-06 Thread Diego Biurrun
It allows attaching other external, opaque data to the frame and passing it through the reordering process, for cases when the caller wants other data than just the plain packet pts. There is no way to cleanly achieve this without the field. --- Updated the log message. doc/APIchanges | 3

Re: [libav-devel] [PATCH 2/2] avcodec: Rename xvidmmx IDCT to xvid

2014-08-06 Thread Vittorio Giovara
On Wed, Aug 6, 2014 at 2:31 PM, Diego Biurrun wrote: > The Xvid IDCT is not MMX-specific. > --- > > This was an eyesore ... > > libavcodec/avcodec.h| 3 +++ > libavcodec/options_table.h | 5 - > libavcodec/version.h| 3 +++ > libavcodec/x86/idct_sse2_xvid.c | 2 +-

[libav-devel] [PATCH] mpeg: Provide an option to write the psm

2014-08-06 Thread Luca Barbato
Some implementations do not probe the streams for codecs. --- Tested with our demuxer and vlc's and works as expected. libavformat/mpegenc.c | 65 +++ 1 file changed, 65 insertions(+) diff --git a/libavformat/mpegenc.c b/libavformat/mpegenc.c inde

Re: [libav-devel] [PATCH 1/3] configure: Die or warn under the right conditions if gas is unavailable

2014-08-06 Thread Janne Grunau
On 2014-08-06 05:57:38 -0700, Diego Biurrun wrote: > --- > configure | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/configure b/configure > index c8c57b9..45da142 100755 > --- a/configure > +++ b/configure > @@ -3779,8 +3779,8 @@ EOF > } > > if enabled asm; then

Re: [libav-devel] [PATCH 3/3] configure: Only run gas checks on ARM and PowerPC

2014-08-06 Thread Janne Grunau
On 2014-08-06 05:57:40 -0700, Diego Biurrun wrote: > Standalone GNU assembly is not used on x86. > --- > configure | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/configure b/configure > index 81dfc09..a0d60e9 100755 > --- a/configure > +++ b/configure > @@ -3775,7 +3775,

Re: [libav-devel] [PATCH 2/3] configure: Only enable gas-preprocessor if the program is available

2014-08-06 Thread Janne Grunau
On 2014-08-06 05:57:39 -0700, Diego Biurrun wrote: the commit message describes the commit only partially. It also uses gas-preprocessor.pl if it's available and not only on darwin. I think that's not going to work since the plain 'gas-preprocessor.pl' without options will produce mach-O specif

Re: [libav-devel] [PATCH 1/3] configure: Die or warn under the right conditions if gas is unavailable

2014-08-06 Thread Janne Grunau
On 2014-08-06 05:57:38 -0700, Diego Biurrun wrote: > --- > configure | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/configure b/configure > index c8c57b9..45da142 100755 > --- a/configure > +++ b/configure > @@ -3779,8 +3779,8 @@ EOF > } > > if enabled asm; then

[libav-devel] [PATCH 1/2] xvididct: Set idct_algo if Xvid IDCT is enabled

2014-08-06 Thread Diego Biurrun
This avoids triggering code that only works with the default IDCT. --- Fixes FATE with mmxext/sse2 disabled. libavcodec/xvididct.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/xvididct.c b/libavcodec/xvididct.c index 1453e51..7f130ba 100644 --- a/libavcodec/xvididct.c +++ b/l

[libav-devel] [PATCH 2/2] avcodec: Rename xvidmmx IDCT to xvid

2014-08-06 Thread Diego Biurrun
The Xvid IDCT is not MMX-specific. --- This was an eyesore ... libavcodec/avcodec.h| 3 +++ libavcodec/options_table.h | 5 - libavcodec/version.h| 3 +++ libavcodec/x86/idct_sse2_xvid.c | 2 +- libavcodec/xvididct.c | 4 ++-- 5 files changed, 13 insert

Re: [libav-devel] [PATCH 01/10] Add AVFormatContext.event_flags for signaling to the user when events have happened in the stream.

2014-08-06 Thread Andrew Stone
On Wed, Aug 6, 2014 at 2:18 AM, Anton Khirnov wrote: > > On Tue, 5 Aug 2014 12:58:47 -0400, Andrew Stone wrote: >> The only flags, for now, indicate if metadata was updated and are set after >> each call to >> av_read_frame(). This comes with the caveat that, on stream start, it might >> not b

[libav-devel] [PATCH 1/3] configure: Die or warn under the right conditions if gas is unavailable

2014-08-06 Thread Diego Biurrun
--- configure | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/configure b/configure index c8c57b9..45da142 100755 --- a/configure +++ b/configure @@ -3779,8 +3779,8 @@ EOF } if enabled asm; then -enabled arm aarch64 && nogas=die -enabled_all ppc altivec && n

[libav-devel] [PATCH 3/3] configure: Only run gas checks on ARM and PowerPC

2014-08-06 Thread Diego Biurrun
Standalone GNU assembly is not used on x86. --- configure | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure b/configure index 81dfc09..a0d60e9 100755 --- a/configure +++ b/configure @@ -3775,7 +3775,7 @@ EOF return 0 } -if enabled asm; then +if enabled_any arm a

[libav-devel] [PATCH 2/3] configure: Only enable gas-preprocessor if the program is available

2014-08-06 Thread Diego Biurrun
--- configure | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/configure b/configure index 45da142..81dfc09 100755 --- a/configure +++ b/configure @@ -2199,8 +2199,6 @@ ranlib="ranlib" strip="strip" yasmexe="yasm" -nogas=":" - # machine arch_default=$(uname -m) cpu=

Re: [libav-devel] [PATCH] configure: Include the armcc build number in the compiler identification

2014-08-06 Thread Diego Biurrun
On Wed, Aug 06, 2014 at 03:43:09PM +0300, Martin Storsjö wrote: > --- > configure | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Fine with me. Diego ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/li

[libav-devel] [PATCH] configure: Include the armcc build number in the compiler identification

2014-08-06 Thread Martin Storsjö
This tries to find the most expressive part of the output of armcc --vsn to include, giving a compiler identification of "ARM Compiler 5.04 update 2 (build 82)" instead of just "ARM Compiler 5.04" for armcc 5.0. 4.x versions of armcc output the following, for "armcc --vsn": ARM C/C++ Compiler, RVC

Re: [libav-devel] [PATCH 1/2] avcodec: Undeprecate reordered_opaque

2014-08-06 Thread Vladimir Pantelic
I agree with Kieran and Martin, it is very useful to be able to pass a reference to frame specific private data through the decoder. so I am happy to see this undeprecated Regards, Vladimir On Wed, Aug 6, 2014 at 1:40 PM, Martin Storsjö wrote: > On Mon, 4 Aug 2014, Diego Biurrun wrote: > > I

[libav-devel] [PATCH 1/4] mpegts: K&R formatting cosmetics

2014-08-06 Thread Diego Biurrun
--- libavformat/mpegtsenc.c | 415 +--- 1 file changed, 218 insertions(+), 197 deletions(-) diff --git a/libavformat/mpegtsenc.c b/libavformat/mpegtsenc.c index 17e44d9..d490188 100644 --- a/libavformat/mpegtsenc.c +++ b/libavformat/mpegtsenc.c @@ -25,7

[libav-devel] [PATCH 2/4] mpegts: Drop some unnecessary parentheses

2014-08-06 Thread Diego Biurrun
--- libavformat/mpegtsenc.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/libavformat/mpegtsenc.c b/libavformat/mpegtsenc.c index d490188..d3f29ba 100644 --- a/libavformat/mpegtsenc.c +++ b/libavformat/mpegtsenc.c @@ -110,20 +110,20 @@ static void mpegts_write_se

[libav-devel] [PATCH 3/4] mpegts: Avoid unnecessary variable shadowing

2014-08-06 Thread Diego Biurrun
--- libavformat/mpegtsenc.c | 1 - 1 file changed, 1 deletion(-) diff --git a/libavformat/mpegtsenc.c b/libavformat/mpegtsenc.c index d3f29ba..31e295d 100644 --- a/libavformat/mpegtsenc.c +++ b/libavformat/mpegtsenc.c @@ -635,7 +635,6 @@ static int mpegts_write_header(AVFormatContext *s) fail:

[libav-devel] [PATCH 4/4] mpegts: Fix memory leaks and related crashes in mpegs_write_header()

2014-08-06 Thread Diego Biurrun
--- libavformat/mpegtsenc.c | 21 ++--- 1 file changed, 18 insertions(+), 3 deletions(-) diff --git a/libavformat/mpegtsenc.c b/libavformat/mpegtsenc.c index 31e295d..c53e145 100644 --- a/libavformat/mpegtsenc.c +++ b/libavformat/mpegtsenc.c @@ -430,9 +430,15 @@ static MpegTSServi

Re: [libav-devel] [PATCH 2/2] mpeg: Suppress a warning on callback type

2014-08-06 Thread Reinhard Tartler
On Wed, Aug 6, 2014 at 7:20 AM, Luca Barbato wrote: > On 06/08/14 13:08, Reinhard Tartler wrote: >> On Tue, Aug 5, 2014 at 11:01 PM, Luca Barbato wrote: >>> --- >>> libavformat/mpegenc.c | 3 ++- >>> 1 file changed, 2 insertions(+), 1 deletion(-) >>> >>> diff --git a/libavformat/mpegenc.c b/liba

Re: [libav-devel] [PATCH 1/2] avcodec: Undeprecate reordered_opaque

2014-08-06 Thread Martin Storsjö
On Mon, 4 Aug 2014, Diego Biurrun wrote: It is required for the operation of the library and cannot be removed. --- Perhaps something like this: "It allows attaching other external, opaque data to the frame and pass it through the reordering process, for cases when the caller wants other dat

Re: [libav-devel] [PATCH 1/2] avcodec: Undeprecate reordered_opaque

2014-08-06 Thread Kieran Kunhya
> IIRC you need it if you want to attach any private data that should follow > along with the frame through the decoder frame reordering and all that - > normally this only is done for the pkt_pts field (and this is what most > people need), but in case you need to track more data than just the pts

Re: [libav-devel] [PATCH 1/2] avcodec: Undeprecate reordered_opaque

2014-08-06 Thread Martin Storsjö
On Mon, 4 Aug 2014, Hendrik Leppkes wrote: Am 04.08.2014 19:49 schrieb "Diego Biurrun" : It is required for the operation of the library and cannot be removed. --- How is that? While I don't fully agree with the wording that "is required for the operation", it's a feature that IMO makes s

Re: [libav-devel] [PATCH 2/2] mpeg: Suppress a warning on callback type

2014-08-06 Thread Luca Barbato
On 06/08/14 13:08, Reinhard Tartler wrote: > On Tue, Aug 5, 2014 at 11:01 PM, Luca Barbato wrote: >> --- >> libavformat/mpegenc.c | 3 ++- >> 1 file changed, 2 insertions(+), 1 deletion(-) >> >> diff --git a/libavformat/mpegenc.c b/libavformat/mpegenc.c >> index 375915e..500ea48 100644 >> --- a/l

Re: [libav-devel] [PATCH 2/2] cdgraphics: do not return 0 from the decode function

2014-08-06 Thread Luca Barbato
On 06/08/14 12:57, Anton Khirnov wrote: > 0 means no data consumed, so it can trigger an infinite loop in the > caller. > > CC:libav-sta...@libav.org > --- > libavcodec/cdgraphics.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/libavcodec/cdgraphics.c b/libavcodec/cd

Re: [libav-devel] [PATCH 1/2] cdgraphics: switch to bytestream2

2014-08-06 Thread Luca Barbato
On 06/08/14 12:57, Anton Khirnov wrote: > Fixes possible invalid memory accesses on corrupted data. > > CC:libav-sta...@libav.org > Bug-ID: CVE-2013-3674 > --- > libavcodec/cdgraphics.c | 16 +++- > 1 file changed, 7 insertions(+), 9 deletions(-) Looks fine _

Re: [libav-devel] [PATCH] jpeg2000: enable 4 component pixel formats

2014-08-06 Thread Luca Barbato
On 06/08/14 12:53, Vittorio Giovara wrote: > On Wed, Aug 6, 2014 at 11:42 AM, Luca Barbato wrote: >> On 06/08/14 12:07, Vittorio Giovara wrote: >>> Bug-Id: 721 >>> CC: libav-sta...@libav.org >>> Sample-Id: 31230.mov >>> --- >>> libavcodec/jpeg2000dec.c | 2 +- >>> 1 file changed, 1 insertion(+),

Re: [libav-devel] [PATCH 2/2] mpeg: Suppress a warning on callback type

2014-08-06 Thread Reinhard Tartler
On Tue, Aug 5, 2014 at 11:01 PM, Luca Barbato wrote: > --- > libavformat/mpegenc.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/libavformat/mpegenc.c b/libavformat/mpegenc.c > index 375915e..500ea48 100644 > --- a/libavformat/mpegenc.c > +++ b/libavformat/mpegenc.c >

[libav-devel] [PATCH 1/2] cdgraphics: switch to bytestream2

2014-08-06 Thread Anton Khirnov
Fixes possible invalid memory accesses on corrupted data. CC:libav-sta...@libav.org Bug-ID: CVE-2013-3674 --- libavcodec/cdgraphics.c | 16 +++- 1 file changed, 7 insertions(+), 9 deletions(-) diff --git a/libavcodec/cdgraphics.c b/libavcodec/cdgraphics.c index b8a6fb8..752003f 10064

[libav-devel] [PATCH 2/2] cdgraphics: do not return 0 from the decode function

2014-08-06 Thread Anton Khirnov
0 means no data consumed, so it can trigger an infinite loop in the caller. CC:libav-sta...@libav.org --- libavcodec/cdgraphics.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/libavcodec/cdgraphics.c b/libavcodec/cdgraphics.c index 752003f..c3e42e7 100644 --- a/libavcodec/

Re: [libav-devel] [PATCH] svq1: do not modify the input packet

2014-08-06 Thread Diego Biurrun
On Mon, Aug 04, 2014 at 06:56:01AM +0200, Anton Khirnov wrote: > > On Sun, 3 Aug 2014 14:37:21 +0200, Diego Biurrun wrote: > > On Sun, Aug 03, 2014 at 02:35:35PM +0200, Diego Biurrun wrote: > > > --- a/libavcodec/svq1dec.c > > > +++ b/libavcodec/svq1dec.c > > > @@ -60,6 +60,10 @@ typedef struct S

Re: [libav-devel] [PATCH] jpeg2000: enable 4 component pixel formats

2014-08-06 Thread Vittorio Giovara
On Wed, Aug 6, 2014 at 11:42 AM, Luca Barbato wrote: > On 06/08/14 12:07, Vittorio Giovara wrote: >> Bug-Id: 721 >> CC: libav-sta...@libav.org >> Sample-Id: 31230.mov >> --- >> libavcodec/jpeg2000dec.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/libavcodec/jpeg2000

Re: [libav-devel] [PATCH] cdgraphics: check buffer size over header size before use

2014-08-06 Thread Vittorio Giovara
On Wed, Aug 6, 2014 at 10:09 AM, Vittorio Giovara wrote: > Being CDG_HEADER_SIZE bigger than CDG_MINIMUM_PKT_SIZE, > this patch fixes out one out of bound array access. > > CC: libav-sta...@libav.org > Bug-Id: CVE-2013-3674 > Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind > --- > I reworke

Re: [libav-devel] [PATCH] jpeg2000: enable 4 component pixel formats

2014-08-06 Thread Luca Barbato
On 06/08/14 12:07, Vittorio Giovara wrote: > Bug-Id: 721 > CC: libav-sta...@libav.org > Sample-Id: 31230.mov > --- > libavcodec/jpeg2000dec.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavcodec/jpeg2000dec.c b/libavcodec/jpeg2000dec.c > index 7d081ea..aed9b2b 10064

Re: [libav-devel] [PATCH 1/1] configure: check $as first before using $gas as GNU as

2014-08-06 Thread Diego Biurrun
On Tue, Aug 05, 2014 at 01:35:22PM +0200, Janne Grunau wrote: > From: Janne Grunau > --- a/configure > +++ b/configure > @@ -3760,22 +3760,32 @@ unsigned int endian = 'B' << 24 | 'I' << 16 | 'G' << > 8 | 'E'; > > - > -if enabled asm; then > -enabled arm && nogas=die > -enabl

[libav-devel] [PATCH] jpeg2000: enable 4 component pixel formats

2014-08-06 Thread Vittorio Giovara
Bug-Id: 721 CC: libav-sta...@libav.org Sample-Id: 31230.mov --- libavcodec/jpeg2000dec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/jpeg2000dec.c b/libavcodec/jpeg2000dec.c index 7d081ea..aed9b2b 100644 --- a/libavcodec/jpeg2000dec.c +++ b/libavcodec/jpeg2000dec

[libav-devel] [PATCH] cdgraphics: check buffer size over header size before use

2014-08-06 Thread Vittorio Giovara
Being CDG_HEADER_SIZE bigger than CDG_MINIMUM_PKT_SIZE, this patch fixes out one out of bound array access. CC: libav-sta...@libav.org Bug-Id: CVE-2013-3674 Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind --- I reworked the patch, expanding the initial sanity check. Comments welcome. Vittor