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 geo...@nsup.org wrote: Le quartidi 4 brumaire, an CCXXIII, Michael Niedermayer a écrit : This fixes the issue that a not set

[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 michae...@gmx.at --- libavcodec/avcodec.h | 17 + libavcodec/utils.c | 14 +- libavformat/avformat.h |4 libavformat/utils.c

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 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 @@ typedef struct

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

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 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 geo...@nsup.org 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.