Re: [FFmpeg-devel] [PATCH 1/3] mpeg12dec: validate color space

2017-01-27 Thread Andreas Cadhalpun
On 26.01.2017 13:59, Ronald S. Bultje wrote: > On Wed, Jan 25, 2017 at 8:56 PM, Andreas Cadhalpun < > andreas.cadhal...@googlemail.com> wrote: >> On 26.01.2017 02:26, Ronald S. Bultje wrote: >>> The question is not whether changing A or B towards the other makes the >>> combination consistent. The

Re: [FFmpeg-devel] [PATCH 1/3] mpeg12dec: validate color space

2017-01-26 Thread Ronald S. Bultje
Hi, On Wed, Jan 25, 2017 at 8:56 PM, Andreas Cadhalpun < andreas.cadhal...@googlemail.com> wrote: > On 26.01.2017 02:26, Ronald S. Bultje wrote: > > Hi, > > > > On Wed, Jan 25, 2017 at 8:20 PM, Andreas Cadhalpun < > > andreas.cadhal...@googlemail.com> wrote: > > > >> On 07.01.2017 13:44, Ronald S

Re: [FFmpeg-devel] [PATCH 1/3] mpeg12dec: validate color space

2017-01-25 Thread Andreas Cadhalpun
On 26.01.2017 02:26, Ronald S. Bultje wrote: > Hi, > > On Wed, Jan 25, 2017 at 8:20 PM, Andreas Cadhalpun < > andreas.cadhal...@googlemail.com> wrote: > >> On 07.01.2017 13:44, Ronald S. Bultje wrote: >>> Doesn't this destroy exporting of newly added colorspaces that have no >>> explicit mention

Re: [FFmpeg-devel] [PATCH 1/3] mpeg12dec: validate color space

2017-01-25 Thread Ronald S. Bultje
Hi, On Wed, Jan 25, 2017 at 8:20 PM, Andreas Cadhalpun < andreas.cadhal...@googlemail.com> wrote: > On 07.01.2017 13:44, Ronald S. Bultje wrote: > > On Fri, Jan 6, 2017 at 9:35 PM, Michael Niedermayer > > > wrote: > > > >> On Fri, Jan 06, 2017 at 09:43:24PM +0100, Andreas Cadhalpun wrote: > >>>

Re: [FFmpeg-devel] [PATCH 1/3] mpeg12dec: validate color space

2017-01-25 Thread Andreas Cadhalpun
On 07.01.2017 13:44, Ronald S. Bultje wrote: > On Fri, Jan 6, 2017 at 9:35 PM, Michael Niedermayer > wrote: > >> On Fri, Jan 06, 2017 at 09:43:24PM +0100, Andreas Cadhalpun wrote: >>> On 23.12.2016 00:57, Andreas Cadhalpun wrote: Signed-off-by: Andreas Cadhalpun --- libavcodec/mp

Re: [FFmpeg-devel] [PATCH 1/3] mpeg12dec: validate color space

2017-01-07 Thread Ronald S. Bultje
Hi, On Fri, Jan 6, 2017 at 9:35 PM, Michael Niedermayer wrote: > On Fri, Jan 06, 2017 at 09:43:24PM +0100, Andreas Cadhalpun wrote: > > On 23.12.2016 00:57, Andreas Cadhalpun wrote: > > > Signed-off-by: Andreas Cadhalpun > > > --- > > > libavcodec/mpeg12dec.c | 4 > > > 1 file changed, 4

Re: [FFmpeg-devel] [PATCH 1/3] mpeg12dec: validate color space

2017-01-06 Thread Michael Niedermayer
On Fri, Jan 06, 2017 at 09:43:24PM +0100, Andreas Cadhalpun wrote: > On 23.12.2016 00:57, Andreas Cadhalpun wrote: > > Signed-off-by: Andreas Cadhalpun > > --- > > libavcodec/mpeg12dec.c | 4 > > 1 file changed, 4 insertions(+) > > > > diff --git a/libavcodec/mpeg12dec.c b/libavcodec/mpeg12

Re: [FFmpeg-devel] [PATCH 1/3] mpeg12dec: validate color space

2017-01-06 Thread Andreas Cadhalpun
On 23.12.2016 00:57, Andreas Cadhalpun wrote: > Signed-off-by: Andreas Cadhalpun > --- > libavcodec/mpeg12dec.c | 4 > 1 file changed, 4 insertions(+) > > diff --git a/libavcodec/mpeg12dec.c b/libavcodec/mpeg12dec.c > index 63979079c8..d3dc67ad6a 100644 > --- a/libavcodec/mpeg12dec.c > +++

[FFmpeg-devel] [PATCH 1/3] mpeg12dec: validate color space

2016-12-22 Thread Andreas Cadhalpun
Signed-off-by: Andreas Cadhalpun --- libavcodec/mpeg12dec.c | 4 1 file changed, 4 insertions(+) diff --git a/libavcodec/mpeg12dec.c b/libavcodec/mpeg12dec.c index 63979079c8..d3dc67ad6a 100644 --- a/libavcodec/mpeg12dec.c +++ b/libavcodec/mpeg12dec.c @@ -1470,6 +1470,10 @@ static void mpe