[FFmpeg-devel] [PATCH] configure: replace arch loongson with arch extra list loongson

2015-05-05 Thread 周晓勇
>From a5031b4c4b97f790a40603cff9a1f45cbb043005 Mon Sep 17 00:00:00 2001 From: ZhouXiaoyong Date: Wed, 6 May 2015 14:05:21 +0800 Subject: [PATCH] configure: replace arch loongson with arch extra list loongson fate pass when do configure without --cc='ccache gcc' option: ./configure --enable-gpl --

[FFmpeg-devel] OS/2:Makedef.cmd cleanup

2015-05-05 Thread Dave Yeo
>From 448cf672d7793e0d5166d66c7daffacdc8a3af72 Mon Sep 17 00:00:00 2001 From: Dave Yeo Date: Tue, 5 May 2015 20:22:51 -0700 Subject: [PATCH] OS/2:Makedef.cmd cleanup Remove PROTMODE as it doesn't make sense for DLLs. Also fixes a warning with the OpenWatcom linker Export symbols as names rather

Re: [FFmpeg-devel] [PATCH] diracdec: check if reference could not be allocated

2015-05-05 Thread Michael Niedermayer
On Tue, May 05, 2015 at 11:51:48PM +0200, Andreas Cadhalpun wrote: > s->ref_pics[i] is later used as ref argument of interpolate_refplane, > where it is dereferenced. > > If it is NULL, it causes a segmentation fault. > > Signed-off-by: Andreas Cadhalpun > --- > libavcodec/diracdec.c | 6 ++

Re: [FFmpeg-devel] [PATCH] diracdec: check that block length is large enough

2015-05-05 Thread Michael Niedermayer
On Tue, May 05, 2015 at 10:39:50PM +0200, Andreas Cadhalpun wrote: > In init_planes p->xblen and p->yblen are set to: > p->xblen = s->plane[0].xblen >> s->chroma_x_shift; > p->yblen = s->plane[0].yblen >> s->chroma_y_shift; > > These are later used as block_w and block_h ar

Re: [FFmpeg-devel] [PATCH] diracdec: avoid overflow of bytes*8 in decode_lowdelay

2015-05-05 Thread Michael Niedermayer
On Tue, May 05, 2015 at 10:10:44PM +0200, Andreas Cadhalpun wrote: > If bytes is large enough, bytes*8 can overflow and become negative. > > In that case 'bufsize -= bytes*8' causes bufsize to increase instead of > decrease. > > This leads to a segmentation fault. > > Signed-off-by: Andreas Cadh

Re: [FFmpeg-devel] [PATCH] diracdec: check if reference could not be allocated

2015-05-05 Thread Andreas Cadhalpun
On 06.05.2015 00:00, Timothy Gu wrote: > On Tue, May 5, 2015 at 2:51 PM Andreas Cadhalpun < > andreas.cadhal...@googlemail.com> wrote: >> >> +if (!s->ref_pics[i]) { >> +av_log(s->avctx, AV_LOG_ERROR, "Reference could not be >> allocated\n"); >> +return -1; >> +

Re: [FFmpeg-devel] [PATCH] diracdec: check if reference could not be allocated

2015-05-05 Thread Timothy Gu
On Tue, May 5, 2015 at 2:51 PM Andreas Cadhalpun < andreas.cadhal...@googlemail.com> wrote: > > +if (!s->ref_pics[i]) { > +av_log(s->avctx, AV_LOG_ERROR, "Reference could not be > allocated\n"); > +return -1; > +} > + AVERROR(ENOMEM) Timothy __

[FFmpeg-devel] [PATCH] diracdec: check if reference could not be allocated

2015-05-05 Thread Andreas Cadhalpun
s->ref_pics[i] is later used as ref argument of interpolate_refplane, where it is dereferenced. If it is NULL, it causes a segmentation fault. Signed-off-by: Andreas Cadhalpun --- libavcodec/diracdec.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/libavcodec/diracdec.c b/libavcodec/

[FFmpeg-devel] [PATCH] diracdec: check that block length is large enough

2015-05-05 Thread Andreas Cadhalpun
In init_planes p->xblen and p->yblen are set to: p->xblen = s->plane[0].xblen >> s->chroma_x_shift; p->yblen = s->plane[0].yblen >> s->chroma_y_shift; These are later used as block_w and block_h arguments of s->vdsp.emulated_edge_mc. If one of them is 0 it triggers an av_as

Re: [FFmpeg-devel] Loongson Optimization Patchs and Machine Express Issuse

2015-05-05 Thread Michael Niedermayer
On Tue, May 05, 2015 at 08:06:57PM +0200, Michael Niedermayer wrote: > On Wed, Apr 22, 2015 at 02:27:11PM +0800, 周晓勇 wrote: > > Hi Michael, > > We have prepared the patchs and Loongson-3A machine for you, but DHL > > needs consignee's phone number in case they could not contact u. Could you >

[FFmpeg-devel] [PATCH] diracdec: avoid overflow of bytes*8 in decode_lowdelay

2015-05-05 Thread Andreas Cadhalpun
If bytes is large enough, bytes*8 can overflow and become negative. In that case 'bufsize -= bytes*8' causes bufsize to increase instead of decrease. This leads to a segmentation fault. Signed-off-by: Andreas Cadhalpun --- libavcodec/diracdec.c | 5 - 1 file changed, 4 insertions(+), 1 del

Re: [FFmpeg-devel] [PATCH] diracdec: prevent overflow in data_unit_size check

2015-05-05 Thread Michael Niedermayer
On Tue, May 05, 2015 at 09:33:08PM +0200, Andreas Cadhalpun wrote: > buf_idx + data_unit_size can overflow, causing the '> buf_size' check to > wrongly fail. > > This causes a segmentation fault. > > Signed-off-by: Andreas Cadhalpun > --- > libavcodec/diracdec.c | 4 ++-- > 1 file changed, 2 in

Re: [FFmpeg-devel] Test case for Closed caption decoder.

2015-05-05 Thread Michael Niedermayer
On Tue, May 05, 2015 at 08:33:42PM +0530, Anshul wrote: [...] > and here is shorter file then previous one. > http://gsocdev.ccextractor.org/~anshul/test_video/Closedcaption_atsc_rollup.ts considering that this is created by FFmpeg Metadata: service_name: Service01 service_provide

[FFmpeg-devel] [PATCH] diracdec: prevent overflow in data_unit_size check

2015-05-05 Thread Andreas Cadhalpun
buf_idx + data_unit_size can overflow, causing the '> buf_size' check to wrongly fail. This causes a segmentation fault. Signed-off-by: Andreas Cadhalpun --- libavcodec/diracdec.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/diracdec.c b/libavcodec/diracdec

Re: [FFmpeg-devel] [libav-devel] [PATCH] aacsbr: break infinite loop in sbr_hf_calc_npatches

2015-05-05 Thread Andreas Cadhalpun
On 23.04.2015 17:50, Claudio Freire wrote: > On Thu, Apr 23, 2015 at 12:43 PM, Michael Niedermayer > wrote: >> ok, so is the patch the correct solution or are the fields already >> invalid before this loop and should have been checked prior ? > > I think either the fields are already invalid bef

Re: [FFmpeg-devel] [PATCH] vp9: error out if chroma subsampling changes between frames.

2015-05-05 Thread Michael Niedermayer
On Tue, May 05, 2015 at 08:26:50AM -0400, Ronald S. Bultje wrote: > This can happen after an intra-only frame. For reference, see: > https://groups.google.com/a/webmproject.org/forum/#!msg/webm-discuss/zaoTxREZjq8/v5Enht87AhcJ > --- > libavcodec/vp9.c | 9 - > 1 file changed, 8 insertions(

Re: [FFmpeg-devel] Loongson Optimization Patchs and Machine Express Issuse

2015-05-05 Thread Michael Niedermayer
On Wed, Apr 22, 2015 at 02:27:11PM +0800, 周晓勇 wrote: > Hi Michael, > We have prepared the patchs and Loongson-3A machine for you, but DHL > needs consignee's phone number in case they could not contact u. Could you > send me your phone number right now? I have make an appointment with courier

Re: [FFmpeg-devel] Loongson Optimization Patchs and Machine Express Issuse

2015-05-05 Thread Michael Niedermayer
On Sat, Apr 25, 2015 at 01:15:11PM +0200, Michael Niedermayer wrote: > On Wed, Apr 22, 2015 at 02:27:11PM +0800, 周晓勇 wrote: > > Hi Michael, > > We have prepared the patchs and Loongson-3A machine for you, but DHL > > needs consignee's phone number in case they could not contact u. Could you >

Re: [FFmpeg-devel] Test case for Closed caption decoder.

2015-05-05 Thread Anshul
On 05/05/2015 08:33 PM, Anshul wrote: On 05/04/2015 08:50 PM, Michael Niedermayer wrote: this is 31mb, that does not seem reasonable for a subtitle test, please provide a smaller file, like one without unused/untested video streams or a shorter one also fate fails with the patch applied pr

Re: [FFmpeg-devel] Test case for Closed caption decoder.

2015-05-05 Thread Anshul
On 05/04/2015 08:50 PM, Michael Niedermayer wrote: On Mon, May 04, 2015 at 05:30:05PM +0530, Anshul wrote: On 05/04/2015 05:27 PM, Anshul wrote: On 05/04/2015 03:09 AM, Michael Niedermayer wrote: On Sun, May 03, 2015 at 08:53:14PM +0200, Clément Bœsch wrote: On Sun, May 03, 2015 at 06:46:

[FFmpeg-devel] [PATCH] vp9: error out if chroma subsampling changes between frames.

2015-05-05 Thread Ronald S. Bultje
This can happen after an intra-only frame. For reference, see: https://groups.google.com/a/webmproject.org/forum/#!msg/webm-discuss/zaoTxREZjq8/v5Enht87AhcJ --- libavcodec/vp9.c | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/libavcodec/vp9.c b/libavcodec/vp9.c index c2

[FFmpeg-devel] [PATCH] vp9: error out if chroma subsampling changes between frames.

2015-05-05 Thread Ronald S. Bultje
This can happen after an intra-only frame. For reference, see: https://groups.google.com/a/webmproject.org/forum/#!msg/webm-discuss/zaoTxREZjq8/v5Enht87AhcJ --- libavcodec/vp9.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/libavcodec/vp9.c b/libavcodec/vp9.c index c2ac

Re: [FFmpeg-devel] [PATCH] avcodec/libx265: use x265 Multi-library Interface to query the API

2015-05-05 Thread Derek Buitenhuis
On 5/5/2015 9:54 AM, Gopu Govindaswamy wrote: >>From c882bbf42b0f13f98fb706139b7d3fbc53665319 Mon Sep 17 00:00:00 2001 > From: Gopu Govindaswamy > Date: Tue, 5 May 2015 12:24:44 +0530 > Subject: [PATCH] avcodec/libx265: use x265 Multi-library Interface to query > the API The patch is malformed.

[FFmpeg-devel] [PATCH] avcodec/libx265: use x265 Multi-library Interface to query the API

2015-05-05 Thread Gopu Govindaswamy
>From c882bbf42b0f13f98fb706139b7d3fbc53665319 Mon Sep 17 00:00:00 2001 From: Gopu Govindaswamy Date: Tue, 5 May 2015 12:24:44 +0530 Subject: [PATCH] avcodec/libx265: use x265 Multi-library Interface to query the API If Application might want to make a runtime selection between a number of libx2