Re: [FFmpeg-devel] [PATCH] add av_enable_strict_whitelists()

2014-10-26 Thread Alexander Strasser
On 2014-10-26 02:49 +0200, Michael Niedermayer wrote: > On Sun, Oct 26, 2014 at 02:09:19AM +0200, wm4 wrote: > > On Sun, 26 Oct 2014 00:16:26 +0200 > > Nicolas George wrote: > > > > > Le quartidi 4 brumaire, an CCXXIII, Michael Niedermayer a écrit : > > > > This fixes the issue that a not set or

Re: [FFmpeg-devel] [PATCH] add av_enable_strict_whitelists()

2014-10-25 Thread Michael Niedermayer
On Sun, Oct 26, 2014 at 02:09:19AM +0200, wm4 wrote: > On Sun, 26 Oct 2014 00:16:26 +0200 > Nicolas George wrote: > > > Le quartidi 4 brumaire, an CCXXIII, Michael Niedermayer a écrit : > > > This fixes the issue that a not set or not forwarded whitelist > > > would allow to bypass it. > > > > I

Re: [FFmpeg-devel] [PATCH] add av_enable_strict_whitelists()

2014-10-25 Thread wm4
On Sun, 26 Oct 2014 00:16:26 +0200 Nicolas George wrote: > Le quartidi 4 brumaire, an CCXXIII, Michael Niedermayer a écrit : > > This fixes the issue that a not set or not forwarded whitelist > > would allow to bypass it. > > I am a bit worried by the sheer amount of extra code and API this simp

Re: [FFmpeg-devel] [PATCH] add av_enable_strict_whitelists()

2014-10-25 Thread Michael Niedermayer
On Sun, Oct 26, 2014 at 12:16:26AM +0200, Nicolas George wrote: > Le quartidi 4 brumaire, an CCXXIII, Michael Niedermayer a écrit : > > This fixes the issue that a not set or not forwarded whitelist > > would allow to bypass it. > > I am a bit worried by the sheer amount of extra code and API this

Re: [FFmpeg-devel] [PATCH] add av_enable_strict_whitelists()

2014-10-25 Thread Nicolas George
Le quartidi 4 brumaire, an CCXXIII, Michael Niedermayer a écrit : > This fixes the issue that a not set or not forwarded whitelist > would allow to bypass it. I am a bit worried by the sheer amount of extra code and API this simple feature requires. Maybe it is a sign that it was not the best appr

Re: [FFmpeg-devel] [PATCH] add av_enable_strict_whitelists()

2014-10-25 Thread Michael Niedermayer
On Sat, Oct 25, 2014 at 05:43:00PM -0300, James Almer wrote: > On 25/10/14 4:51 PM, Michael Niedermayer wrote: > > diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h > > index eac3fc7..1000c80 100644 > > --- a/libavcodec/avcodec.h > > +++ b/libavcodec/avcodec.h > > @@ -3118,6 +3118,8 @@ typed

Re: [FFmpeg-devel] [PATCH] add av_enable_strict_whitelists()

2014-10-25 Thread James Almer
On 25/10/14 4:51 PM, Michael Niedermayer wrote: > diff --git a/libavcodec/avcodec.h b/libavcodec/avcodec.h > index eac3fc7..1000c80 100644 > --- a/libavcodec/avcodec.h > +++ b/libavcodec/avcodec.h > @@ -3118,6 +3118,8 @@ typedef struct AVCodecContext { > * If NULL then all are allowed >

Re: [FFmpeg-devel] [PATCH] add av_enable_strict_whitelists()

2014-10-25 Thread wm4
On Sat, 25 Oct 2014 21:51:25 +0200 Michael Niedermayer wrote: > This fixes the issue that a not set or not forwarded whitelist > would allow to bypass it. > > Signed-off-by: Michael Niedermayer > --- > libavcodec/avcodec.h | 17 + > libavcodec/utils.c | 14 +++

[FFmpeg-devel] [PATCH] add av_enable_strict_whitelists()

2014-10-25 Thread Michael Niedermayer
This fixes the issue that a not set or not forwarded whitelist would allow to bypass it. Signed-off-by: Michael Niedermayer --- libavcodec/avcodec.h | 17 + libavcodec/utils.c | 14 +- libavformat/avformat.h |4 libavformat/utils.c|6 --