Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: creation of hls variant streams with master playlist in a single hlsenc instance

2017-11-15 Thread 刘歧
> 在 2017年11月15日,21:48,Dixit, Vishwanath 写道: > > >> On 11/15/17, 9:56 AM, "刘歧" wrote: >> all test info bellow: >> >> 1st, look at the ffmpeg.exe banner >> 2nd, test with fate-filter-hls-append >> 3nd, test with filter-hls-vs-with-master >> >> >> /home/liuqi/ffmpeg/tests/fate-run

Re: [FFmpeg-devel] [PATCH 1/3] libavformat/avio: Utility function to return URLContext

2017-11-15 Thread Jeyapal, Karthick
>On 11/15/17, 6:34 PM, "Nicolas George" wrote: >This is rather fragile. Some kind of check that is is actually a real >URLContext would be a very good idea. Thanks for the feedback. I have a relevant condition check so that only a real URLContext is returned. Please find the new patch attached.

Re: [FFmpeg-devel] [PATCH] avcodec/proresdec: align dequantization matrix buffers

2017-11-15 Thread James Almer
On 11/16/2017 12:38 AM, Carl Eugen Hoyos wrote: > 2017-11-16 3:51 GMT+01:00 James Almer : > >> gcc -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=2 -fno-strict-overflow >> -fstack-protector-all -fPIE -c -o /tmp/ffconf.OTilhXct/test.o >> /tmp/ffconf.OTilhXct/test.c >> gcc -Wl,-z,relro -Wl,-z,now -fPIE -pie -o

Re: [FFmpeg-devel] [PATCH] avcodec/proresdec: align dequantization matrix buffers

2017-11-15 Thread Carl Eugen Hoyos
2017-11-16 3:51 GMT+01:00 James Almer : > gcc -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=2 -fno-strict-overflow > -fstack-protector-all -fPIE -c -o /tmp/ffconf.OTilhXct/test.o > /tmp/ffconf.OTilhXct/test.c > gcc -Wl,-z,relro -Wl,-z,now -fPIE -pie -o /tmp/ffconf.OTilhXct/test.exe > /tmp/ffconf.OTilhXct/te

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/snowdec: Check intra block dc differences.

2017-11-15 Thread Carl Eugen Hoyos
2017-11-16 4:06 GMT+01:00 Ronald S. Bultje : > So, commit it without the error message? I really don't see the issue. As explained, the issue is that without an error message, it is impossible to parse any related bug report. Carl Eugen ___ ffmpeg-deve

Re: [FFmpeg-devel] [PATCH] avcodec/proresdec: align dequantization matrix buffers

2017-11-15 Thread James Almer
On 11/16/2017 12:04 AM, James Almer wrote: > On 11/15/2017 11:55 PM, Carl Eugen Hoyos wrote: >> 2017-11-16 3:51 GMT+01:00 James Almer : >> >>> configure can't even succeed with --toolchain=hardened on mingw-w64, so >>> looks like i wouldn't have been able to reproduce it to begin with. >> >> Try re

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/snowdec: Check intra block dc differences.

2017-11-15 Thread Ronald S. Bultje
Hi, On Wed, Nov 15, 2017 at 7:02 PM, Michael Niedermayer wrote: > On Wed, Nov 15, 2017 at 03:26:42PM -0500, Ronald S. Bultje wrote: > > Hi, > > > > On Wed, Nov 15, 2017 at 3:17 PM, Michael Niedermayer > > > wrote: > > > > > Fixes: Timeout > > > Fixes: 3142/clusterfuzz-testcase-5007853163118592

Re: [FFmpeg-devel] [PATCH] avcodec/proresdec: align dequantization matrix buffers

2017-11-15 Thread James Almer
On 11/15/2017 11:55 PM, Carl Eugen Hoyos wrote: > 2017-11-16 3:51 GMT+01:00 James Almer : > >> configure can't even succeed with --toolchain=hardened on mingw-w64, so >> looks like i wouldn't have been able to reproduce it to begin with. > > Try replacing toolchain with --extra-cflags=-fstack-pro

Re: [FFmpeg-devel] Added HW H.264 and HEVC encoding for AMD GPUs based on AMF SDK

2017-11-15 Thread Michael Niedermayer
On Tue, Nov 14, 2017 at 05:55:33PM -0500, mmironov wrote: > From 643006c4be514dd513232f7438b17add2a763685 Mon Sep 17 00:00:00 2001 > From: mmironov > Date: Tue, 14 Nov 2017 17:54:24 -0500 > Subject: [PATCH] Added HW H.264 and HEVC encoding for AMD GPUs based on AMF > SDK > > Signed-off-by: mmiro

Re: [FFmpeg-devel] [PATCH] avcodec/proresdec: align dequantization matrix buffers

2017-11-15 Thread Carl Eugen Hoyos
2017-11-16 3:51 GMT+01:00 James Almer : > configure can't even succeed with --toolchain=hardened on mingw-w64, so > looks like i wouldn't have been able to reproduce it to begin with. Try replacing toolchain with --extra-cflags=-fstack-protector-all: $ configure --extra-cflags=-fstack-protector-a

[FFmpeg-devel] [PATCH] avformat/hlsenc: checking return value of avio_open_dyn_buf

2017-11-15 Thread Steven Liu
fix CID: 1421196 Signed-off-by: Steven Liu --- libavformat/hlsenc.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c index e0cef8b879..54342cf288 100644 --- a/libavformat/hlsenc.c +++ b/libavformat/hlsenc.c @@ -1678,7 +1678,11 @

Re: [FFmpeg-devel] [PATCH] avcodec/proresdec: align dequantization matrix buffers

2017-11-15 Thread James Almer
On 11/15/2017 11:01 PM, Carl Eugen Hoyos wrote: > 2017-11-16 2:52 GMT+01:00 James Almer : >> On 11/15/2017 10:35 PM, Carl Eugen Hoyos wrote: >>> 2017-11-16 2:29 GMT+01:00 James Almer : >>> The OP configure line is a massive dump of pointless "--disable" options typical from Gentoo builds,

Re: [FFmpeg-devel] [PATCH v3 2/2] avformat/{http, tls}: enable tcp_nodelay

2017-11-15 Thread Aman Gupta
On Wed, Nov 15, 2017 at 2:35 AM, Jeyapal, Karthick wrote: > >Hmm, I guess if we want this to propagate down we need to add the option > to the tls protocol as well, then pass it down when it instantiates the tcp > protocol. Then add the same option to hls/dash, and again propagate it > >down to t

Re: [FFmpeg-devel] [PATCH v3 1/2] avcodec/mpeg12dec: parse A53 caption data embedded in SCTE-20 user data

2017-11-15 Thread Aman Gupta
On Tue, Nov 14, 2017 at 5:15 PM, Michael Niedermayer wrote: > On Tue, Nov 14, 2017 at 11:18:57AM -0800, Aman Gupta wrote: > > From: Aman Gupta > > > > Signed-off-by: Aman Gupta > > --- > > libavcodec/mpeg12dec.c | 39 ++ > + > > tests/fate/subtitles.ma

[FFmpeg-devel] [PATCH] configure: use subarch instead of arch to create .def files on mingw

2017-11-15 Thread James Almer
arch is "x86" regardless of target being x86_32 or x86_64, and if configuring with asm disabled it's "c" instead. Using subarch (Always either "x86_32" or "x86_64") and adapting makedef makes sure the symbols are always detected correctly on x86_32. --- compat/windows/makedef | 2 +- configure

Re: [FFmpeg-devel] [PATCH] avcodec/proresdec: align dequantization matrix buffers

2017-11-15 Thread Carl Eugen Hoyos
2017-11-16 2:52 GMT+01:00 James Almer : > On 11/15/2017 10:35 PM, Carl Eugen Hoyos wrote: >> 2017-11-16 2:29 GMT+01:00 James Almer : >> >>> The OP configure line is a massive dump of pointless >>> "--disable" options typical from Gentoo builds, >> >> Good to know we agree on something. >> >>> so i

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/mpeg12dec: avoid adding PANSCAN side data unless present

2017-11-15 Thread Michael Niedermayer
On Tue, Nov 14, 2017 at 08:38:51PM -0800, Aman Gupta wrote: > On Tue, Nov 14, 2017 at 4:01 PM Michael Niedermayer > wrote: > > > On Tue, Nov 14, 2017 at 10:04:02AM -0800, Aman Gupta wrote: > > > From: Aman Gupta > > > > > > --- > > > libavcodec/mpeg12dec.c | 20 ++-- > > > 1 fil

Re: [FFmpeg-devel] [Patch] Download dash content with byte range info

2017-11-15 Thread Michael Niedermayer
On Thu, Nov 16, 2017 at 12:59:51AM +, Colin NG wrote: > Made change suggested by Carl and add some minor fixes. please submit git patches so they can be applied automatically (git format-patch / send-email) while technically this can be applied, it cant in practice as the whole email ends in

Re: [FFmpeg-devel] [PATCH] avfoundation, unsupported framerate/activeVideoMinFrameDuration

2017-11-15 Thread Michael Niedermayer
On Wed, Nov 15, 2017 at 02:49:40AM +, Roman Puttkammer wrote: > Hi, > > Attached patch fixes an issue with avfoundation; the code continues to > loop through > the list of > supported formats/framerates even after finding the chosen one, ending up > with a bad format/rate > combination. (Code

Re: [FFmpeg-devel] [PATCH] avcodec/proresdec: align dequantization matrix buffers

2017-11-15 Thread James Almer
On 11/15/2017 10:35 PM, Carl Eugen Hoyos wrote: > 2017-11-16 2:29 GMT+01:00 James Almer : > >> The OP configure line is a massive dump of pointless >> "--disable" options typical from Gentoo builds, > > Good to know we agree on something. > >> so i didn't even bother looking at it for specific t

Re: [FFmpeg-devel] [Patch] Download dash content with byte range info

2017-11-15 Thread Steven Liu
s/get_Fragment/get_fragment_range/g Maybe better > 在 2017年11月16日,上午8:59,Colin NG 写道: > > Made change suggested by Carl and add some minor fixes. > > > > From: ffmpeg-devel on behalf of Carl Eugen > Hoyos > Sent: November 15, 2017 5:37 PM > To: FFmpeg de

Re: [FFmpeg-devel] [DEVEL][PATCH v3] ffmpeg: fix channel_layout bug on non-default layout

2017-11-15 Thread Michael Niedermayer
On Tue, Nov 14, 2017 at 11:49:26PM +0100, pkv.stream wrote: > Le 14/11/2017 à 1:13 PM, Michael Niedermayer a écrit : > >On Sun, Nov 12, 2017 at 06:26:18PM +0100, pkv.stream wrote: > >>Le 12/11/2017 à 5:38 PM, Michael Niedermayer a écrit : > >>>On Sun, Nov 12, 2017 at 05:07:04PM +0100, Kv Pham wrote

Re: [FFmpeg-devel] [PATCH] libavcodec/videotoolbox: fix decoding of h264 streams with minor SPS changes

2017-11-15 Thread Aman Gupta
On Wed, Nov 15, 2017 at 1:57 PM, Hendrik Leppkes wrote: > On Wed, Nov 15, 2017 at 10:15 PM, Aman Gupta wrote: > > From: Aman Gupta > > > > Previously the codec kept an entire copy of the SPS, and restarted the > VT decoder > > session whenever it changed. This fixed decoding errors in [1], as >

Re: [FFmpeg-devel] [PATCH] avcodec/proresdec: align dequantization matrix buffers

2017-11-15 Thread Carl Eugen Hoyos
2017-11-16 2:29 GMT+01:00 James Almer : > The OP configure line is a massive dump of pointless > "--disable" options typical from Gentoo builds, Good to know we agree on something. > so i didn't even bother looking at it for specific things. > And now that i look at yours they are completely di

Re: [FFmpeg-devel] [PATCH] avcodec/proresdec: align dequantization matrix buffers

2017-11-15 Thread James Almer
On 11/15/2017 9:50 PM, Carl Eugen Hoyos wrote: > 2017-11-16 1:49 GMT+01:00 James Almer : >> On 11/15/2017 9:33 PM, Carl Eugen Hoyos wrote: >>> 2017-11-16 1:29 GMT+01:00 James Almer : On 11/15/2017 9:21 PM, Carl Eugen Hoyos wrote: > 2017-11-16 1:14 GMT+01:00 James Almer : >> Should fix

Re: [FFmpeg-devel] Check size of STSC allocation

2017-11-15 Thread Fredrik Hubinette
Fixed indentation. On Wed, Nov 15, 2017 at 3:40 PM, Carl Eugen Hoyos wrote: > 2017-11-16 0:21 GMT+01:00 Fredrik Hubinette org>: > > This patch checks that the memory allocated for stsc entries isn't larger > > than the atom. > > Consider fixing the indentation of the second added line, > makin

Re: [FFmpeg-devel] [Patch] Download dash content with byte range info

2017-11-15 Thread Colin NG
Made change suggested by Carl and add some minor fixes. From: ffmpeg-devel on behalf of Carl Eugen Hoyos Sent: November 15, 2017 5:37 PM To: FFmpeg development discussions and patches Subject: Re: [FFmpeg-devel] [Patch] Download dash content with byte range inf

Re: [FFmpeg-devel] [PATCH] avcodec/proresdec: align dequantization matrix buffers

2017-11-15 Thread Carl Eugen Hoyos
2017-11-16 1:49 GMT+01:00 James Almer : > On 11/15/2017 9:33 PM, Carl Eugen Hoyos wrote: >> 2017-11-16 1:29 GMT+01:00 James Almer : >>> On 11/15/2017 9:21 PM, Carl Eugen Hoyos wrote: 2017-11-16 1:14 GMT+01:00 James Almer : > Should fix ticket #6838 > Untested as i can't reproduce.

Re: [FFmpeg-devel] [PATCH] avcodec/proresdec: align dequantization matrix buffers

2017-11-15 Thread James Almer
On 11/15/2017 9:33 PM, Carl Eugen Hoyos wrote: > 2017-11-16 1:29 GMT+01:00 James Almer : >> On 11/15/2017 9:21 PM, Carl Eugen Hoyos wrote: >>> 2017-11-16 1:14 GMT+01:00 James Almer : Should fix ticket #6838 >>> Untested as i can't reproduce. >>> >>> What did you try? >> >> GCC 7.2.0 mingw

Re: [FFmpeg-devel] [PATCH] avcodec/proresdec: align dequantization matrix buffers

2017-11-15 Thread Carl Eugen Hoyos
2017-11-16 1:29 GMT+01:00 James Almer : > On 11/15/2017 9:21 PM, Carl Eugen Hoyos wrote: >> 2017-11-16 1:14 GMT+01:00 James Almer : >>> Should fix ticket #6838 >> >>> Untested as i can't reproduce. >> >> What did you try? > > GCC 7.2.0 mingw-w64 x86_64. The buffers were sufficiently > aligned every

Re: [FFmpeg-devel] [PATCH] avcodec/proresdec: align dequantization matrix buffers

2017-11-15 Thread James Almer
On 11/15/2017 9:21 PM, Carl Eugen Hoyos wrote: > 2017-11-16 1:14 GMT+01:00 James Almer : >> Should fix ticket #6838 > >> Untested as i can't reproduce. > > What did you try? GCC 7.2.0 mingw-w64 x86_64. The buffers were sufficiently aligned every time i decoded the file without this patch. > >

Re: [FFmpeg-devel] [PATCH] avcodec/proresdec: align dequantization matrix buffers

2017-11-15 Thread Carl Eugen Hoyos
2017-11-16 1:14 GMT+01:00 James Almer : > Should fix ticket #6838 > Untested as i can't reproduce. What did you try? Thank you! Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH] avcodec/proresdec: align dequantization matrix buffers

2017-11-15 Thread James Almer
Should fix ticket #6838 Signed-off-by: James Almer --- Untested as i can't reproduce. libavcodec/proresdec2.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/proresdec2.c b/libavcodec/proresdec2.c index e9e0153ee9..d97e264e44 100644 --- a/libavcodec/proresdec2

Re: [FFmpeg-devel] FFmpeg 3.4.1

2017-11-15 Thread Carl Eugen Hoyos
2017-11-15 13:34 GMT+01:00 Michael Niedermayer : > Hi all > > I intend to make 3.4.1 very soon Shouldn't we first decide on how to proceed with #6775? Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ff

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/snowdec: Check intra block dc differences.

2017-11-15 Thread Michael Niedermayer
On Wed, Nov 15, 2017 at 03:26:42PM -0500, Ronald S. Bultje wrote: > Hi, > > On Wed, Nov 15, 2017 at 3:17 PM, Michael Niedermayer > wrote: > > > Fixes: Timeout > > Fixes: 3142/clusterfuzz-testcase-5007853163118592 > > > > Found-by: continuous fuzzing process https://github.com/google/oss- > > fuz

Re: [FFmpeg-devel] Check size of STSC allocation

2017-11-15 Thread Carl Eugen Hoyos
2017-11-16 0:21 GMT+01:00 Fredrik Hubinette : > This patch checks that the memory allocated for stsc entries isn't larger > than the atom. Consider fixing the indentation of the second added line, making the committer's life easier. Thank you, Carl Eugen __

[FFmpeg-devel] Check size of STSC allocation

2017-11-15 Thread Fredrik Hubinette
This patch checks that the memory allocated for stsc entries isn't larger than the atom. Without this, corrupt data can easily try to allocate all the memory, causing a crash. From e79de93ea8426985d63a2369e8274ceaf296ba2d Mon Sep 17 00:00:00 2001 From: Fredrik Hubinette Date: Wed, 15 Nov 2017 15:1

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/snowdec: Check intra block dc differences.

2017-11-15 Thread Carl Eugen Hoyos
2017-11-15 21:26 GMT+01:00 Ronald S. Bultje : > Hi, > > On Wed, Nov 15, 2017 at 3:17 PM, Michael Niedermayer > wrote: > >> Fixes: Timeout >> Fixes: 3142/clusterfuzz-testcase-5007853163118592 >> >> Found-by: continuous fuzzing process https://github.com/google/oss- >> fuzz/tree/master/projects/ffmpe

Re: [FFmpeg-devel] [Patch] Download dash content with byte range info

2017-11-15 Thread Carl Eugen Hoyos
2017-11-15 22:00 GMT+01:00 Colin NG : > This patch is partial fix for ticket 6658 (Dash demuxer segfault). > +static struct fragment * get_Fragment(char *range) { > +struct fragment * seg = av_mallocz(sizeof(struct fragment)); Please make it (code-style): ... fragment *get_Fragment... ... fr

Re: [FFmpeg-devel] [PATCH] avfilter/vf_cropdetect: change license to LGPL

2017-11-15 Thread Carl Eugen Hoyos
2017-11-14 23:35 GMT+01:00 Aman Gupta : > From: Aman Gupta > > This filter was imported from mplayer's libmpcodecs/vf_cropdetect.c, and > marked as GPL when originally copied over. > > On the mplayer side, it was authored originally by arpi and subsequently > touched by > the following contributo

Re: [FFmpeg-devel] [PATCH] libavcodec/videotoolbox: fix decoding of h264 streams with minor SPS changes

2017-11-15 Thread Hendrik Leppkes
On Wed, Nov 15, 2017 at 10:15 PM, Aman Gupta wrote: > From: Aman Gupta > > Previously the codec kept an entire copy of the SPS, and restarted the VT > decoder > session whenever it changed. This fixed decoding errors in [1], as > described in 9519983c. On further inspection, that sample features

Re: [FFmpeg-devel] [PATCH] avformat/avio: remove must_flush from AVIOContext

2017-11-15 Thread Marton Balint
On Wed, 15 Nov 2017, Nicolas George wrote: Le quintidi 25 brumaire, an CCXXVI, Marton Balint a écrit : It is unused. Signed-off-by: Marton Balint --- libavformat/avio.h | 1 - 1 file changed, 1 deletion(-) Breaks ABI by moving all the fields below. We are still in unstable ABI period AF

[FFmpeg-devel] [PATCH] avfilter/vf_cropdetect: change license to LGPL

2017-11-15 Thread Aman Gupta
From: Aman Gupta This filter was imported from mplayer's libmpcodecs/vf_cropdetect.c, and marked as GPL when originally copied over. On the mplayer side, it was authored originally by arpi and subsequently touched by the following contributors. See https://github.com/pigoz/mplayer-svn/commits/

Re: [FFmpeg-devel] [PATCH] avformat/avio: remove must_flush from AVIOContext

2017-11-15 Thread Nicolas George
Le quintidi 25 brumaire, an CCXXVI, Marton Balint a écrit : > It is unused. > > Signed-off-by: Marton Balint > --- > libavformat/avio.h | 1 - > 1 file changed, 1 deletion(-) Breaks ABI by moving all the fields below. Regards, -- Nicolas George signature.asc Description: Digital signatur

[FFmpeg-devel] [PATCH] avformat/avio: remove must_flush from AVIOContext

2017-11-15 Thread Marton Balint
It is unused. Signed-off-by: Marton Balint --- libavformat/avio.h | 1 - 1 file changed, 1 deletion(-) diff --git a/libavformat/avio.h b/libavformat/avio.h index 76ff7cd81e..75912ce6be 100644 --- a/libavformat/avio.h +++ b/libavformat/avio.h @@ -236,7 +236,6 @@ typedef struct AVIOContext {

[FFmpeg-devel] [PATCH] libavcodec/videotoolbox: fix decoding of h264 streams with minor SPS changes

2017-11-15 Thread Aman Gupta
From: Aman Gupta Previously the codec kept an entire copy of the SPS, and restarted the VT decoder session whenever it changed. This fixed decoding errors in [1], as described in 9519983c. On further inspection, that sample features an SPS change from High/4.0 to High/3.2 while moving from one s

Re: [FFmpeg-devel] [Patch] Download dash content with byte range info

2017-11-15 Thread Colin NG
This patch is partial fix for ticket 6658 (Dash demuxer segfault). From: ffmpeg-devel on behalf of Steven Liu Sent: November 15, 2017 2:58 AM To: FFmpeg development discussions and patches Subject: Re: [FFmpeg-devel] [Patch] Download dash content with byte rang

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/snowdec: Check intra block dc differences.

2017-11-15 Thread Ronald S. Bultje
Hi, On Wed, Nov 15, 2017 at 3:17 PM, Michael Niedermayer wrote: > Fixes: Timeout > Fixes: 3142/clusterfuzz-testcase-5007853163118592 > > Found-by: continuous fuzzing process https://github.com/google/oss- > fuzz/tree/master/projects/ffmpeg > Signed-off-by: Michael Niedermayer > --- > libavcode

[FFmpeg-devel] [PATCH 1/2] avcodec/snowdec: Check intra block dc differences.

2017-11-15 Thread Michael Niedermayer
Fixes: Timeout Fixes: 3142/clusterfuzz-testcase-5007853163118592 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/snowdec.c | 19 +++ 1 file changed, 15 insertions(+), 4 deletion

[FFmpeg-devel] [PATCH 2/2] avcodec/snowdec: Check for remaining bitstream in decode_blocks()

2017-11-15 Thread Michael Niedermayer
Fixes: Timeout Fixes: 3142/clusterfuzz-testcase-5007853163118592 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/snowdec.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/snowd

Re: [FFmpeg-devel] [PATCH 1/1] avdevice/decklink_dec: Autodetect the video input format

2017-11-15 Thread Marton Balint
On Tue, 7 Nov 2017, Jeyapal, Karthick wrote: On 11/6/17, 6:19 AM, "Marton Balint" wrote: On Fri, 27 Oct 2017, Jeyapal, Karthick wrote: Please find the patch attached. Thanks, below some comments: Thanks a lot for your detailed comments. I have incorporated all your comments in this

[FFmpeg-devel] [PATCH] hwcontext_d3d11va: add missing stdint.h include

2017-11-15 Thread Timo Rothenpieler
This caused checkheaders to fail on cygwin on this file. --- libavutil/hwcontext_d3d11va.h | 1 + 1 file changed, 1 insertion(+) diff --git a/libavutil/hwcontext_d3d11va.h b/libavutil/hwcontext_d3d11va.h index 98db7ce343..9f91e9b1b6 100644 --- a/libavutil/hwcontext_d3d11va.h +++ b/libavutil/hwcon

[FFmpeg-devel] [PATCH] lavf/movenc: allow writing avc3 sample entry type

2017-11-15 Thread John Stebbins
The avc3 sample entry type is useful for adaptive streaming. It permits parameter sets to be written inline in the video stream. --- libavformat/movenc.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavformat/movenc.c b/libavformat/movenc.c index cc3fc19d9b..01ae467fa1 100644 --- a/libav

Re: [FFmpeg-devel] [PATCH]lavf/matroskaenc: Do not write 0 duration for subtitles

2017-11-15 Thread John Stebbins
On 11/14/2017 02:18 PM, Carl Eugen Hoyos wrote: > 2017-11-14 19:02 GMT+01:00 Jerome Martinez : >> On 12/11/2017 03:12, Carl Eugen Hoyos wrote: >>> -put_ebml_uint(pb, MATROSKA_ID_BLOCKDURATION, duration); >>> +if (duration > 0) >>> +put_ebml_uint(pb, MATROSKA

Re: [FFmpeg-devel] [PATCH] vf_zscale: Add more supported input properties

2017-11-15 Thread Carl Eugen Hoyos
2017-11-15 17:26 GMT+01:00 Vittorio Giovara : > There is a stable release with this code, and it's a minor update, > not sure if it warrants a configure check, but I'll add it if requested. Does it compile without? Does configure succeed? The patch must not lead to a situation where configure pa

[FFmpeg-devel] [PATCH] vf_zscale: Add more supported input properties

2017-11-15 Thread Vittorio Giovara
--- Now without mxf code >_> There is a stable release with this code, and it's a minor update, not sure if it warrants a configure check, but I'll add it if requested. Vittorio libavfilter/vf_zscale.c | 26 -- 1 file changed, 24 insertions(+), 2 deletions(-) diff --git

[FFmpeg-devel] [PATCH] avcodec/zmbv: Check that the buffer is large enough for mvec

2017-11-15 Thread Michael Niedermayer
Fixes: Timeout Fixes: 4143/clusterfuzz-testcase-4736864637419520 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --- libavcodec/zmbv.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/zmbv.c b

Re: [FFmpeg-devel] [PATCH 00/15] OpenCL infrastructure, filters

2017-11-15 Thread Mark Thompson
On 15/11/17 15:39, Carl Eugen Hoyos wrote: > 2017-11-15 15:34 GMT+01:00 Mark Thompson : >> On 14/11/17 22:30, Carl Eugen Hoyos wrote: >>> 2017-11-14 20:47 GMT+01:00 Mark Thompson : >>> Silly example using everything (for i965 VAAPI + Beignet): ./ffmpeg_g -y -init_hw_device vaapi=va:/

Re: [FFmpeg-devel] [Patch] Fix for ticket 6658 (Dash demuxer segfault)

2017-11-15 Thread Colin NG
Have submitted already for the byte range issue in another email with title "Download dash content with byte range info" From: ffmpeg-devel on behalf of Carl Eugen Hoyos Sent: November 15, 2017 10:54 AM To: FFmpeg development discussions and patches Subject: R

Re: [FFmpeg-devel] [Patch] Fix for ticket 6658 (Dash demuxer segfault)

2017-11-15 Thread Carl Eugen Hoyos
2017-11-15 3:54 GMT+01:00 Colin NG : > - Add a function to handle the base URL Processing described in section 5.6.5 > of IEC_23009-1. > > - Fix for downloading dash content with byte range info As said: If these are two separate issues, please send two patches. Use tools/patcheck to check your

Re: [FFmpeg-devel] libavcodec/als: remove check for predictor order of a block

2017-11-15 Thread Carl Eugen Hoyos
2017-11-15 8:54 GMT+01:00 Umair Khan : > I'm now moving to the ALS encoder tasks before we start with the next GSoC. Great! Thank you, Carl Eugen ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH 3/3] lavf/qsv_vpp: fix compiling warning

2017-11-15 Thread Carl Eugen Hoyos
2017-11-15 10:04 GMT+01:00 Zhong Li : > fix the compiling warning of "ignoring return value" > -ff_formats_ref(in_fmts, &ctx->inputs[0]->out_formats); > -ff_formats_ref(out_fmts, &ctx->outputs[0]->in_formats); > +if ((ret = ff_formats_ref(in_fmts, &ctx->inputs[0]->out_formats)) < 0) >

Re: [FFmpeg-devel] [PATCH 00/15] OpenCL infrastructure, filters

2017-11-15 Thread Carl Eugen Hoyos
2017-11-15 15:34 GMT+01:00 Mark Thompson : > On 14/11/17 22:30, Carl Eugen Hoyos wrote: >> 2017-11-14 20:47 GMT+01:00 Mark Thompson : >> >>> Silly example using everything (for i965 VAAPI + Beignet): >>> >>> ./ffmpeg_g -y -init_hw_device vaapi=va:/dev/dri/renderD128 >>> -init_hw_device opencl=ocl@v

Re: [FFmpeg-devel] [PATCH 00/15] OpenCL infrastructure, filters

2017-11-15 Thread Mark Thompson
On 14/11/17 22:30, Carl Eugen Hoyos wrote: > 2017-11-14 20:47 GMT+01:00 Mark Thompson : > >> Silly example using everything (for i965 VAAPI + Beignet): >> >> ./ffmpeg_g -y -init_hw_device vaapi=va:/dev/dri/renderD128 >> -init_hw_device opencl=ocl@va -hwaccel vaapi >> -hwaccel_device va -hwaccel_ou

Re: [FFmpeg-devel] [PATCH 1/3] libavformat/avio: Utility function to return URLContext

2017-11-15 Thread Nicolas George
Le septidi 17 brumaire, an CCXXVI, Karthick J a écrit : > --- > libavformat/avio_internal.h | 8 > libavformat/aviobuf.c | 8 > 2 files changed, 16 insertions(+) > > diff --git a/libavformat/avio_internal.h b/libavformat/avio_internal.h > index c01835d..04c1ad5 100644 > --

[FFmpeg-devel] FFmpeg 3.4.1

2017-11-15 Thread Michael Niedermayer
Hi all I intend to make 3.4.1 very soon, if you want something in it, please backport&push to release/3.4 thx -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB You can kill me, but you cannot change the truth. signature.asc Description: Digital signature

Re: [FFmpeg-devel] [PATCH] avformat/subfile: allow to extract till EOF

2017-11-15 Thread Gyan Doshi
Disregard patch in first message. Didn't remove residue of aborted doc change earlier. Regards, Gyan From 418879680f6d009b37aea29f667a5df142a83f83 Mon Sep 17 00:00:00 2001 From: Gyan Doshi Date: Wed, 15 Nov 2017 17:36:19 +0530 Subject: [PATCH] avformat/subfile: allow to extract till EOF Users

[FFmpeg-devel] [PATCH] avformat/subfile: allow to extract till EOF

2017-11-15 Thread Gyan Doshi
Attached patch allows users to set end offset as 0 to extract till end of file. Tested locally and documented. Regards, Gyan From e5c555774f88639f0e9337f1b55733d0ad156b64 Mon Sep 17 00:00:00 2001 From: Gyan Doshi Date: Wed, 15 Nov 2017 17:12:30 +0530 Subject: [PATCH] avformat/subfile: allow to

Re: [FFmpeg-devel] [PATCH 1/3] libavformat/avio: Utility function to return URLContext

2017-11-15 Thread Jeyapal, Karthick
>>On Tue, Nov 7, 2017 at 2:34 AM Karthick J wrote: >>--- >> libavformat/avio_internal.h | 8 >> libavformat/aviobuf.c       | 8 >> 2 files changed, 16 insertions(+) > >LGTM. This would make my hls demuxer keepalive patch simpler as well. > >I know there were some concerns earlier

Re: [FFmpeg-devel] [PATCH v3 2/2] avformat/{http, tls}: enable tcp_nodelay

2017-11-15 Thread Jeyapal, Karthick
>Hmm, I guess if we want this to propagate down we need to add the option to >the tls protocol as well, then pass it down when it instantiates the tcp >protocol. Then add the same option to hls/dash, and again propagate it >down >to tls or tcp. > >This is turning into a much bigger change than I

Re: [FFmpeg-devel] [PATCH 2/2] nvenc: support d3d11 surface input

2017-11-15 Thread Hendrik Leppkes
On Sun, Nov 12, 2017 at 3:32 PM, Timo Rothenpieler wrote: > Am 12.11.2017 um 10:30 schrieb Hendrik Leppkes: >> >> --- >> libavcodec/nvenc.c | 106 >> ++--- >> libavcodec/nvenc.h | 11 +- >> 2 files changed, 95 insertions(+), 22 deletions(-)

[FFmpeg-devel] [PATCH 3/3] lavf/qsv_vpp: fix compiling warning

2017-11-15 Thread Zhong Li
fix the compiling warning of "ignoring return value" Signed-off-by: Zhong Li --- libavfilter/vf_vpp_qsv.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/libavfilter/vf_vpp_qsv.c b/libavfilter/vf_vpp_qsv.c index eb2f1cc..bc059cc 100644 --- a/libavfilter/vf_vpp_qsv.c ++

[FFmpeg-devel] [PATCH] Add FAQs about running in background (rev 2)

2017-11-15 Thread Jim DeLaHunt
Add two FAQs about running FFmpeg in the background. The first explains the use of the -nostdin option in a straightforward way. Text revised based on review. The second FAQ starts from a confusing error message, and leads to the solution, use of the -nostdin option. The purpose of the second FAQ

Re: [FFmpeg-devel] [PATCH v3 1/2] avformat/tcp: add option to enable TCP_NODELAY

2017-11-15 Thread Steven Liu
2017-11-15 13:01 GMT+08:00 Aman Gupta : > From: Aman Gupta > > This can reduce latency and increase throughput, particularly on high > latency networks. > > Signed-off-by: Aman Gupta > Reviewed-by: Jeyapal, Karthick > --- > doc/protocols.texi| 3 +++ > libavformat/network.h | 1 + > libavfo