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

2017-05-30 Thread James Almer
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 --git a/libavutil/pixfmt.h b/libavutil/pixfmt.h >

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/audiotoolboxdec: always use a copy of the AVCodecContext extradata

2017-05-30 Thread James Almer
Ping for set. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH 1/2] libavformat/http: Ignore expired cookies

2017-05-30 Thread Micah Galizia
Signed-off-by: Micah Galizia Signed-off-by: Michael Niedermayer (cherry picked from commit 28b24670741e1de25bfc7b5ea7c1d6dbae1aef6f) Signed-off-by: Micah Galizia --- libavformat/http.c | 213

[FFmpeg-devel] [PATCH 2/2] libavformat/hls: Observe Set-Cookie headers

2017-05-30 Thread Micah Galizia
Signed-off-by: Micah Galizia Signed-off-by: Michael Niedermayer (cherry picked from commit c4c73020f4bbf261f0b263be82de575c17fa5a60) Signed-off-by: Micah Galizia --- libavformat/hls.c | 12 ++-- 1 file changed, 10

[FFmpeg-devel] Backport HLS Cookie Authentication Fixes to 3.3

2017-05-30 Thread Micah Galizia
Hello, I'd like to backport the HLS cookie authentication fixes to 3.3 so that Kodi 18 picks them up. Originally, this was a three patch set but the first fix (to av_small_strptime) is already in 3.3 and doesn't need to be backported. The two that are still required are: -

Re: [FFmpeg-devel] [PATCH] lavc/golomb: Fix UE golomb overwrite issue.

2017-05-30 Thread Jun Zhao
On 2017/5/29 7:40, Michael Niedermayer wrote: > On Fri, May 26, 2017 at 09:19:09AM +0800, Jun Zhao wrote: >> golomb.h |5 - >> 1 file changed, 4 insertions(+), 1 deletion(-) >> 228c7180856b65d095dd0b8d59f3d3ff4f65774a >> 0001-lavc-golomb-Fix-UE-golomb-overwrite-issue.patch >> From

Re: [FFmpeg-devel] [PATCH] libavformat/hls: Observe Set-Cookie headers

2017-05-30 Thread Micah Galizia
On 2017-05-28 08:44 PM, Michael Niedermayer wrote: On Fri, May 26, 2017 at 09:29:04PM -0400, Micah Galizia wrote: On Sat, May 20, 2017 at 9:36 PM, Micah Galizia wrote: On 2017-05-17 05:23 AM, wm4 wrote: On Sat, 6 May 2017 14:28:10 -0400 Micah Galizia

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

2017-05-30 Thread Michael Niedermayer
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 information leak > > > > The existing exploit depends on a specific decoder as well. > > It does

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

2017-05-30 Thread Hendrik Leppkes
On Wed, May 31, 2017 at 12:52 AM, Michael Niedermayer wrote: > This prevents an exploit leading to an information leak > > The existing exploit depends on a specific decoder as well. > It does appear though that the exploit should be possible with any decoder. > The

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/x86: move simple_idct to external assembly

2017-05-30 Thread Michael Niedermayer
On Mon, May 29, 2017 at 09:44:21PM -0400, Ronald S. Bultje wrote: > Hi, > > On Mon, May 29, 2017 at 5:26 PM, Michael Niedermayer > wrote: > > > On Mon, May 29, 2017 at 09:40:49PM +0200, James Darnley wrote: > > > On 2017-05-29 16:51, James Darnley wrote: > > > > --- > >

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

2017-05-30 Thread Michael Niedermayer
This prevents an exploit leading to an information leak The existing exploit depends on a specific decoder as well. It does appear though that the exploit should be possible with any decoder. The problem is that as long as sensitive information gets into the decoder, the output of the decoder

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

2017-05-30 Thread wm4
On Tue, 30 May 2017 18:46:36 -0300 James Almer wrote: > On 5/30/2017 9:07 AM, wm4 wrote: > > On Tue, 23 May 2017 13:36:51 +0200 > > wm4 wrote: > > > >> Fixes detection of some TV sample as 24.5 FPS. With the patch applied, > >> it's detected as 25

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

2017-05-30 Thread James Almer
On 5/30/2017 9:07 AM, wm4 wrote: > On Tue, 23 May 2017 13:36:51 +0200 > wm4 wrote: > >> Fixes detection of some TV sample as 24.5 FPS. With the patch applied, >> it's detected as 25 FPS. >> --- >> libavformat/utils.c | 22 ++ >> 1 file changed, 22

[FFmpeg-devel] [PATCH] libavfilter/scale2ref: Add constants for the primary input

2017-05-30 Thread Kevin Mark
Variables pertaining to the main video are now available when using the scale2ref filter. This allows, as an example, scaling a video with another as a reference point while maintaining the original aspect ratio of the primary/non-reference video. Consider the following graph: scale2ref=iw/6:-1

[FFmpeg-devel] [PATCH 2/2] avcodec/vorbisenc: Use a bufqueue in encoding with smaller lengths

2017-05-30 Thread Tyler Jones
Switching the vorbis encoder to use a buffer queue for input frames allows saving lookahead samples more easily and safely for psychoacoustic systems, requiring less pointer arithmetic in the case of transient windows. --- libavcodec/vorbisenc.c | 120

[FFmpeg-devel] [PATCH 1/2] avcodec/vorbisenc: Include bufqueue and afqueue

2017-05-30 Thread Tyler Jones
--- libavcodec/vorbisenc.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/libavcodec/vorbisenc.c b/libavcodec/vorbisenc.c index 2974ca2..fdce864 100644 --- a/libavcodec/vorbisenc.c +++ b/libavcodec/vorbisenc.c @@ -33,6 +33,9 @@ #include "vorbis.h" #include "vorbis_enc_data.h"

Re: [FFmpeg-devel] [PATCH] libavfilter/scale2ref: Add constants for the primary input

2017-05-30 Thread Kevin Mark
On Tue, May 30, 2017 at 10:40 AM, Kevin Mark wrote: > +const AVFilterLink *main; Unfortunately that line results in a warning on GCC (but not LLVM): libavfilter/scale.c: In function ‘ff_scale_eval_dimensions’: libavfilter/scale.c:121:25: warning: ‘main’ is usually a

[FFmpeg-devel] [PATCH 1/2] avcodec/vorbisenc: Include fdsp

2017-05-30 Thread Tyler Jones
Signed-off-by: Tyler Jones --- libavcodec/vorbisenc.c | 31 +-- 1 file changed, 25 insertions(+), 6 deletions(-) diff --git a/libavcodec/vorbisenc.c b/libavcodec/vorbisenc.c index 2974ca2..7c3cd51 100644 --- a/libavcodec/vorbisenc.c +++

[FFmpeg-devel] [PATCH 2/2] avcodec/vorbisenc: Use fdsp for applying windows

2017-05-30 Thread Tyler Jones
Using fdsp improves readability and allows using architecture-specific optimizations. Signed-off-by: Tyler Jones --- libavcodec/vorbisenc.c | 16 +--- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/libavcodec/vorbisenc.c b/libavcodec/vorbisenc.c

Re: [FFmpeg-devel] [PATCH] libavfilter/scale2ref: Add constants for the primary input

2017-05-30 Thread Kevin Mark
I'm hoping this is the proper means of submitting an updated patch. I used git send-email with the --in-reply-to option set to the Message-Id of my original patch. It looks like it created a new patch in Patchwork (instead of updating the old one) and I'm not sure if that's what we want it to do.

[FFmpeg-devel] [PATCH] libavfilter/scale2ref: Add constants for the primary input

2017-05-30 Thread Kevin Mark
Variables pertaining to the main video are now available when using the scale2ref filter. This allows, as an example, scaling a video with another as a reference point while maintaining the original aspect ratio of the primary/non-reference video. Consider the following graph: scale2ref=iw/6:-1

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

2017-05-30 Thread wm4
On Tue, 23 May 2017 13:36:51 +0200 wm4 wrote: > Fixes detection of some TV sample as 24.5 FPS. With the patch applied, > it's detected as 25 FPS. > --- > libavformat/utils.c | 22 ++ > 1 file changed, 22 insertions(+) > > diff --git

Re: [FFmpeg-devel] [PATCH 2/2] reindent

2017-05-30 Thread Ronald S. Bultje
Hi, On May 30, 2017 6:44 AM, "James Darnley" wrote: On 2017-05-29 16:51, James Darnley wrote: > Commit message: reindent Is this acceptable? Should I be more verbose? Make it x86/idctdsp_init: reindent. Otherwise ok. Thanks, Ronald

Re: [FFmpeg-devel] [PATCH 2/2] reindent

2017-05-30 Thread James Darnley
On 2017-05-29 16:51, James Darnley wrote: > Commit message: reindent Is this acceptable? Should I be more verbose? ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] Freelancer wanted to work on ffmpeg integration

2017-05-30 Thread Daniel Walz
Dear fellow developers, For the integration of FFMPEG reading and writing video files we are looking for a freelance worker. The objectives are to create video files that play on the standard players. We want to make as many formats/codecs as possible available to our users, but will focus on the

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/x86: move simple_idct to external assembly

2017-05-30 Thread James Darnley
On 2017-05-29 23:26, Michael Niedermayer wrote: > On Mon, May 29, 2017 at 09:40:49PM +0200, James Darnley wrote: >> On 2017-05-29 16:51, James Darnley wrote: >>> --- >>> Changes: >>> - Changed type of d4 constant to dwords because it gets used as dwords. >>> - Changed or removed

Re: [FFmpeg-devel] [PATCH 1/3] libavfilter/scale2ref: Add constants for the non-ref input

2017-05-30 Thread Gyan
On Tue, May 30, 2017 at 3:27 PM, Kevin Mark wrote: > > Sounds good. If my understanding is correct, you would recommend changing > the > scale2ref=iw/6:iw/(6*main_a) [main][ref] > example to > scale2ref=iw/6:iw/(6*mdar) [main][ref] > correct? > I prefer

Re: [FFmpeg-devel] [PATCH 1/3] libavfilter/scale2ref: Add constants for the non-ref input

2017-05-30 Thread Kevin Mark
On Sun, May 28, 2017 at 12:14 PM, Gyan wrote: > If you do, I suggest 'mdar' for main_dar which is the only one that ought > to be required. > > The scale filter adjusts the SAR of the result so that the DAR is preserved > i.e. when a 100x100 canvas w/ SAR of 2.0 is scaled to

Re: [FFmpeg-devel] [PATCH 1/3] avcodec/aacenc: Reduce loglevel of aac_encode_end() from INFO to VER

2017-05-30 Thread Rostislav Pehlivanov
On 30 May 2017 at 08:43, Ingo Brückl wrote: > Rostislav Pehlivanov wrote on Mon, 29 May 2017 16:19:51 +0100: > > > On 29 May 2017 at 14:55, Ingo Brückl wrote: > > >> AV_LOG_INFO is the default and meant for informational output. > >> The information

Re: [FFmpeg-devel] [PATCH 1/3] avcodec/aacenc: Reduce loglevel of aac_encode_end() from INFO to VER

2017-05-30 Thread Ingo Brückl
Rostislav Pehlivanov wrote on Mon, 29 May 2017 16:19:51 +0100: > On 29 May 2017 at 14:55, Ingo Brückl wrote: >> AV_LOG_INFO is the default and meant for informational output. >> The information given by aac_encode_end() is rather technical >> and of less interest to a