[libav-devel] [PATCH] rtmpproto: Ignore errors from the getStreamLength method

2014-11-03 Thread Martin Storsjö
It is never an error if this method failed. If rt->live was explicitly set to 0 (known to be a recorded file), print it as a warning, otherwise print it as a debug message. Based on a patch by Michael Niedermayer. --- libavformat/rtmpproto.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/

Re: [libav-devel] [PATCH 3/4] lavc: add a public API for parsing vorbis packets.

2014-11-03 Thread James Almer
On 29/10/14 12:46 PM, Anton Khirnov wrote: > It is required by (at least) the ogg demuxer. > > Mark the current semi-public apriv API for removal. > --- > doc/APIchanges | 3 +++ > libavcodec/Makefile| 12 ++ > libavcodec/version.h | 4 ++-- > libavcodec/vorbis

Re: [libav-devel] [PATCH] img: Add a step option

2014-11-03 Thread Vittorio Giovara
On Mon, Nov 3, 2014 at 4:20 PM, Luca Barbato wrote: > Useful if you want to pick only the N-th image of a serie. > --- > libavformat/img2dec.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/libavformat/img2dec.c b/libavformat/img2dec.c > index f7f0a11..370ff17 100644

[libav-devel] [PATCH] aac: Simplify decode_mid_side_stereo

2014-11-03 Thread Luca Barbato
Might spare few cycles if the compiler is naive and makes the function more readable. --- libavcodec/aacdec.c | 7 +++ 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/libavcodec/aacdec.c b/libavcodec/aacdec.c index 4bdf52f..47f9b59 100644 --- a/libavcodec/aacdec.c +++ b/libavcode

Re: [libav-devel] [PATCH 11/15] aacdec: change order of multiplications

2014-11-03 Thread Luca Barbato
On 03/11/14 20:42, Vittorio Giovara wrote: On Mon, Nov 3, 2014 at 12:53 PM, Luca Barbato wrote: On 30/10/14 01:52, Vittorio Giovara wrote: From: Michael Niedermayer Make code safer against too large input values. CC: libav-sta...@libav.org Bug-Id: CID 700697 --- libavcodec/aacdec.c | 2

[libav-devel] [PATCH] img: Add a step option

2014-11-03 Thread Luca Barbato
Useful if you want to pick only the N-th image of a serie. --- libavformat/img2dec.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libavformat/img2dec.c b/libavformat/img2dec.c index f7f0a11..370ff17 100644 --- a/libavformat/img2dec.c +++ b/libavformat/img2dec.c @@ -33,6 +

Re: [libav-devel] [PATCH 09/15] ratecontrol: prevent a null pointer dereference in loop

2014-11-03 Thread Luca Barbato
On 03/11/14 20:40, Vittorio Giovara wrote: p can probably become NULL at the end of the loop because of strchr Then ok. ___ libav-devel mailing list libav-devel@libav.org https://lists.libav.org/mailman/listinfo/libav-devel

Re: [libav-devel] [PATCH 05/15] motion_est: avoid out-of-bounds reads in loop index

2014-11-03 Thread Luca Barbato
On 03/11/14 20:41, Vittorio Giovara wrote: On Mon, Nov 3, 2014 at 12:55 PM, Luca Barbato wrote: On 30/10/14 01:51, Vittorio Giovara wrote: if(s->me.dia_size>=2){ const int tl= score_map[(index-(1penalt

Re: [libav-devel] [PATCH 11/15] aacdec: change order of multiplications

2014-11-03 Thread Vittorio Giovara
On Mon, Nov 3, 2014 at 12:53 PM, Luca Barbato wrote: > On 30/10/14 01:52, Vittorio Giovara wrote: >> >> From: Michael Niedermayer >> >> Make code safer against too large input values. >> >> CC: libav-sta...@libav.org >> Bug-Id: CID 700697 >> --- >> libavcodec/aacdec.c | 2 +- >> 1 file changed

Re: [libav-devel] [PATCH 05/15] motion_est: avoid out-of-bounds reads in loop index

2014-11-03 Thread Vittorio Giovara
On Mon, Nov 3, 2014 at 12:55 PM, Luca Barbato wrote: > On 30/10/14 01:51, Vittorio Giovara wrote: >> >> if(s->me.dia_size>=2){ >> const int tl= >> score_map[(index-(1<> const int bl= >> score_map[(index+(1<> @@ -281,7 +281,7 @@ static int qpel_motion_search(Mp

Re: [libav-devel] [PATCH 09/15] ratecontrol: prevent a null pointer dereference in loop

2014-11-03 Thread Vittorio Giovara
On Mon, Nov 3, 2014 at 12:54 PM, Luca Barbato wrote: > On 30/10/14 01:51, Vittorio Giovara wrote: >> >> CC: libav-sta...@libav.org >> Bug-Id: CID 29254 >> --- >> libavcodec/ratecontrol.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/libavcodec/ratecontrol.c b/libav

Re: [libav-devel] win32 Jenkins node

2014-11-03 Thread Luca Barbato
On 03/11/14 20:19, Sean McGovern wrote: Hi folks, There are no less than 3 Bugzilla tickets open for the nightly autobuilds for win32. > Do we have some kind of a time line for when the node will be available again? Within this week, hopefully. The host had been eventually updated to someth

[libav-devel] win32 Jenkins node

2014-11-03 Thread Sean McGovern
Hi folks, There are no less than 3 Bugzilla tickets open for the nightly autobuilds for win32. Do we have some kind of a time line for when the node will be available again? Thanks, Sean McG. ___ libav-devel mailing list libav-devel@libav.org https:

Re: [libav-devel] [PATCH 05/15] motion_est: avoid out-of-bounds reads in loop index

2014-11-03 Thread Luca Barbato
On 30/10/14 01:51, Vittorio Giovara wrote: if(s->me.dia_size>=2){ const int tl= score_map[(index-(1penalty_factor; -for(i=0; i<8; i++){ +for (i = 0; i < FF_ARRAY_ELEM

Re: [libav-devel] [PATCH 09/15] ratecontrol: prevent a null pointer dereference in loop

2014-11-03 Thread Luca Barbato
On 30/10/14 01:51, Vittorio Giovara wrote: CC: libav-sta...@libav.org Bug-Id: CID 29254 --- libavcodec/ratecontrol.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/ratecontrol.c b/libavcodec/ratecontrol.c index e18878d..08f5ccc 100644 --- a/libavcodec/ratecontro

Re: [libav-devel] [PATCH 11/15] aacdec: change order of multiplications

2014-11-03 Thread Luca Barbato
On 30/10/14 01:52, Vittorio Giovara wrote: From: Michael Niedermayer Make code safer against too large input values. CC: libav-sta...@libav.org Bug-Id: CID 700697 --- libavcodec/aacdec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/aacdec.c b/libavcodec/aac

Re: [libav-devel] [PATCH 12/15] aacdec: add default case in channel layout

2014-11-03 Thread Luca Barbato
On 30/10/14 01:52, Vittorio Giovara wrote: Bug-Id: CID 732291 --- libavcodec/aacdec.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavcodec/aacdec.c b/libavcodec/aacdec.c index 498b278..cc74bd6 100644 --- a/libavcodec/aacdec.c +++ b/libavcodec/aacdec.c @@ -624,6 +624,9 @@ static vo

Re: [libav-devel] [PATCH 15/15] aacsbr: change order of operation to prevent out of array read

2014-11-03 Thread Luca Barbato
On 30/10/14 01:52, Vittorio Giovara wrote: From: Michael Niedermayer CC: libav-sta...@libav.org Bug-Id: CID 732250 --- libavcodec/aacsbr.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavcodec/aacsbr.c b/libavcodec/aacsbr.c index 0363aff..76c1014 100644 --- a/libav

Re: [libav-devel] [PATCH 14/15] aacsbr: treat 1-d arrays as such

2014-11-03 Thread Luca Barbato
On 30/10/14 01:52, Vittorio Giovara wrote: CC: libav-sta...@libav.org Bug-Id: CID 1238802 / CID 1238793 / CID 1238785 CID 1238808 / CID 1238809 --- libavcodec/aacsbr.c | 10 -- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/libavcodec/aacsbr.c b/libavcodec/aacsb

Re: [libav-devel] [PATCH 08/15] error_resilience: initialize prev_* variables

2014-11-03 Thread Luca Barbato
On 30/10/14 01:51, Vittorio Giovara wrote: CC: libav-sta...@libav.org Bug-Id: CID 732293 / CID 732294 --- libavcodec/error_resilience.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/error_resilience.c b/libavcodec/error_resilience.c index 33b0360..b1ca2d9 10064

Re: [libav-devel] [PATCH 07/15] celp_filters: don't use filter lenght as loop bound

2014-11-03 Thread Luca Barbato
On 30/10/14 01:51, Vittorio Giovara wrote: Makes Coverity less confused regarding out-of-bounds accesses. CC: libav-sta...@libav.org Bug-Id: CID 717906 / CID 717907 / CID 717916 / CID 717917 / CID 717919 / CID 732259 --- libavcodec/celp_filters.c | 2 +- 1 file changed, 1 insertion(+

Re: [libav-devel] [PATCH 06/15] motion_est: widen before multiply

2014-11-03 Thread Luca Barbato
On 30/10/14 01:51, Vittorio Giovara wrote: From: Michael Niedermayer Bug-Id: CID 700556 / CID 700557 / CID700558 --- libavcodec/x86/me_cmp_init.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavcodec/x86/me_cmp_init.c b/libavcodec/x86/me_cmp_init.c index e93b67

Re: [libav-devel] [PATCH 3/4] lavc: add a public API for parsing vorbis packets.

2014-11-03 Thread Anton Khirnov
Quoting wm4 (2014-11-03 16:33:07) > On Mon, 03 Nov 2014 12:32:13 +0100 > Anton Khirnov wrote: > > > Quoting Luca Barbato (2014-10-29 16:55:31) > > > On 29/10/14 16:46, Anton Khirnov wrote: > > > > It is required by (at least) the ogg demuxer. > > > > > > > > Mark the current semi-public apriv API

Re: [libav-devel] [PATCH 3/4] lavc: add a public API for parsing vorbis packets.

2014-11-03 Thread wm4
On Mon, 03 Nov 2014 12:32:13 +0100 Anton Khirnov wrote: > Quoting Luca Barbato (2014-10-29 16:55:31) > > On 29/10/14 16:46, Anton Khirnov wrote: > > > It is required by (at least) the ogg demuxer. > > > > > > Mark the current semi-public apriv API for removal. > > > --- > > > doc/APIchanges

Re: [libav-devel] [PATCH] h264_parser: don't stop on SPS_EXT in split

2014-11-03 Thread Vittorio Giovara
On Fri, Oct 31, 2014 at 11:24 AM, John Stebbins wrote: > Add SPS_EXT, SEI, and subset SPS to codes that are skipped during split. > These codes can come before the PPS and results in incomplete extradata. > --- > libavcodec/h264_parser.c | 6 -- > 1 file changed, 4 insertions(+), 2 deletions(

Re: [libav-devel] [PATCH 4/4] lavc, lavf: switch to the new vorbis parse API

2014-11-03 Thread Luca Barbato
On 29/10/14 16:47, Anton Khirnov wrote: --- libavcodec/libvorbis.c | 11 +++ libavcodec/vorbis_parser.c | 24 +++- libavformat/oggparsevorbis.c | 20 3 files changed, 38 insertions(+), 17 deletions(-) Seems fine. __

Re: [libav-devel] [PATCH 3/4] lavc: add a public API for parsing vorbis packets.

2014-11-03 Thread Luca Barbato
On 03/11/14 12:32, Anton Khirnov wrote: Quoting Luca Barbato (2014-10-29 16:55:31) On 29/10/14 16:46, Anton Khirnov wrote: It is required by (at least) the ogg demuxer. Mark the current semi-public apriv API for removal. --- doc/APIchanges | 3 +++ libavcodec/Makefile

Re: [libav-devel] [PATCH 3/4] lavc: add a public API for parsing vorbis packets.

2014-11-03 Thread Anton Khirnov
Quoting Luca Barbato (2014-10-29 16:55:31) > On 29/10/14 16:46, Anton Khirnov wrote: > > It is required by (at least) the ogg demuxer. > > > > Mark the current semi-public apriv API for removal. > > --- > > doc/APIchanges | 3 +++ > > libavcodec/Makefile| 12 ++ > >