On Wed, Feb 29, 2012 at 05:50:28PM -0800, Ronald S. Bultje wrote:
> From: "Ronald S. Bultje"
>
> Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
> CC: libav-sta...@libav.org
> ---
> libavcodec/lcldec.c | 14 +++---
> 1 files changed, 7 insertions(+), 7 deletions(-)
OK
_
> 2012/2/29 Christophe Gisquet :
>>> + int mclms_prevvalues[64];
>> [...]
>>> + int lms_prevvalues[512];
>>
>> Those should probably be int32_t
>
> Actually, all samples I tested it on never overflow int16_t. Even with
> dsp functions doing a lot of work, switching to int16_t i
From: "Ronald S. Bultje"
Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-sta...@libav.org
---
libavcodec/lcldec.c | 14 +++---
1 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/libavcodec/lcldec.c b/libavcodec/lcldec.c
index d3a85f7..b486ea0 100644
--- a/l
Hi,
On Wed, Feb 29, 2012 at 4:50 PM, Diego Biurrun wrote:
> On Thu, Mar 01, 2012 at 01:42:45AM +0100, Diego Biurrun wrote:
>> On Wed, Feb 29, 2012 at 01:13:07PM -0800, Ronald S. Bultje wrote:
>> > On Tue, Feb 28, 2012 at 12:53 PM, Ronald S. Bultje
>> > wrote:
>> > > From: "Ronald S. Bultje"
>>
On Thu, Mar 01, 2012 at 01:42:45AM +0100, Diego Biurrun wrote:
> On Wed, Feb 29, 2012 at 01:13:07PM -0800, Ronald S. Bultje wrote:
> > On Tue, Feb 28, 2012 at 12:53 PM, Ronald S. Bultje
> > wrote:
> > > From: "Ronald S. Bultje"
> > >
> > > Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
>
On Wed, Feb 29, 2012 at 01:13:07PM -0800, Ronald S. Bultje wrote:
> On Tue, Feb 28, 2012 at 12:53 PM, Ronald S. Bultje wrote:
> > From: "Ronald S. Bultje"
> >
> > Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
> > CC: libav-sta...@libav.org
> > ---
> > libavcodec/cabac_functions.h | 6
Hi,
On Wed, Feb 29, 2012 at 8:16 AM, Sean McGovern wrote:
> On Sunday, February 19, 2012, Aneesh Dogra wrote:
>> ---
>> libavcodec/sunrast.c | 6 +-
>> 1 files changed, 5 insertions(+), 1 deletions(-)
>
> Not to sound like a nag, but this should probably have spelling agreement
> between
Hi,
On Wed, Feb 29, 2012 at 10:39 AM, Vitor Sessak wrote:
> ---
> libavcodec/nellymoserdec.c | 16
> 1 files changed, 8 insertions(+), 8 deletions(-)
Thanks, pushed.
Ronald
___
libav-devel mailing list
libav-devel@libav.org
https:/
Hi,
On Wed, Feb 29, 2012 at 3:07 PM, Ronald S. Bultje wrote:
> From: "Ronald S. Bultje"
>
> Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
> CC: libav-sta...@libav.org
> ---
> libavcodec/dsicinav.c | 38 +++---
> 1 files changed, 23 insertions(+), 15 de
From: "Ronald S. Bultje"
Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-sta...@libav.org
---
libavcodec/dsicinav.c | 38 +++---
1 files changed, 23 insertions(+), 15 deletions(-)
diff --git a/libavcodec/dsicinav.c b/libavcodec/dsicinav.c
index
From: "Ronald S. Bultje"
Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-sta...@libav.org
---
libavcodec/smc.c | 46 ++
1 files changed, 38 insertions(+), 8 deletions(-)
diff --git a/libavcodec/smc.c b/libavcodec/smc.c
index f4a0b6a
From: "Ronald S. Bultje"
Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-sta...@libav.org
---
libavcodec/smc.c | 26 +-
1 files changed, 25 insertions(+), 1 deletions(-)
diff --git a/libavcodec/smc.c b/libavcodec/smc.c
index f4a0b6a..09f3b54 100644
---
Hi,
On Wed, Feb 29, 2012 at 1:12 PM, Ronald S. Bultje wrote:
> On Wed, Feb 29, 2012 at 1:09 PM, Vitor Sessak wrote:
>> ---
>> libavcodec/amrnbdec.c | 8
>> 1 files changed, 8 insertions(+), 0 deletions(-)
>>
>> diff --git a/libavcodec/amrnbdec.c b/libavcodec/amrnbdec.c
>> index fff0
Hendrik Leppkes writes:
> 2012/2/29 Måns Rullgård :
>> There's much more to it than that. Almost anything using
>> variable-length codes will need more than a simple packet size check, or
>> a damaged/malicious bitstream may cause over-reads.
>>
>
> At least sipr, ra144 and ra288 do not use any
On 2/29/12, Ronald S. Bultje wrote:
> From: "Ronald S. Bultje"
>
> Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
> CC: libav-sta...@libav.org
> ---
> libavcodec/cscd.c |4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
LGTM
___
2012/2/29 Måns Rullgård :
> There's much more to it than that. Almost anything using
> variable-length codes will need more than a simple packet size check, or
> a damaged/malicious bitstream may cause over-reads.
>
At least sipr, ra144 and ra288 do not use any variable length codes. I
didn't che
From: "Ronald S. Bultje"
Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-sta...@libav.org
---
libavcodec/cscd.c |4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/libavcodec/cscd.c b/libavcodec/cscd.c
index 43f0e5e..17f4878 100644
--- a/libavcodec/cscd.c
Hi,
On Wed, Feb 29, 2012 at 10:24 AM, Diego Biurrun wrote:
> ---
> libavcodec/asv1.c | 2 +-
> libavcodec/avs.c | 2 +-
> libavcodec/cavsdec.c | 6 +++---
> libavcodec/h261dec.c | 3 ++-
> libavcodec/h263dec.c | 6 +++---
> libavcodec/h264.c | 2
On 2/29/12, Diego Biurrun wrote:
> ---
> libavcodec/utils.c |1 -
> 1 files changed, 0 insertions(+), 1 deletions(-)
>
LGTM
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
"Ronald S. Bultje" writes:
> Hi,
>
> 2012/2/29 Måns Rullgård :
>> Vitor Sessak writes:
>>
>>> On 02/29/2012 04:28 PM, Janne Grunau wrote:
On 2012-02-26 09:52:44 +0100, Vitor Sessak wrote:
> ---
> libavcodec/ra144dec.c | 2 ++
> libavcodec/ra288.c | 2 ++
> libav
Hi,
2012/2/29 Måns Rullgård :
> Vitor Sessak writes:
>
>> On 02/29/2012 04:28 PM, Janne Grunau wrote:
>>> On 2012-02-26 09:52:44 +0100, Vitor Sessak wrote:
---
libavcodec/ra144dec.c | 2 ++
libavcodec/ra288.c | 2 ++
libavcodec/sipr.c | 2 ++
libavcod
Vitor Sessak writes:
> On 02/29/2012 04:28 PM, Janne Grunau wrote:
>> On 2012-02-26 09:52:44 +0100, Vitor Sessak wrote:
>>> ---
>>> libavcodec/ra144dec.c |2 ++
>>> libavcodec/ra288.c|2 ++
>>> libavcodec/sipr.c |2 ++
>>> libavcodec/twinvq.c |2 ++
>>> 4 files cha
2012/2/29 Ronald S. Bultje :
> - quant_index_table[i] = quant_index_table[i - 1] + j - 12; //
> differential encoding
> + quant_index_table[i] = av_clip(quant_index_table[i - 1] + j - 12,
> -63, 63); // differential encoding
It seems those values are used to determine bit length of
Hi,
On Wed, Feb 29, 2012 at 1:13 PM, Ronald S. Bultje wrote:
> On Tue, Feb 28, 2012 at 12:53 PM, Ronald S. Bultje wrote:
>> From: "Ronald S. Bultje"
>>
>> Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
>> CC: libav-sta...@libav.org
>> ---
>> libavcodec/cabac_functions.h | 6 --
>
Hi,
On Tue, Feb 28, 2012 at 12:53 PM, Ronald S. Bultje wrote:
> From: "Ronald S. Bultje"
>
> Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
> CC: libav-sta...@libav.org
> ---
> libavcodec/cabac_functions.h | 6 --
> libavcodec/x86/cabac.h | 12
> 2 files chan
Hi,
On Wed, Feb 29, 2012 at 1:09 PM, Vitor Sessak wrote:
> ---
> libavcodec/amrnbdec.c | 8
> 1 files changed, 8 insertions(+), 0 deletions(-)
>
> diff --git a/libavcodec/amrnbdec.c b/libavcodec/amrnbdec.c
> index fff0e72..cdfcfc7 100644
> --- a/libavcodec/amrnbdec.c
> +++ b/libavcod
On 02/29/2012 10:02 PM, Ronald S. Bultje wrote:
Hi,
On Wed, Feb 29, 2012 at 12:51 PM, Vitor Sessak wrote:
---
libavcodec/amrnbdec.c |7 +++
1 files changed, 7 insertions(+), 0 deletions(-)
diff --git a/libavcodec/amrnbdec.c b/libavcodec/amrnbdec.c
index fff0e72..d88f5f0 100644
---
---
libavcodec/amrnbdec.c |8
1 files changed, 8 insertions(+), 0 deletions(-)
diff --git a/libavcodec/amrnbdec.c b/libavcodec/amrnbdec.c
index fff0e72..cdfcfc7 100644
--- a/libavcodec/amrnbdec.c
+++ b/libavcodec/amrnbdec.c
@@ -200,6 +200,10 @@ static enum Mode unpack_bitstream(AMRCo
2012/2/29 Christophe Gisquet :
>> + int mclms_prevvalues[64];
> [...]
>> + int lms_prevvalues[512];
>
> Those should probably be int32_t
Actually, all samples I tested it on never overflow int16_t. Even with
dsp functions doing a lot of work, switching to int16_t is a 14%
decoding sp
Hi,
On Wed, Feb 29, 2012 at 12:51 PM, Vitor Sessak wrote:
> ---
> libavcodec/amrnbdec.c | 7 +++
> 1 files changed, 7 insertions(+), 0 deletions(-)
>
> diff --git a/libavcodec/amrnbdec.c b/libavcodec/amrnbdec.c
> index fff0e72..d88f5f0 100644
> --- a/libavcodec/amrnbdec.c
> +++ b/libavcod
---
libavcodec/amrnbdec.c |7 +++
1 files changed, 7 insertions(+), 0 deletions(-)
diff --git a/libavcodec/amrnbdec.c b/libavcodec/amrnbdec.c
index fff0e72..d88f5f0 100644
--- a/libavcodec/amrnbdec.c
+++ b/libavcodec/amrnbdec.c
@@ -946,6 +946,13 @@ static int amrnb_decode_frame(AVCodecCon
On 02/29/2012 09:42 PM, Ronald S. Bultje wrote:
Hi,
On Wed, Feb 29, 2012 at 12:41 PM, Vitor Sessak wrote:
On 02/29/2012 09:08 PM, Ronald S. Bultje wrote:
On Wed, Feb 29, 2012 at 12:03 PM, Vitor Sessak
wrote:
---
libavcodec/amrnbdec.c |7 +++
1 files changed, 7 insertions(+), 0
From: Michael Niedermayer
Signed-off-by: Michael Niedermayer
Signed-off-by: Justin Ruggles
---
libavcodec/mpegaudio_parser.c |3 ++-
1 files changed, 2 insertions(+), 1 deletions(-)
diff --git a/libavcodec/mpegaudio_parser.c b/libavcodec/mpegaudio_parser.c
index 5fd9037..3748f5d 100644
--
On 02/28/2012 11:23 PM, Ronald S. Bultje wrote:
From: "Ronald S. Bultje"
Check that the mode is valid before using it as an index into tables.
Also provide a size parameter to ff_amr_bit_reorder() to prevent
overreads.
Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-sta...@liba
Hi,
On Wed, Feb 29, 2012 at 12:41 PM, Vitor Sessak wrote:
> On 02/29/2012 09:08 PM, Ronald S. Bultje wrote:
>> On Wed, Feb 29, 2012 at 12:03 PM, Vitor Sessak
>> wrote:
>>>
>>> ---
>>> libavcodec/amrnbdec.c | 7 +++
>>> 1 files changed, 7 insertions(+), 0 deletions(-)
>>>
>>> diff --git a
On 02/28/2012 03:53 PM, Ronald S. Bultje wrote:
> From: "Ronald S. Bultje"
>
> Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
> CC: libav-sta...@libav.org
> ---
> libavcodec/cabac_functions.h |6 --
> libavcodec/x86/cabac.h | 12
> 2 files changed, 12 insert
On 02/29/2012 09:08 PM, Ronald S. Bultje wrote:
Hi,
On Wed, Feb 29, 2012 at 12:03 PM, Vitor Sessak wrote:
---
libavcodec/amrnbdec.c |7 +++
1 files changed, 7 insertions(+), 0 deletions(-)
diff --git a/libavcodec/amrnbdec.c b/libavcodec/amrnbdec.c
index fff0e72..d88f5f0 100644
---
On 02/28/2012 09:48 PM, Ronald S. Bultje wrote:
> From: "Ronald S. Bultje"
>
> Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
> CC: libav-sta...@libav.org
> ---
> libavcodec/h264_cabac.c |2 ++
> 1 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git a/libavcodec/h264_caba
Hi,
On Tue, Feb 21, 2012 at 9:22 AM, Derek Buitenhuis
wrote:
> From: Reimar Döffinger
>
> Signed-off-by: Reimar Döffinger
> ---
> tests/fate/microsoft.mak | 3 +
> tests/ref/fate/wmv8-x8intra | 474
> +++
> 2 files changed, 477 insertions(+), 0 d
Hi,
On Wed, Feb 29, 2012 at 12:06 PM, Vitor Sessak wrote:
> ---
> libavcodec/mpegaudiodec.c | 7 +++
> 1 files changed, 7 insertions(+), 0 deletions(-)
Sorry, forgot earlier one - pushed now.
Ronald
___
libav-devel mailing list
libav-devel@lib
Hi,
On Wed, Feb 29, 2012 at 12:03 PM, Vitor Sessak wrote:
> ---
> libavcodec/amrnbdec.c | 7 +++
> 1 files changed, 7 insertions(+), 0 deletions(-)
>
> diff --git a/libavcodec/amrnbdec.c b/libavcodec/amrnbdec.c
> index fff0e72..d88f5f0 100644
> --- a/libavcodec/amrnbdec.c
> +++ b/libavcod
On 02/26/2012 02:10 PM, Benjamin Larsson wrote:
OK
ping (and patch updated with better commit message).
-Vitor
___
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
---
libavcodec/mpegaudiodec.c |7 +++
1 files changed, 7 insertions(+), 0 deletions(-)
diff --git a/libavcodec/mpegaudiodec.c b/libavcodec/mpegaudiodec.c
index d82432c..d6a09c8 100644
--- a/libavcodec/mpegaudiodec.c
+++ b/libavcodec/mpegaudiodec.c
@@ -29,6 +29,7 @@
#include "get_bits.h"
On 02/26/2012 03:21 PM, Diego Biurrun wrote:
On Sun, Feb 26, 2012 at 12:00:21PM +0100, Vitor Sessak wrote:
--- a/libavcodec/amrnbdec.c
+++ b/libavcodec/amrnbdec.c
@@ -946,6 +946,13 @@ static int amrnb_decode_frame(AVCodecContext *avctx, void
*data,
+if (buf_size< frame_sizes_nb[p->cur_fram
---
libavcodec/amrnbdec.c |7 +++
1 files changed, 7 insertions(+), 0 deletions(-)
diff --git a/libavcodec/amrnbdec.c b/libavcodec/amrnbdec.c
index fff0e72..d88f5f0 100644
--- a/libavcodec/amrnbdec.c
+++ b/libavcodec/amrnbdec.c
@@ -946,6 +946,13 @@ static int amrnb_decode_frame(AVCodecCon
Hi,
On Sun, Feb 26, 2012 at 8:57 AM, Diego Biurrun wrote:
> + vector signed short
> \
> + lCY = c->CY,
> \
> + lOY = c->OY,
On 02/29/2012 02:04 PM, Alex Converse wrote:
> From: Christophe Gisquet
>
> ---
> libavcodec/aacsbr.c | 33 +++--
> libavcodec/sbr.h|1 +
> 2 files changed, 20 insertions(+), 14 deletions(-)
LGTM.
-Justin
___
l
On 02/29/2012 01:49 PM, Diego Biurrun wrote:
> libavcodec/mjpegdec.c:1463: warning: format ‘%x’ expects type ‘unsigned int’,
> but argument 5 has type ‘const uint8_t *’
> ---
> libavcodec/mjpegdec.c |2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/libavcodec/mjpegd
On Wed, Feb 29, 2012 at 11:20:24AM -0800, Ronald S. Bultje wrote:
> Hi,
>
> On Tue, Feb 28, 2012 at 9:59 PM, Kostya Shishkov
> wrote:
> > On Tue, Feb 28, 2012 at 07:00:54PM -0800, Ronald S. Bultje wrote:
> >> From: "Ronald S. Bultje"
> >>
> >> Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwi
On 02/27/2012 10:53 AM, Justin Ruggles wrote:
> Takes encoder delay into account by comparing first the coded page
> duration with the calculated page duration. Handles last packet duration
> if needed, also by comparing coded duration with calculated duration.
> Also does better handling of times
Hi,
On Tue, Feb 28, 2012 at 9:59 PM, Kostya Shishkov
wrote:
> On Tue, Feb 28, 2012 at 07:00:54PM -0800, Ronald S. Bultje wrote:
>> From: "Ronald S. Bultje"
>>
>> Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
>> CC: libav-sta...@libav.org
>> ---
>> libavcodec/vmnc.c | 1 +
>> 1 files
---
libavcodec/utils.c |1 -
1 files changed, 0 insertions(+), 1 deletions(-)
diff --git a/libavcodec/utils.c b/libavcodec/utils.c
index 2d79f66..0e611e2 100644
--- a/libavcodec/utils.c
+++ b/libavcodec/utils.c
@@ -1091,7 +1091,6 @@ int attribute_align_arg
avcodec_encode_video2(AVCodecContex
From: Christophe Gisquet
---
libavcodec/aacsbr.c | 33 +++--
libavcodec/sbr.h|1 +
2 files changed, 20 insertions(+), 14 deletions(-)
diff --git a/libavcodec/aacsbr.c b/libavcodec/aacsbr.c
index 3a30fca..f5ac30f 100644
--- a/libavcodec/aacsbr.c
+++ b/libavc
libavcodec/mjpegdec.c:1463: warning: format ‘%x’ expects type ‘unsigned int’,
but argument 5 has type ‘const uint8_t *’
---
libavcodec/mjpegdec.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/libavcodec/mjpegdec.c b/libavcodec/mjpegdec.c
index 47a85cb..ea6230c 100644
-
On Wed, Feb 29, 2012 at 01:40:05PM -0500, Sean McGovern wrote:
> On 2/29/12, Diego Biurrun wrote:
> > libavformat/rmdec.c:383: warning: format ‘%d’ expects type ‘int’, but
> > argument 7 has type ‘int64_t’
> > ---
> > libavformat/rmdec.c |2 +-
> > 1 files changed, 1 insertions(+), 1 deletion
On 02/29/2012 04:28 PM, Janne Grunau wrote:
On 2012-02-26 09:52:44 +0100, Vitor Sessak wrote:
---
libavcodec/ra144dec.c |2 ++
libavcodec/ra288.c|2 ++
libavcodec/sipr.c |2 ++
libavcodec/twinvq.c |2 ++
4 files changed, 8 insertions(+), 0 deletions(-)
Why?
Hav
On 2/29/12, Diego Biurrun wrote:
> libavformat/rmdec.c:383: warning: format ‘%d’ expects type ‘int’, but
> argument 7 has type ‘int64_t’
> ---
> libavformat/rmdec.c |2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
Looks like there is a similar format specifier issue in lavc mjpegdec
---
libavcodec/nellymoserdec.c | 16
1 files changed, 8 insertions(+), 8 deletions(-)
diff --git a/libavcodec/nellymoserdec.c b/libavcodec/nellymoserdec.c
index 96fb053..1583c20 100644
--- a/libavcodec/nellymoserdec.c
+++ b/libavcodec/nellymoserdec.c
@@ -49,14 +49,15 @@ typedef
On Mon, 27 Feb 2012 22:18:04 +0100, Diego Biurrun wrote:
> ---
> libavcodec/asv1.c|2 +-
> libavcodec/avs.c |2 +-
> libavcodec/cavsdec.c |6 +++---
> libavcodec/h261dec.c |3 ++-
> libavcodec/h263dec.c |6 +++---
> libavcodec/h264.c|4 ++--
> libavcodec/indeo
---
libavcodec/asv1.c|2 +-
libavcodec/avs.c |2 +-
libavcodec/cavsdec.c |6 +++---
libavcodec/h261dec.c |3 ++-
libavcodec/h263dec.c |6 +++---
libavcodec/h264.c| 26 +++---
libavcodec/h264_direct.c |3 +--
libavcod
libavformat/rmdec.c:383: warning: format ‘%d’ expects type ‘int’, but argument
7 has type ‘int64_t’
---
libavformat/rmdec.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/libavformat/rmdec.c b/libavformat/rmdec.c
index 6775294..a5e6d2d 100644
--- a/libavformat/rmdec.c
+
On Mon, 27 Feb 2012 22:18:03 +0100, Diego Biurrun wrote:
> ---
> libavcodec/cavsdec.c | 10
> libavcodec/error_resilience.c |6 ++--
> libavcodec/h264.c | 35 +++--
> libavcodec/h264_direct.c |4 ++-
> libavcodec/ljpegenc
On Mon, Feb 27, 2012 at 08:32:22PM +0100, Hendrik Leppkes wrote:
> On Mon, Feb 27, 2012 at 7:07 PM, Diego Biurrun wrote:
> > --- a/libavformat/rmdec.c
> > +++ b/libavformat/rmdec.c
> > @@ -378,7 +378,7 @@ static int rm_read_index(AVFormatContext *s)
> > av_log(s, AV_LOG_ERROR,
> >
On Mon, 27 Feb 2012 22:18:02 +0100, Diego Biurrun wrote:
> ---
> libavcodec/a64multienc.c |2 +-
> libavcodec/avs.c |2 +-
> libavcodec/bmp.c |4 ++--
> libavcodec/bmpenc.c |6 +++---
> libavcodec/fraps.c |4 ++--
> libavcodec/gif.c |2 +
Christophe Gisquet writes:
> 2012/2/29 Mashiat Sarker Shakkhar :
+ if (s->bV3RTM)
+ for (icoef = 0; icoef < s->cdlms[ich][ilms].order; icoef++)
+ s->cdlms[ich][ilms].lms_updates[icoef + recent] /= 2;
+ else
+ for (ic
From: "Ronald S. Bultje"
It is used as an index in a table of that size, therefore larger values
should not be allowed since they lead to stack overreads.
Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC: libav-sta...@libav.org
---
libavcodec/cook.c |2 +-
1 files changed, 1 insert
On Wed, Feb 29, 2012 at 09:32:31AM -0800, Mashiat Sarker Shakkhar wrote:
>
> --- /dev/null
> +++ b/libavcodec/wmalosslessdec.c
> @@ -0,0 +1,1286 @@
> +/* frame info */
> +s->skip_frame = 1; /* skip first frame */
> +s->packet_loss = 1;
> +s->len_prefix = (s->decode_flags & 0x40)
On 2/29/12, Diego Biurrun wrote:
> On Sun, Feb 26, 2012 at 07:23:50PM +, Paul B Mahol wrote:
>>
>> --- a/tests/fate/cdxl.mak
>> +++ b/tests/fate/cdxl.mak
>> @@ -10,5 +10,8 @@ fate-cdxl-pal8: CMD = framecrc -i
>> $(SAMPLES)/cdxl/maku.cdxl -pix_fmt rgb24 -fram
>> FATE_CDXL += fate-cdxl-pal8-sma
On Wed, 29 Feb 2012 18:21:28 +0100, Diego Biurrun wrote:
> ---
> libavcodec/Makefile | 12 +-
> libavcodec/msmpeg4.c | 700
> +-
> libavcodec/msmpeg4.h | 12 +
> libavcodec/msmpeg4data.c |4 +
> libavcodec/msmpeg4data.h |4 +
2012/2/29 Mashiat Sarker Shakkhar :
>>> + if (s->bV3RTM)
>>> + for (icoef = 0; icoef < s->cdlms[ich][ilms].order;
>> icoef++)
>>> + s->cdlms[ich][ilms].lms_updates[icoef + recent] /= 2;
>>> + else
>>> + for (icoef = 0; icoef < s->cdlms[ich][il
- Original Message -
> From: Diego Biurrun
> To: libav development
> Cc:
> Sent: Wednesday, February 29, 2012 9:00 PM
> Subject: Re: [libav-devel] [PATCH] Windows Media Audio Lossless decoder
[...]
>> +typedef struct WmallDecodeCtx {
>> + /* generic decoder variables */
>> + A
---
libavcodec/Makefile | 12 +-
libavcodec/msmpeg4.c | 700 +-
libavcodec/msmpeg4.h | 12 +
libavcodec/msmpeg4data.c |4 +
libavcodec/msmpeg4data.h |4 +
libavcodec/msmpeg4enc.c | 692 +
On Tue, Feb 28, 2012 at 03:05:14PM +0100, Anton Khirnov wrote:
>
> On Sat, 18 Feb 2012 14:40:25 +0100, Diego Biurrun wrote:
> > --- a/libavcodec/msmpeg4.c
> > +++ b/libavcodec/msmpeg4.c
> > @@ -52,21 +53,12 @@
> >
> > -static uint32_t v2_dc_lum_table[512][2];
> > -static uint32_t v2_dc_chroma_t
On Wed, Feb 29, 2012 at 11:29:18AM -0500, Sean McGovern wrote:
> On Wednesday, February 29, 2012, Diego Biurrun wrote:
> >
> > I wanted to talk about your SunCC FATE box.
> >
> > It is known-broken and we know that it will not be fixed any time soon.
>
> OK. Since we've heard hide nor hair of Dan
On 2012-02-22 20:59:23 +0100, Diego Biurrun wrote:
> ---
> libavcodec/aacdec.c |2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/libavcodec/aacdec.c b/libavcodec/aacdec.c
> index 80d8fbf..af8774d 100644
> --- a/libavcodec/aacdec.c
> +++ b/libavcodec/aacdec.c
> @@ -5,7
Sean McGovern writes:
> On Wednesday, February 29, 2012, Diego Biurrun wrote:
>>
>> I wanted to talk about your SunCC FATE box.
>>
>> It is known-broken and we know that it will not be fixed any time soon.
>
> OK. Since we've heard hide nor hair of Daniel regarding mlpdsp I will go
> ahead and p
On Wednesday, February 29, 2012, Diego Biurrun wrote:
>
> I wanted to talk about your SunCC FATE box.
>
> It is known-broken and we know that it will not be fixed any time soon.
OK. Since we've heard hide nor hair of Daniel regarding mlpdsp I will go
ahead and pull the suncc build. It will take 1
On Sunday, February 19, 2012, Aneesh Dogra wrote:
> ---
> libavcodec/sunrast.c |6 +-
> 1 files changed, 5 insertions(+), 1 deletions(-)
Not to sound like a nag, but this should probably have spelling agreement
between the patch and the commit message (color/colour).
-- Sean McG.
__
- Original Message -
> From: Diego Biurrun
> To: libav development
> Cc:
> Sent: Wednesday, February 29, 2012 9:05 PM
> Subject: Re: [libav-devel] [PATCH] Windows Media Audio Lossless decoder
>
> On Wed, Feb 29, 2012 at 06:38:41AM -0800, Mashiat Sarker Shakkhar wrote:
>> - Orig
On 29/02/12 16:36, Martin Storsjö wrote:
> This avoids breaking fate every time the lavc version is bumped.
> ---
Ok
lu
--
Luca Barbato
Gentoo/linux
http://dev.gentoo.org/~lu_zero
___
libav-devel mailing list
libav-devel@libav.org
https://lists.liba
This avoids breaking fate every time the lavc version is bumped.
---
libavcodec/dpxenc.c |3 ++-
tests/ref/lavf/dpx |2 +-
2 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/libavcodec/dpxenc.c b/libavcodec/dpxenc.c
index c731bb4..89548ba 100644
--- a/libavcodec/dpxenc.c
+++ b
On 2012-02-26 09:52:44 +0100, Vitor Sessak wrote:
> ---
> libavcodec/ra144dec.c |2 ++
> libavcodec/ra288.c|2 ++
> libavcodec/sipr.c |2 ++
> libavcodec/twinvq.c |2 ++
> 4 files changed, 8 insertions(+), 0 deletions(-)
Why?
Have you proofed that each of the decoder ca
On Sun, Feb 26, 2012 at 07:23:50PM +, Paul B Mahol wrote:
>
> --- a/tests/fate/cdxl.mak
> +++ b/tests/fate/cdxl.mak
> @@ -10,5 +10,8 @@ fate-cdxl-pal8: CMD = framecrc -i $(SAMPLES)/cdxl/maku.cdxl
> -pix_fmt rgb24 -fram
> FATE_CDXL += fate-cdxl-pal8-small
> fate-cdxl-pal8-small: CMD = framec
On Wed, Feb 29, 2012 at 06:38:41AM -0800, Mashiat Sarker Shakkhar wrote:
> - Original Message -
>
> > From: Diego Biurrun
> > To: libav development
> > Cc:
> > Sent: Wednesday, February 29, 2012 6:44 PM
> > Subject: Re: [libav-devel] [PATCH] Windows Media Audio Lossless decoder
> >
> >
You are missing a version bump now that the tree is declared stable again.
On Tue, Feb 28, 2012 at 06:20:27PM -0800, Mashiat Sarker Shakkhar wrote:
> --- a/doc/general.texi
> +++ b/doc/general.texi
> @@ -770,6 +770,7 @@ following image formats are supported:
> @item Windows Media Audio 2 @tab X
On Wed, Feb 29, 2012 at 06:38:41AM -0800, Mashiat Sarker Shakkhar wrote:
> - Original Message -
>
> > From: Diego Biurrun
> > To: libav development
> > Cc:
> > Sent: Wednesday, February 29, 2012 6:44 PM
> > Subject: Re: [libav-devel] [PATCH] Windows Media Audio Lossless decoder
> >
> >
On 29/02/12 13:51, Martin Storsjö wrote:
> This fixes cases where the user had specified one desired MTU
> via an option, and the protocol indicates another one.
Looks fine for me =)
lu
--
Luca Barbato
Gentoo/linux
http://dev.gentoo.org/~lu_zero
___
On 29/02/12 15:32, Martin Storsjö wrote:
> On Wed, 29 Feb 2012, Luca Barbato wrote:
>
>> On 29/02/12 14:07, Martin Storsjö wrote:
>>> +{ "mb_info", "byte interval for outputting info about mb
>>> offsets", OFFSET(mb_info), AV_OPT_TYPE_INT, { 0 }, 0, INT_MAX, VE },
>>
>> The description is
- Original Message -
> From: Diego Biurrun
> To: libav development
> Cc:
> Sent: Wednesday, February 29, 2012 6:44 PM
> Subject: Re: [libav-devel] [PATCH] Windows Media Audio Lossless decoder
>
> Please get another mailer, this Yahoo shit is unbearable.
I can get another mailer, but y
On Wed, 29 Feb 2012, Luca Barbato wrote:
On 29/02/12 14:07, Martin Storsjö wrote:
+{ "mb_info", "byte interval for outputting info about mb offsets",
OFFSET(mb_info), AV_OPT_TYPE_INT, { 0 }, 0, INT_MAX, VE },
The description isn't completely clear, beside that seems ok.
Amended th
On 29/02/12 14:07, Martin Storsjö wrote:
> ---
> libavcodec/avcodec.h | 11 +++
> libavcodec/avpacket.c | 16
> 2 files changed, 27 insertions(+), 0 deletions(-)
>
Ok, I guess.
lu
--
Luca Barbato
Gentoo/linux
http://dev.gentoo.org/~lu_zero
_
On 29/02/12 14:07, Martin Storsjö wrote:
> This makes the packetization spec compliant for cases where one single
> GOB doesn't fit into an RTP packet.
> ---
> This is a much simpler version of http://patches.libav.org/patch/17485/.
> The upside is that we don't need to halfway decode the packets i
On 29/02/12 14:07, Martin Storsjö wrote:
> +{ "mb_info", "byte interval for outputting info about mb offsets",
> OFFSET(mb_info), AV_OPT_TYPE_INT, { 0 }, 0, INT_MAX, VE },
The description isn't completely clear, beside that seems ok.
lu
--
Luca Barbato
Gentoo/linux
http://dev.gentoo.
---
libavcodec/avcodec.h | 19 +++
libavcodec/mpegvideo.h |4 +++
libavcodec/mpegvideo_enc.c | 56
3 files changed, 79 insertions(+), 0 deletions(-)
diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h
index af2a5ca..7
This makes the packetization spec compliant for cases where one single
GOB doesn't fit into an RTP packet.
---
This is a much simpler version of http://patches.libav.org/patch/17485/.
The upside is that we don't need to halfway decode the packets in the
packetizer any longer - the downside is that
---
libavcodec/avcodec.h | 11 +++
libavcodec/avpacket.c | 16
2 files changed, 27 insertions(+), 0 deletions(-)
diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h
index f4db083..af2a5ca 100644
--- a/libavcodec/avcodec.h
+++ b/libavcodec/avcodec.h
@@ -3079,6 +3
On Wed, Feb 29, 2012 at 01:44:23PM +0100, Diego Biurrun wrote:
> Please get another mailer, this Yahoo shit is unbearable.
>
> On Tue, Feb 28, 2012 at 06:16:59PM -0800, Mashiat Sarker Shakkhar wrote:
> > >> +static void decode_cdlms(WmallDecodeCtx *s)
> > >
> > > And what is cdlms anyway? You h
On Wed, Feb 29, 2012 at 09:30:15AM +0100, Anton Khirnov wrote:
> ---
> Fixed a few bugs from the previous version -- properly finish the tag
> and write the xing frame even when the user didn't send some pictures;
> really don't write more than one picture per stream.
> ---
> libavformat/mp3enc.c
On Wed, Feb 29, 2012 at 09:30:14AM +0100, Anton Khirnov wrote:
> It will be need by new functions called from mp3_write_trailer().
> ---
> libavformat/mp3enc.c | 94
> ++
> 1 files changed, 49 insertions(+), 45 deletions(-)
should be OK
_
This fixes cases where the user had specified one desired MTU
via an option, and the protocol indicates another one.
---
libavformat/rtpenc.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/libavformat/rtpenc.c b/libavformat/rtpenc.c
index f559eb8..41d5843 100644
--- a/li
1 - 100 of 140 matches
Mail list logo
