Re: [libav-devel] [PATCH 6/6] fmtconvert: port float_to_int16_interleave() 2-channel x86 inline asm to yasm

2011-10-10 Thread Jason Garrett-Glaser
On Sun, Oct 9, 2011 at 9:48 PM, Justin Ruggles justin.rugg...@gmail.com wrote: ---  libavcodec/x86/fmtconvert.asm   |   54 ++  libavcodec/x86/fmtconvert_mmx.c |   69 +-  2 files changed, 63 insertions(+), 60 deletions(-) diff

Re: [libav-devel] [PATCH 1/3] new interrupt callback interface, that works per URLContext instead of globally

2011-10-10 Thread Luca Barbato
On 10/9/11 7:40 PM, Nicolas George wrote: L'octidi 18 vendémiaire, an CCXX, aviad rozenhek a écrit : I think the proper way to solve this is to explicitly model nested URLContexts in the API. in this way, the demuxer could ask the inner-most URLContext if it should be aborted Luca and me

[libav-devel] [PATCH] configure: Add -Wstrict-prototypes to CFLAGS if available.

2011-10-10 Thread Diego Biurrun
--- configure |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/configure b/configure index 22735e7..f5a285e 100755 --- a/configure +++ b/configure @@ -2992,6 +2992,7 @@ check_cflags -Wwrite-strings check_cflags -Wtype-limits check_cflags -Wundef check_cflags

Re: [libav-devel] [PATCH 1/6] fmtconvert: fix and extend documentation for float_interleave()

2011-10-10 Thread Diego Biurrun
On Mon, Oct 10, 2011 at 12:48:14AM -0400, Justin Ruggles wrote: --- a/libavcodec/fmtconvert.h +++ b/libavcodec/fmtconvert.h @@ -70,7 +70,15 @@ typedef struct FmtConvertContext { /** - * Convert an array of interleaved float to multiple arrays of float. + * Convert multiple

Re: [libav-devel] [PATCH 3/6] fmtconvert: port float_to_int16() x86 inline asm to yasm

2011-10-10 Thread Måns Rullgård
Justin Ruggles justin.rugg...@gmail.com writes: --- libavcodec/x86/fmtconvert.asm | 42 ++ libavcodec/x86/fmtconvert_mmx.c | 89 ++- 2 files changed, 55 insertions(+), 76 deletions(-) \o/ (This is not a patch review) -- Måns

Re: [libav-devel] [PATCH] configure: Add -Wstrict-prototypes to CFLAGS if available.

2011-10-10 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: --- configure |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/configure b/configure index 22735e7..f5a285e 100755 --- a/configure +++ b/configure @@ -2992,6 +2992,7 @@ check_cflags -Wwrite-strings check_cflags

[libav-devel] patch to libavformat/mov.c, metadata various

2011-10-10 Thread Raivo Hool
Hello! I’ve stumbled across a few metadata issues with both ffmpeg and libav that I think I’ve managed to fix. The issues fixed: 1. Tracktotal not reported. 2. Disctotal not reported. 3. Album artist not reported (the ‘aALB' atom). Tracktotal and disctotal are represented as XX/XX, which is

Re: [libav-devel] [PATCH v2] avidec: fix signed overflow in avi_sync()

2011-10-10 Thread Janne Grunau
On Sun, Oct 09, 2011 at 01:07:09PM +0100, Mans Rullgard wrote: Keeping byte values read from the file as unsigned is consistent with how they are subsequently used and avoids an undefined left shift by 24 when bit 7 is set. Signed-off-by: Mans Rullgard m...@mansr.com ---

[libav-devel] [PATCH] avidec: simplify size parameter in memset()

2011-10-10 Thread Janne Grunau
On Mon, Oct 10, 2011 at 12:23:04PM +0100, Måns Rullgård wrote: Janne Grunau janne-li...@jannau.net writes: please fix the ugly memset(d, -1, sizeof(int)*8); while you're at it I refuse. I thought we agreed to have no more patch ok if you fix everything else first reviews. Holding a fix

Re: [libav-devel] [PATCH] avidec: simplify size parameter in memset()

2011-10-10 Thread Måns Rullgård
Janne Grunau janne-li...@jannau.net writes: libavformat/avidec.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/libavformat/avidec.c b/libavformat/avidec.c index 8c0ef13..81f72d3 100644 --- a/libavformat/avidec.c +++ b/libavformat/avidec.c @@ -848,7 +848,7 @@

Re: [libav-devel] [PATCH v2] avidec: fix signed overflow in avi_sync()

2011-10-10 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: On Mon, Oct 10, 2011 at 12:23:04PM +0100, Måns Rullgård wrote: Janne Grunau janne-li...@jannau.net writes: On Sun, Oct 09, 2011 at 01:07:09PM +0100, Mans Rullgard wrote: Keeping byte values read from the file as unsigned is consistent with how they

Re: [libav-devel] [PATCH] vaapi: fix VC-1 decoding (reconstruct bitstream TTFRM correctly).

2011-10-10 Thread Diego Biurrun
On Mon, Oct 10, 2011 at 01:59:35PM +0200, Kostya Shishkov wrote: On 10 October 2011 13:51, Gwenole Beauchesne gwenole.beauche...@intel.com wrote: --- a/libavcodec/vaapi_vc1.c +++ b/libavcodec/vaapi_vc1.c @@ -116,6 +116,18 @@ static inline VAMvModeVC1 vc1_get_MVMODE2(VC1Context *v)

[libav-devel] [PATCH 6/6] fmtconvert: port float_to_int16_interleave() 2-channel x86 inline asm to yasm

2011-10-10 Thread Justin Ruggles
--- libavcodec/x86/fmtconvert.asm | 53 ++ libavcodec/x86/fmtconvert_mmx.c | 69 +- 2 files changed, 62 insertions(+), 60 deletions(-) diff --git a/libavcodec/x86/fmtconvert.asm b/libavcodec/x86/fmtconvert.asm index

Re: [libav-devel] [PATCH 6/6] fmtconvert: port float_to_int16_interleave() 2-channel x86 inline asm to yasm

2011-10-10 Thread Loren Merritt
On Mon, 10 Oct 2011, Justin Ruggles wrote: +cglobal float_to_int16_interleave2_%1, 2,3,%2, dst, src0, src1 +%define lenq r3q cglobal float_to_int16_interleave2_%1, 3,4,0, dst, src0, src1, len --Loren Merritt ___ libav-devel mailing list

[libav-devel] [PATCH] mpegts: Attempt to seek back to beginning in non-seekable mpegts files.

2011-10-10 Thread Tomas Härdin
Hi Attached patch fixes demuxing/probing of http://titan.codemill.se/~tomhar/samples/not_playable_on_pipe.ts when opened via a pipe (ffplay/avplay - not_playable_on_pipe.ts). Sent to both lists since the problem exists in both projects. Meh. Passes make fate on both as well. /Tomas From

[libav-devel] [PATCH] id2v2: fix parameterless function pointer declaration

2011-10-10 Thread Diego Biurrun
--- libavformat/id3v2.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/libavformat/id3v2.h b/libavformat/id3v2.h index 13dec39..7ac7a91 100644 --- a/libavformat/id3v2.h +++ b/libavformat/id3v2.h @@ -63,7 +63,7 @@ typedef struct ID3v2EMFunc { const char *tag3;

[libav-devel] [PATCH] id3v2: reduce the scope of some non-globally-used symbols/structures

2011-10-10 Thread Diego Biurrun
--- libavformat/id3v2.c | 137 +++ libavformat/id3v2.h | 10 2 files changed, 72 insertions(+), 75 deletions(-) diff --git a/libavformat/id3v2.c b/libavformat/id3v2.c index c30ab4c..f703742 100644 --- a/libavformat/id3v2.c +++

Re: [libav-devel] [PATCH] id3v2: reduce the scope of some non-globally-used symbols/structures

2011-10-10 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: --- libavformat/id3v2.c | 137 +++ libavformat/id3v2.h | 10 2 files changed, 72 insertions(+), 75 deletions(-) diff --git a/libavformat/id3v2.c b/libavformat/id3v2.c index c30ab4c..f703742

Re: [libav-devel] [PATCH] mpegts: Attempt to seek back to beginning in non-seekable mpegts files.

2011-10-10 Thread Luca Barbato
On 10/10/11 2:34 PM, Tomas Härdin wrote: Hi Attached patch fixes demuxing/probing of http://titan.codemill.se/~tomhar/samples/not_playable_on_pipe.ts when opened via a pipe (ffplay/avplay - not_playable_on_pipe.ts). Sent to both lists since the problem exists in both projects. Meh. Passes make

Re: [libav-devel] [PATCH] id3v2: reduce the scope of some non-globally-used symbols/structures

2011-10-10 Thread Diego Biurrun
On Mon, Oct 10, 2011 at 01:51:42PM +0100, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: --- a/libavformat/id3v2.c +++ b/libavformat/id3v2.c @@ -26,6 +26,66 @@ #include libavutil/dict.h #include avio_internal.h +const AVMetadataConv ff_id3v2_34_metadata_conv[] = {

Re: [libav-devel] [PATCH] id3v2: reduce the scope of some non-globally-used symbols/structures

2011-10-10 Thread Måns Rullgård
Diego Biurrun di...@biurrun.de writes: On Mon, Oct 10, 2011 at 01:51:42PM +0100, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: --- a/libavformat/id3v2.c +++ b/libavformat/id3v2.c @@ -26,6 +26,66 @@ #include libavutil/dict.h #include avio_internal.h +const

Re: [libav-devel] patch to libavformat/mov.c, metadata various

2011-10-10 Thread Luca Barbato
On 10/10/11 12:07 PM, Raivo Hool wrote: Hello! I’ve stumbled across a few metadata issues with both ffmpeg and libav that I think I’ve managed to fix. The issues fixed: 1. Tracktotal not reported. 2. Disctotal not reported. 3. Album artist not reported (the ‘aALB' atom). Tracktotal and

Re: [libav-devel] [PATCH] id3v2: reduce the scope of some non-globally-used symbols/structures

2011-10-10 Thread Diego Biurrun
On Mon, Oct 10, 2011 at 02:09:01PM +0100, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: On Mon, Oct 10, 2011 at 01:51:42PM +0100, Måns Rullgård wrote: Diego Biurrun di...@biurrun.de writes: --- a/libavformat/id3v2.c +++ b/libavformat/id3v2.c @@ -26,6 +26,66 @@

[libav-devel] [PATCH] id2v2: fix parameterless function pointer declaration

2011-10-10 Thread Diego Biurrun
--- libavformat/id3v2.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/libavformat/id3v2.h b/libavformat/id3v2.h index 13dec39..9f7ea03 100644 --- a/libavformat/id3v2.h +++ b/libavformat/id3v2.h @@ -63,7 +63,7 @@ typedef struct ID3v2EMFunc { const char *tag3;

[libav-devel] [PATCH 1/2] id3v2: cosmetics: move some declarations before the places they are used

2011-10-10 Thread Diego Biurrun
--- libavformat/id3v2.c | 130 +- 1 files changed, 65 insertions(+), 65 deletions(-) diff --git a/libavformat/id3v2.c b/libavformat/id3v2.c index c30ab4c..cda02c6 100644 --- a/libavformat/id3v2.c +++ b/libavformat/id3v2.c @@ -26,6 +26,66 @@

[libav-devel] [PATCH 3/4] Fixed MOV/MP4 album_artist metadata

2011-10-10 Thread Raivo Hool
--- libavformat/mov.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 9893367..8371728 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -165,6 +165,7 @@ static int mov_read_udta_string(MOVContext *c, AVIOContext *pb,

[libav-devel] [PATCH 1/4] Fixed MOV/MP4 tracktotal metadata

2011-10-10 Thread Raivo Hool
--- libavformat/mov.c | 13 - 1 files changed, 8 insertions(+), 5 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 3190afd..084d34f 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -81,15 +81,18 @@ typedef struct MOVParseTableEntry { static const

[libav-devel] [PATCH 2/4] Fixed MOV/MP4 disc and tracktotal metadata

2011-10-10 Thread Raivo Hool
--- libavformat/mov.c | 19 +++ 1 files changed, 19 insertions(+), 0 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 084d34f..9893367 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -97,6 +97,22 @@ static int mov_metadata_trkn(MOVContext *c,

[libav-devel] [PATCH 4/4] Added support for tves, tvsn and stik atoms

2011-10-10 Thread Raivo Hool
From: Ace Jones ace.jon...@yahoo.com --- libavformat/mov.c | 40 +--- 1 files changed, 37 insertions(+), 3 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 8371728..dcd5903 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@

Re: [libav-devel] [PATCH 1/4] Fixed MOV/MP4 tracktotal metadata

2011-10-10 Thread Martin Storsjö
On Mon, 10 Oct 2011, Raivo Hool wrote: --- libavformat/mov.c | 13 - 1 files changed, 8 insertions(+), 5 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 3190afd..084d34f 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -81,15 +81,18 @@ typedef struct

Re: [libav-devel] [PATCH] id2v2: fix parameterless function pointer declaration

2011-10-10 Thread Diego Biurrun
On Mon, Oct 10, 2011 at 03:37:10PM +0200, Anton Khirnov wrote: On Mon, 10 Oct 2011 14:37:45 +0200, Diego Biurrun di...@biurrun.de wrote: --- libavformat/id3v2.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/libavformat/id3v2.h b/libavformat/id3v2.h index

[libav-devel] [PATCH 4/4] smacker: validate channels and sample format.

2011-10-10 Thread Justin Ruggles
--- libavcodec/smacker.c |8 1 files changed, 8 insertions(+), 0 deletions(-) diff --git a/libavcodec/smacker.c b/libavcodec/smacker.c index 20ddc15..d8a3d77 100644 --- a/libavcodec/smacker.c +++ b/libavcodec/smacker.c @@ -606,6 +606,14 @@ static int smka_decode_frame(AVCodecContext

Re: [libav-devel] [PATCH 4/5] adpcmdec: check remaining buffer size before decoding next block in the ADPCM IMA WAV decoder.

2011-10-10 Thread Justin Ruggles
On 10/04/2011 06:02 PM, Justin Ruggles wrote: --- libavcodec/adpcm.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/libavcodec/adpcm.c b/libavcodec/adpcm.c index 023805b..c26c515 100644 --- a/libavcodec/adpcm.c +++ b/libavcodec/adpcm.c @@ -431,7 +431,7 @@

Re: [libav-devel] [PATCH] id2v2: fix parameterless function pointer declaration

2011-10-10 Thread Anton Khirnov
On Mon, 10 Oct 2011 18:02:31 +0200, Diego Biurrun di...@biurrun.de wrote: On Mon, Oct 10, 2011 at 03:37:10PM +0200, Anton Khirnov wrote: On Mon, 10 Oct 2011 14:37:45 +0200, Diego Biurrun di...@biurrun.de wrote: --- libavformat/id3v2.h |2 +- 1 files changed, 1 insertions(+), 1

Re: [libav-devel] [PATCH 5/5] adpcmdec: calculate actual number of output samples for each decoder.

2011-10-10 Thread Justin Ruggles
On 10/04/2011 06:02 PM, Justin Ruggles wrote: This also allows for removing some of the buf_size checks and using the sample count for some of the decoding loops. --- libavcodec/adpcm.c | 352 +++- 1 files changed, 235 insertions(+), 117

Re: [libav-devel] [PATCH v2] avidec: fix signed overflow in avi_sync()

2011-10-10 Thread Alex Converse
On Sun, Oct 9, 2011 at 5:07 AM, Mans Rullgard m...@mansr.com wrote: Keeping byte values read from the file as unsigned is consistent with how they are subsequently used and avoids an undefined left shift by 24 when bit 7 is set. Looks good ___

Re: [libav-devel] [PATCH 6/6] fmtconvert: port float_to_int16_interleave() 2-channel x86 inline asm to yasm

2011-10-10 Thread Loren Merritt
On Mon, 10 Oct 2011, Justin Ruggles wrote: On 10/10/2011 08:31 AM, Loren Merritt wrote: On Mon, 10 Oct 2011, Justin Ruggles wrote: +cglobal float_to_int16_interleave2_%1, 2,3,%2, dst, src0, src1 +%define lenq r3q cglobal float_to_int16_interleave2_%1, 3,4,0, dst, src0, src1, len I

Re: [libav-devel] [PATCH 01/15] AVOptions: split av_set_string3 into opt type-specific functions

2011-10-10 Thread Luca Barbato
On 10/6/11 10:10 PM, Anton Khirnov wrote: Also stop attempting to honor the alloc parameter, as things break horribly in that case. It will be removed in upcoming successor to av_set_string3. Seems ok, I'd print a warning on alloc though. lu ___

Re: [libav-devel] [PATCH 02/15] AVOptions: cosmetics, move and rename static av_set_number().

2011-10-10 Thread Luca Barbato
On 10/6/11 10:10 PM, Anton Khirnov wrote: --- libavutil/opt.c | 24 1 files changed, 12 insertions(+), 12 deletions(-) Ok ___ libav-devel mailing list libav-devel@libav.org

Re: [libav-devel] [PATCH 03/15] AVOptions: cosmetics, rename static av_set_number2() to write_number().

2011-10-10 Thread Luca Barbato
On 10/6/11 10:10 PM, Anton Khirnov wrote: --- libavutil/opt.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) Ok ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH 04/15] AVOptions: refactor set_number/write_number

2011-10-10 Thread Luca Barbato
On 10/6/11 10:10 PM, Anton Khirnov wrote: write_number() does the actual writing of the supplied number to destination. Move finding the option and choosing destination address out of it. --- Ok ___ libav-devel mailing list libav-devel@libav.org

Re: [libav-devel] [PATCH 1/4] Fixed MOV/MP4 tracktotal metadata

2011-10-10 Thread Anton Khirnov
On Mon, 10 Oct 2011 18:36:24 +0300, Raivo Hool raivo.h...@gmail.com wrote: --- libavformat/mov.c | 13 - 1 files changed, 8 insertions(+), 5 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 3190afd..084d34f 100644 --- a/libavformat/mov.c +++

Re: [libav-devel] [PATCH 3/4] Fixed MOV/MP4 album_artist metadata

2011-10-10 Thread Anton Khirnov
On Mon, 10 Oct 2011 18:36:26 +0300, Raivo Hool raivo.h...@gmail.com wrote: --- libavformat/mov.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 9893367..8371728 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c

Re: [libav-devel] [PATCH 2/4] Fixed MOV/MP4 disc and tracktotal metadata

2011-10-10 Thread Anton Khirnov
On Mon, 10 Oct 2011 18:36:25 +0300, Raivo Hool raivo.h...@gmail.com wrote: --- libavformat/mov.c | 19 +++ 1 files changed, 19 insertions(+), 0 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 084d34f..9893367 100644 --- a/libavformat/mov.c +++

Re: [libav-devel] [PATCH] mpc8: Check out of bound bands limit

2011-10-10 Thread Laurent Aimar
On Fri, Oct 07, 2011 at 05:45:56PM +0200, Janne Grunau wrote: On Fri, Oct 07, 2011 at 05:22:24PM +0200, Laurent Aimar wrote: On Fri, Oct 07, 2011 at 05:08:20PM +0200, Janne Grunau wrote: From: Laurent Aimar fen...@videolan.org On Fri, Oct 07, 2011 at 03:46:14PM +0200, Kostya Shishkov

Re: [libav-devel] [PATCH 4/4] Added support for tves, tvsn and stik atoms

2011-10-10 Thread Anton Khirnov
On Mon, 10 Oct 2011 18:36:27 +0300, Raivo Hool raivo.h...@gmail.com wrote: From: Ace Jones ace.jon...@yahoo.com --- libavformat/mov.c | 40 +--- 1 files changed, 37 insertions(+), 3 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c

Re: [libav-devel] [PATCH 16/22] 4xm: reject video with non multiple of 16 width/height

2011-10-10 Thread Laurent Aimar
On Sat, Oct 08, 2011 at 06:29:21PM +0200, Kostya Shishkov wrote: On Sat, Oct 08, 2011 at 06:01:26PM +0200, Janne Grunau wrote: From: Laurent Aimar fen...@videolan.org Signed-off-by: Janne Grunau janne-li...@jannau.net --- libavcodec/4xm.c |4 1 files changed, 4

Re: [libav-devel] [PATCH 2/2] libx264: support yuv422/444 output.

2011-10-10 Thread Kieran Kunhya
Looks ok. Maybe worth checking that 10-bit data isn't being encoded by a 9-bit encoder but that's a bit of a theoretical discussion. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH 4/4] Added support for tves, tvsn and stik atoms

2011-10-10 Thread Raivo Hool
On 10.10.2011, at 21:12, Anton Khirnov wrote: +parse = mov_metadata_int8; break; +case MKTAG( 's','t','i','k'): key = stik; What does stik mean? Content type. Movie, Normal, TV Show“ etc. Raivo ___ libav-devel mailing list

Re: [libav-devel] [PATCH] id3v2: fix NULL pointer dereference

2011-10-10 Thread Anton Khirnov
On Fri, 7 Oct 2011 20:14:37 +0200, Anton Khirnov an...@khirnov.net wrote: Bug found by Laurent Aimar fenrir at videolan org --- I'd prefer seomthing like this. --- libavformat/id3v2.c | 12 ++-- 1 files changed, 6 insertions(+), 6 deletions(-) ping. -- Anton Khirnov

Re: [libav-devel] [PATCH 2/2] libx264: support yuv422/444 output.

2011-10-10 Thread Anton Khirnov
On Mon, 10 Oct 2011 19:18:44 +0100, Kieran Kunhya kier...@ob-encoder.com wrote: Looks ok. Maybe worth checking that 10-bit data isn't being encoded by a 9-bit encoder but that's a bit of a theoretical discussion. See X264_init_static(). -- Anton Khirnov

Re: [libav-devel] [PATCH 11/22] ptx: check for out of bound reads

2011-10-10 Thread Janne Grunau
On Sat, Oct 08, 2011 at 06:23:01PM +0200, Kostya Shishkov wrote: On Sat, Oct 08, 2011 at 06:01:21PM +0200, Janne Grunau wrote: From: Laurent Aimar fen...@videolan.org Signed-off-by: Janne Grunau janne-li...@jannau.net --- libavcodec/ptx.c |8 1 files changed, 8

Re: [libav-devel] [PATCH 11/22] ptx: check for out of bound reads

2011-10-10 Thread Ronald S. Bultje
Hi, On Mon, Oct 10, 2011 at 12:29 PM, Janne Grunau janne-li...@jannau.net wrote: On Sat, Oct 08, 2011 at 06:23:01PM +0200, Kostya Shishkov wrote: On Sat, Oct 08, 2011 at 06:01:21PM +0200, Janne Grunau wrote: From: Laurent Aimar fen...@videolan.org Signed-off-by: Janne Grunau

[libav-devel] [PATCH] sipr: fix get_bits(0) calls

2011-10-10 Thread Mans Rullgard
Zero-length get_bits() is undefined, must check before calling. Signed-off-by: Mans Rullgard m...@mansr.com --- libavcodec/sipr.c |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/libavcodec/sipr.c b/libavcodec/sipr.c index f57ec4f..08dd63a 100644 ---

Re: [libav-devel] [PATCH 4/4] smacker: validate channels and sample format.

2011-10-10 Thread Kostya Shishkov
On Mon, Oct 10, 2011 at 12:05:38PM -0400, Justin Ruggles wrote: --- libavcodec/smacker.c |8 1 files changed, 8 insertions(+), 0 deletions(-) diff --git a/libavcodec/smacker.c b/libavcodec/smacker.c index 20ddc15..d8a3d77 100644 --- a/libavcodec/smacker.c +++

Re: [libav-devel] [PATCH] sipr: fix get_bits(0) calls

2011-10-10 Thread Janne Grunau
On Mon, Oct 10, 2011 at 08:41:40PM +0100, Mans Rullgard wrote: Zero-length get_bits() is undefined, must check before calling. Signed-off-by: Mans Rullgard m...@mansr.com --- libavcodec/sipr.c |6 -- 1 files changed, 4 insertions(+), 2 deletions(-) diff --git a/libavcodec/sipr.c

Re: [libav-devel] [PATCH 06/15] AVOptions: add av_opt_set*().

2011-10-10 Thread Luca Barbato
On 10/6/11 10:10 PM, Anton Khirnov wrote: Deprecate av_set_* New functions support setting values on children, return error codes instead of options and have consistent naming and signatures. Ok ___ libav-devel mailing list libav-devel@libav.org

Re: [libav-devel] [PATCH 07/15] AVOptions: add av_opt_get*, deprecate av_get*.

2011-10-10 Thread Luca Barbato
On 10/6/11 10:10 PM, Anton Khirnov wrote: New functions can get values from child objects, properly report error codes and have consistent naming and signatures. --- Ok ___ libav-devel mailing list libav-devel@libav.org

Re: [libav-devel] [PATCH 02/22] anmdemux: prevent double free on error

2011-10-10 Thread Luca Barbato
On 10/8/11 6:01 PM, Janne Grunau wrote: From: Laurent Aimarfen...@videolan.org Signed-off-by: Janne Grunaujanne-li...@jannau.net --- uhm I wonder why that code was written that way from start... lu ___ libav-devel mailing list

[libav-devel] RGB H.264 encoding and decoding, swscale GBR24P

2011-10-10 Thread Hans-Kristian Arntzen
There has been some work in FFmpeg for RGB H.264 along with GBR24P swscale support. There were however some merge conflicts which I attempted to solve. swscale will not work as is, as some function definitions break. GBR24P - RGB is missing as well, which would be neat to have. From

Re: [libav-devel] RGB H.264 encoding and decoding, swscale GBR24P

2011-10-10 Thread Ronald S. Bultje
Hi, On Mon, Oct 10, 2011 at 3:04 PM, Hans-Kristian Arntzen mais...@archlinux.us wrote: There has been some work in FFmpeg for RGB H.264 along with GBR24P swscale support. There were however some merge conflicts which I attempted to solve. swscale will not work as is, as some function