Re: [FFmpeg-devel] [PATCH] mov: Add an option to toggle dref opening

2016-01-20 Thread Marton Balint
On Wed, 20 Jan 2016, Derek Buitenhuis wrote: On 1/16/2016 7:58 PM, Marton Balint wrote: CLI users still going to need an option for it. What do you mean? Demuxer options are all exposed in the CLI. I was referring to the AVFormatContext.open_cb URL validator approach which is not directl

Re: [FFmpeg-devel] [PATCH] mov: Add an option to toggle dref opening

2016-01-20 Thread Derek Buitenhuis
On 1/16/2016 7:58 PM, Marton Balint wrote: > CLI users still going to need an option for it. What do you mean? Demuxer options are all exposed in the CLI. -enable_drefs 1 works fine. - Derek ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://

Re: [FFmpeg-devel] [PATCH] mov: Add an option to toggle dref opening

2016-01-20 Thread Michael Niedermayer
On Sat, Jan 16, 2016 at 02:22:21PM +0100, Michael Niedermayer wrote: > On Fri, Jan 15, 2016 at 05:03:49PM +, Derek Buitenhuis wrote: > > This feature is mostly only used by NLE software, and is > > both of dubious value being enabled by default, and a > > possible security risk. > > > > Signed

Re: [FFmpeg-devel] [PATCH] mov: Add an option to toggle dref opening

2016-01-16 Thread Marton Balint
On Sat, 16 Jan 2016, wm4 wrote: On Sat, 16 Jan 2016 14:22:21 +0100 Michael Niedermayer wrote: On Fri, Jan 15, 2016 at 05:03:49PM +, Derek Buitenhuis wrote: > This feature is mostly only used by NLE software, and is > both of dubious value being enabled by default, and a > possible security

Re: [FFmpeg-devel] [PATCH] mov: Add an option to toggle dref opening

2016-01-16 Thread wm4
On Sat, 16 Jan 2016 14:22:21 +0100 Michael Niedermayer wrote: > On Fri, Jan 15, 2016 at 05:03:49PM +, Derek Buitenhuis wrote: > > This feature is mostly only used by NLE software, and is > > both of dubious value being enabled by default, and a > > possible security risk. > > > > Signed-off-

Re: [FFmpeg-devel] [PATCH] mov: Add an option to toggle dref opening

2016-01-16 Thread Michael Niedermayer
On Fri, Jan 15, 2016 at 05:03:49PM +, Derek Buitenhuis wrote: > This feature is mostly only used by NLE software, and is > both of dubious value being enabled by default, and a > possible security risk. > > Signed-off-by: Derek Buitenhuis > --- > libavformat/isom.h| 1 + > libavformat/m

[FFmpeg-devel] [PATCH] mov: Add an option to toggle dref opening

2016-01-15 Thread Derek Buitenhuis
This feature is mostly only used by NLE software, and is both of dubious value being enabled by default, and a possible security risk. Signed-off-by: Derek Buitenhuis --- libavformat/isom.h| 1 + libavformat/mov.c | 22 +- libavformat/version.h | 4 ++-- 3 files cha