Re: [FFmpeg-devel] [PATCH] avutil/pixfmt: remove superfluous define

2017-05-31 Thread James Almer
On 5/31/2017 12:04 AM, James Almer wrote: > On 5/23/2017 7:07 PM, James Almer wrote: >> It's an AVColorSpace value since 82ad9cbd32c873bced9adf4a2bb67dcda7294c61. >> >> Signed-off-by: James Almer >> --- >> libavutil/pixfmt.h | 2 -- >> 1 file changed, 2 deletions(-) >> >> diff

Re: [FFmpeg-devel] [PATCH] vf_framestep: add blend parameter for motion blur effect

2017-05-31 Thread Paul B Mahol
On 5/31/17, Matthias Troffaes wrote: > Dear Paul, > > On Wed, May 31, 2017 at 7:59 PM, Paul B Mahol wrote: >> This code does not belong in this filter. >> >> Make new filter instead. > > May I kindly ask why you think so? I considered this as well

Re: [FFmpeg-devel] [PATCH] vf_framestep: add blend parameter for motion blur effect

2017-05-31 Thread wm4
On Wed, 31 May 2017 20:59:07 +0200 Paul B Mahol wrote: > On 5/31/17, Matthias Troffaes wrote: > > Dear Moritz, > > > > On Wed, May 31, 2017 at 2:17 PM, Moritz Barsnick wrote: > >> On Wed, May 31, 2017 at 13:31:14 +0100,

Re: [FFmpeg-devel] [PATCH] vf_framestep: add blend parameter for motion blur effect

2017-05-31 Thread Matthias Troffaes
Dear Paul, On Wed, May 31, 2017 at 7:59 PM, Paul B Mahol wrote: > This code does not belong in this filter. > > Make new filter instead. May I kindly ask why you think so? I considered this as well but then decided against it, as this new filter would behave like framestep as

Re: [FFmpeg-devel] [PATCH] vf_framestep: add blend parameter for motion blur effect

2017-05-31 Thread Paul B Mahol
On 5/31/17, Matthias Troffaes wrote: > Dear Moritz, > > On Wed, May 31, 2017 at 2:17 PM, Moritz Barsnick wrote: >> On Wed, May 31, 2017 at 13:31:14 +0100, Matthias C. M. Troffaes wrote: >>> @@ -2,6 +2,7 @@ Entries are sorted chronologically from

Re: [FFmpeg-devel] [PATCH] vf_framestep: add blend parameter for motion blur effect

2017-05-31 Thread Matthias Troffaes
Dear Moritz, On Wed, May 31, 2017 at 2:17 PM, Moritz Barsnick wrote: > On Wed, May 31, 2017 at 13:31:14 +0100, Matthias C. M. Troffaes wrote: >> @@ -2,6 +2,7 @@ Entries are sorted chronologically from oldest to youngest >> within each release, >> releases are sorted from

Re: [FFmpeg-devel] [PATCH] avformat/hls: Disallow local file access by default

2017-05-31 Thread Michael Niedermayer
On Wed, May 31, 2017 at 05:18:57PM +0200, Tobias Rapp wrote: > On 31.05.2017 15:42, wm4 wrote: > >On Wed, 31 May 2017 14:49:19 +0200 > >Michael Niedermayer wrote: > > > >> [...] > >> > >>Security fixes should be as simple as > >> possible. > > > >Well, your fix isn't

[FFmpeg-devel] [PATCH] lavf: consider codec framerate for framerate detection

2017-05-31 Thread wm4
Fixes detection of some TV sample as 24.5 FPS. With the patch applied, it's detected as 25 FPS. This is enabled for mpegts only. --- libavformat/internal.h | 5 + libavformat/mpegts.c | 2 ++ libavformat/utils.c| 10 ++ 3 files changed, 17 insertions(+) diff --git

Re: [FFmpeg-devel] [PATCH] avformat/hls: Disallow local file access by default

2017-05-31 Thread Michael Niedermayer
On Wed, May 31, 2017 at 03:42:41PM +0200, wm4 wrote: > On Wed, 31 May 2017 14:49:19 +0200 > Michael Niedermayer wrote: > > > On Wed, May 31, 2017 at 01:13:50PM +0200, wm4 wrote: > > > On Wed, 31 May 2017 12:51:35 +0200 > > > Michael Niedermayer

[FFmpeg-devel] [PATCH 1/2] ffprobe: Print AVMasteringDisplayMetadata side data contents

2017-05-31 Thread Vittorio Giovara
--- ffprobe.c | 39 +++ 1 file changed, 39 insertions(+) diff --git a/ffprobe.c b/ffprobe.c index f2a3cc7a73..f6d0264334 100644 --- a/ffprobe.c +++ b/ffprobe.c @@ -35,6 +35,7 @@ #include "libavutil/bprint.h" #include "libavutil/display.h" #include

[FFmpeg-devel] [PATCH 2/2] ffprobe: Print AVContentLightMetadata side data contents

2017-05-31 Thread Vittorio Giovara
--- ffprobe.c | 8 1 file changed, 8 insertions(+) diff --git a/ffprobe.c b/ffprobe.c index f6d0264334..b3464aac8f 100644 --- a/ffprobe.c +++ b/ffprobe.c @@ -1913,6 +1913,10 @@ static void print_pkt_side_data(WriterContext *w, print_q("min_luminance",

Re: [FFmpeg-devel] [PATCH] avformat/hls: Disallow local file access by default

2017-05-31 Thread Tobias Rapp
On 31.05.2017 15:42, wm4 wrote: On Wed, 31 May 2017 14:49:19 +0200 Michael Niedermayer wrote: >> [...] >> Security fixes should be as simple as possible. Well, your fix isn't simple. It adds yet another exception with questionable effect. It makes it more complex

Re: [FFmpeg-devel] [PATCH] avformat/hls: Disallow local file access by default

2017-05-31 Thread wm4
On Wed, 31 May 2017 14:49:19 +0200 Michael Niedermayer wrote: > On Wed, May 31, 2017 at 01:13:50PM +0200, wm4 wrote: > > On Wed, 31 May 2017 12:51:35 +0200 > > Michael Niedermayer wrote: > > > > > On Wed, May 31, 2017 at 11:52:06AM +0200, wm4

Re: [FFmpeg-devel] [PATCH] vf_framestep: add blend parameter for motion blur effect

2017-05-31 Thread Moritz Barsnick
On Wed, May 31, 2017 at 13:31:14 +0100, Matthias C. M. Troffaes wrote: > @@ -2,6 +2,7 @@ Entries are sorted chronologically from oldest to youngest > within each release, > releases are sorted from youngest to oldest. > > version : > +- framestep filter: add blend parameter for motion blur

Re: [FFmpeg-devel] [PATCH] avformat/hls: Disallow local file access by default

2017-05-31 Thread Michael Niedermayer
On Wed, May 31, 2017 at 01:13:50PM +0200, wm4 wrote: > On Wed, 31 May 2017 12:51:35 +0200 > Michael Niedermayer wrote: > > > On Wed, May 31, 2017 at 11:52:06AM +0200, wm4 wrote: > > > On Wed, 31 May 2017 11:29:56 +0200 > > > Michael Niedermayer

Re: [FFmpeg-devel] [PATCH] vf_framestep: add blend parameter for motion blur effect

2017-05-31 Thread Matthias Troffaes
Note that this is a resubmission of https://ffmpeg.org/pipermail/ffmpeg-devel/2017-April/209794.html - in particular the mips test failure reported in https://ffmpeg.org/pipermail/ffmpeg-devel/2017-April/209820.html has been fixed. Kind regards, Matthias On Wed, May 31, 2017 at 1:31 PM, Matthias

[FFmpeg-devel] [PATCH] vf_framestep: add blend parameter for motion blur effect

2017-05-31 Thread Matthias C. M. Troffaes
This patch adds a "blend" parameter to the framestep filter, to blend frames together at each step, for a motion blur effect. The number of frames that are blended (i.e. the exposure time, in frames) can be controlled, allowing control over the strength of the motion blur. The filter has timeline

Re: [FFmpeg-devel] [PATCH v10] - Added Turing codec interface for ffmpeg

2017-05-31 Thread Saverio Blasi
Hello, Could you please provide us with some feedback on this patch? Thanks, Saverio -Original Message- From: Saverio Blasi [mailto:saverio.bl...@bbc.co.uk] Sent: 09 May 2017 11:08 To: ffmpeg-devel@ffmpeg.org Cc: Saverio Blasi Subject: [PATCH v10] - Added

Re: [FFmpeg-devel] [PATCH] avformat/hls: Disallow local file access by default

2017-05-31 Thread wm4
On Wed, 31 May 2017 12:51:35 +0200 Michael Niedermayer wrote: > On Wed, May 31, 2017 at 11:52:06AM +0200, wm4 wrote: > > On Wed, 31 May 2017 11:29:56 +0200 > > Michael Niedermayer wrote: > > > > > On Wed, May 31, 2017 at 09:03:34AM +0200,

Re: [FFmpeg-devel] [PATCH] avformat/hls: Disallow local file access by default

2017-05-31 Thread Michael Niedermayer
On Wed, May 31, 2017 at 11:52:06AM +0200, wm4 wrote: > On Wed, 31 May 2017 11:29:56 +0200 > Michael Niedermayer wrote: > > > On Wed, May 31, 2017 at 09:03:34AM +0200, Hendrik Leppkes wrote: > > > On Wed, May 31, 2017 at 2:09 AM, Michael Niedermayer > > >

Re: [FFmpeg-devel] [PATCH] videotoolbox: log errors

2017-05-31 Thread wm4
On Wed, 24 May 2017 15:47:36 +0200 wm4 wrote: > With the new decode API, you can't handle errors directly in the API > user - you only know that the hwaccel did not initialize at all. > > Add some approximate logging. > --- > libavcodec/videotoolbox.c | 6 ++ > 1

Re: [FFmpeg-devel] [PATCH] avformat/hls: Disallow local file access by default

2017-05-31 Thread wm4
On Wed, 31 May 2017 11:29:56 +0200 Michael Niedermayer wrote: > On Wed, May 31, 2017 at 09:03:34AM +0200, Hendrik Leppkes wrote: > > On Wed, May 31, 2017 at 2:09 AM, Michael Niedermayer > > wrote: > > > On Wed, May 31, 2017 at 01:14:58AM +0200,

Re: [FFmpeg-devel] [PATCH] avformat/hls: Disallow local file access by default

2017-05-31 Thread Michael Niedermayer
On Wed, May 31, 2017 at 09:03:34AM +0200, Hendrik Leppkes wrote: > On Wed, May 31, 2017 at 2:09 AM, Michael Niedermayer > wrote: > > On Wed, May 31, 2017 at 01:14:58AM +0200, Hendrik Leppkes wrote: > >> On Wed, May 31, 2017 at 12:52 AM, Michael Niedermayer > >>

Re: [FFmpeg-devel] [PATCH] avformat/hls: Disallow local file access by default

2017-05-31 Thread Hendrik Leppkes
On Wed, May 31, 2017 at 2:09 AM, Michael Niedermayer wrote: > On Wed, May 31, 2017 at 01:14:58AM +0200, Hendrik Leppkes wrote: >> On Wed, May 31, 2017 at 12:52 AM, Michael Niedermayer >> wrote: >> > This prevents an exploit leading to an