Re: [libav-devel] [PATCH] tests: Add ATRAC3plus tests

2015-04-24 Thread Diego Biurrun
On Fri, Apr 24, 2015 at 04:48:53PM +, Timothy Gu wrote: > On Thu, Apr 23, 2015 at 5:27 AM Diego Biurrun wrote: > > > On Sun, Apr 19, 2015 at 08:26:15PM -0700, Timothy Gu wrote: > > > 12:25 <+ Timothy_Gu> DonDiego: the first one has a higher bitrate, > > the > > >

Re: [libav-devel] [FFmpeg-devel] [PATCH] alac: validate k before using show_bits in decode_scalar

2015-04-24 Thread Luca Barbato
On 24/04/15 18:09, Andreas Cadhalpun wrote: > On 24.04.2015 11:10, Diego Biurrun wrote: >> On Thu, Apr 23, 2015 at 08:53:29PM +0200, Andreas Cadhalpun wrote: >>> --- a/libavcodec/alac.c >>> +++ b/libavcodec/alac.c >>> @@ -569,6 +569,10 @@ static int alac_set_info(ALACContext *alac) >>> alac->r

Re: [libav-devel] [PATCH] tests: Add ATRAC3plus tests

2015-04-24 Thread Timothy Gu
On Thu, Apr 23, 2015 at 5:27 AM Diego Biurrun wrote: > On Sun, Apr 19, 2015 at 08:26:15PM -0700, Timothy Gu wrote: > > 12:25 <+ Timothy_Gu> DonDiego: the first one has a higher bitrate, > the > > second one has a lower one. From the coverage > report > >

Re: [libav-devel] [FFmpeg-devel] [PATCH] alac: validate k before using show_bits in decode_scalar

2015-04-24 Thread Andreas Cadhalpun
On 24.04.2015 11:10, Diego Biurrun wrote: > On Thu, Apr 23, 2015 at 08:53:29PM +0200, Andreas Cadhalpun wrote: >> --- a/libavcodec/alac.c >> +++ b/libavcodec/alac.c >> @@ -569,6 +569,10 @@ static int alac_set_info(ALACContext *alac) >> alac->rice_history_mult= bytestream2_get_byteu(&gb); >

Re: [libav-devel] [PATCH 1/2] jpeglsenc: Check memory allocations

2015-04-24 Thread Luca Barbato
On 24/04/15 17:59, Vittorio Giovara wrote: > Convert exisiting free functions to av_freep() to avoid accidentaly > double free one, and always intialize all buffers to NULL. > --- > libavcodec/jpeglsenc.c | 29 +++-- > 1 file changed, 23 insertions(+), 6 deletions(-) > > d

[libav-devel] [PATCH 1/2] jpeglsenc: Check memory allocations

2015-04-24 Thread Vittorio Giovara
Convert exisiting free functions to av_freep() to avoid accidentaly double free one, and always intialize all buffers to NULL. --- libavcodec/jpeglsenc.c | 29 +++-- 1 file changed, 23 insertions(+), 6 deletions(-) diff --git a/libavcodec/jpeglsenc.c b/libavcodec/jpeglsenc

[libav-devel] [PATCH 2/2] jpeglsenc: Mark codec as init-thread-safe and init-cleanup

2015-04-24 Thread Vittorio Giovara
Fixes a memory leak on error init. --- libavcodec/jpeglsenc.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/jpeglsenc.c b/libavcodec/jpeglsenc.c index 2a24a5d..ac89436 100644 --- a/libavcodec/jpeglsenc.c +++ b/libavcodec/jpeglsenc.c @@ -452,4 +452,6 @@ AVCodec ff_jpegls_encoder

Re: [libav-devel] [PATCH 8/8] lavc: check memory allocations

2015-04-24 Thread Vittorio Giovara
On Thu, Apr 23, 2015 at 3:20 PM, Vittorio Giovara wrote: >>> --- a/libavcodec/eatgv.c >>> +++ b/libavcodec/eatgv.c >>> @@ -174,12 +174,15 @@ static int tgv_decode_inter(TgvContext *s, AVFrame >>> *frame, >>> /* allocate codebook buffers as necessary */ >>> if (num_mvs > s->num_mvs) { >>

Re: [libav-devel] [PATCH] rtsp: Make sure we don't write too many transport entries into a fixed-size array

2015-04-24 Thread Vittorio Giovara
On Fri, Apr 24, 2015 at 10:39 AM, Martin Storsjö wrote: > CC: libav-sta...@libav.org > --- > libavformat/rtsp.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/libavformat/rtsp.c b/libavformat/rtsp.c > index 1b60b4f..cbea685 100644 > --- a/libavformat/rtsp.c > +++ b/libavformat/rtsp.c

[libav-devel] [PATCH] rtsp: Make sure we don't write too many transport entries into a fixed-size array

2015-04-24 Thread Martin Storsjö
CC: libav-sta...@libav.org --- libavformat/rtsp.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavformat/rtsp.c b/libavformat/rtsp.c index 1b60b4f..cbea685 100644 --- a/libavformat/rtsp.c +++ b/libavformat/rtsp.c @@ -972,6 +972,8 @@ static void rtsp_parse_transport(RTSPMessageHeader *re

Re: [libav-devel] [FFmpeg-devel] [PATCH] alac: validate k before using show_bits in decode_scalar

2015-04-24 Thread Diego Biurrun
On Thu, Apr 23, 2015 at 08:53:29PM +0200, Andreas Cadhalpun wrote: > --- a/libavcodec/alac.c > +++ b/libavcodec/alac.c > @@ -569,6 +569,10 @@ static int alac_set_info(ALACContext *alac) > alac->rice_history_mult= bytestream2_get_byteu(&gb); > alac->rice_initial_history = bytestream2_g