Re: [FFmpeg-devel] [PATCH] libavutil/hwcontext_qsv: Command line using hwaccel 'QSV' doesn't work

2017-01-05 Thread Huang, Zhengxu
Hi According to the suggestion update the patch. thanks. 在 2017/1/3 21:14, Mark Thompson 写道: On 03/01/17 07:13, Huang, Zhengxu wrote: From 687ce9c804b2618f021100235c46a33b48fa522c Mon Sep 17 00:00:00 2001 From: Zhengxu Date: Wed, 14 Dec 2016 11:55:31 +0800

Re: [FFmpeg-devel] mov: support for multiple edits and cenc decryption

2017-01-05 Thread Sasi Inguva
I agree . #4 makes the most sense. On Thu, Jan 5, 2017 at 12:38 PM, Eran Kornblau wrote: > Hi all, > > We found today that this change: > https://github.com/FFmpeg/FFmpeg/commit/ca6cae73db207f17a0d5507609de12 > 842d8f0ca3 > can break the decryption of MP4s encrypted

Re: [FFmpeg-devel] [PATCH] lavc/cuvid: fail early if GPU can't handle given video resolution

2017-01-05 Thread Ruta Gadkari
On Jan 3, 2017 03:38 AM, "Pavel Koshevoy" wrote: >> I don't recall seeing it mentioned that they do not support 422 and >> 444 in nvidia docs > It decodes "fine", except for one green line along the bottom edge of the > frame. > The other file I have that decodes with

Re: [FFmpeg-devel] [PATCH 2/2] avformat/matroskaenc: Regression fix for invalid MKV headers

2017-01-05 Thread James Almer
On 1/5/2017 3:07 PM, Soft Works wrote: > The following three commits created a regression by writing initially > invalid mkv headers: > > 650e17d88b63b5aca6e0a43483e89e64b0f7d2dd avformat/matroskaenc: write a > CRC32 element on Tags > 3bcadf822711720ff0f8d14db71ae47cdf97e652 avformat/matroskaenc:

Re: [FFmpeg-devel] [PATCH 1/4] dxva2: make ff_dxva2_get_surface() static and rename it

2017-01-05 Thread Hendrik Leppkes
On Fri, Jan 6, 2017 at 11:11 AM, Michael Niedermayer wrote: > On Fri, Jan 06, 2017 at 09:52:48AM +1100, Hendrik Leppkes wrote: >> On Fri, Jan 6, 2017 at 9:15 AM, Michael Niedermayer >> wrote: >> > On Wed, Jan 04, 2017 at 02:36:41PM +0100, Steve

[FFmpeg-devel] [PATCH] avcodec/atrac3: use AVCodec.init_static_data() to initialize static data

2017-01-05 Thread James Almer
Signed-off-by: James Almer --- libavcodec/atrac3.c | 8 ++-- 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/libavcodec/atrac3.c b/libavcodec/atrac3.c index 256990b..208762d 100644 --- a/libavcodec/atrac3.c +++ b/libavcodec/atrac3.c @@ -771,7 +771,7 @@ static

Re: [FFmpeg-devel] [PATCH 1/4] dxva2: make ff_dxva2_get_surface() static and rename it

2017-01-05 Thread Michael Niedermayer
On Fri, Jan 06, 2017 at 09:52:48AM +1100, Hendrik Leppkes wrote: > On Fri, Jan 6, 2017 at 9:15 AM, Michael Niedermayer > wrote: > > On Wed, Jan 04, 2017 at 02:36:41PM +0100, Steve Lhomme wrote: > >> --- > >> libavcodec/dxva2.c | 8 > >>

Re: [FFmpeg-devel] [PATCH] configure: disable the new optimizer in Visual Studio 2015 Update 3

2017-01-05 Thread Kacper Michajlow
2016-07-04 3:53 GMT+02:00 Kacper Michajlow : > 2016-07-03 23:39 GMT+02:00 Hendrik Leppkes : >> On Tue, Jun 28, 2016 at 12:01 PM, Hendrik Leppkes >> wrote: >>> On Tue, Jun 28, 2016 at 11:48 AM, Hendrik Leppkes

Re: [FFmpeg-devel] [PATCH 1/4] dxva2: make ff_dxva2_get_surface() static and rename it

2017-01-05 Thread Hendrik Leppkes
On Fri, Jan 6, 2017 at 9:15 AM, Michael Niedermayer wrote: > On Wed, Jan 04, 2017 at 02:36:41PM +0100, Steve Lhomme wrote: >> --- >> libavcodec/dxva2.c | 8 >> libavcodec/dxva2_internal.h | 2 -- >> 2 files changed, 4 insertions(+), 6 deletions(-) > >

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: fix Explicit null dereferenced

2017-01-05 Thread Steven Liu
2017-01-06 2:00 GMT+08:00 Michael Niedermayer : > On Thu, Jan 05, 2017 at 08:40:56AM +0800, Steven Liu wrote: > > CID: 1323076 > > Passing null pointer loc to avformat_new_stream, which dereferences it > > > fix: because the vtt_oc maybe have not value, so fix it. > > hmm

Re: [FFmpeg-devel] [PATCH] avformat/test/fifo_muxer: add check for FailingMuxerPacketData alloc

2017-01-05 Thread Steven Liu
2017-01-06 2:02 GMT+08:00 Michael Niedermayer : > On Thu, Jan 05, 2017 at 10:45:23AM +0800, Steven Liu wrote: > > CID: 1396257 > > Signed-off-by: Steven Liu > > --- > > libavformat/tests/fifo_muxer.c | 3 +++ > > 1 file changed, 3 insertions(+) > >

Re: [FFmpeg-devel] [PATCH 1/2] imdct15: remove the AArch64 assembly

2017-01-05 Thread Rostislav Pehlivanov
On 4 January 2017 at 10:16, Rostislav Pehlivanov wrote: > Prep work for the next commit, which will add a new FFT algorithm > which makes the iMDCT over 3x faster than it is currently (standalone, > the FFT is with some framesizes over 10x faster). > > The new FFT algorithm

Re: [FFmpeg-devel] [PATCH 2/2] imdct15: replace the FFT with a faster PFA FFT algorithm

2017-01-05 Thread Rostislav Pehlivanov
On 4 January 2017 at 22:45, Rostislav Pehlivanov wrote: > > > On 4 January 2017 at 14:14, Peter Barfuss wrote: > >> First off, many thanks. >> >> > +const int inv_1 = l_ptwo << ((4 - b_ptwo) & 3); >> > +const int inv_2 = 0xeeef & ((1U <<

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: fix Explicit null dereferenced in hlsenc

2017-01-05 Thread Steven Liu
2017-01-06 1:53 GMT+08:00 Michael Niedermayer : > On Thu, Jan 05, 2017 at 08:28:56AM +0800, Steven Liu wrote: > > CID: 1398228 > > Passing null pointer dirname to strlen, which dereferences it. > > > > Signed-off-by: Steven Liu > > --- > >

Re: [FFmpeg-devel] [PATCH] doc/muxers/hlsenc: typo hls_flag: discont_starts => discont_start

2017-01-05 Thread Michael Niedermayer
On Thu, Jan 05, 2017 at 08:56:09PM +0800, Steven Liu wrote: > 2017-01-05 20:25 GMT+08:00 Bodecs Bela : > > > Dear All, > > > > in doc/muxers/hlsenc, there is a typo in hls_flags section: > > discont_starts instead of discont_start > > > > > > best regards, > > > > Bela

Re: [FFmpeg-devel] [PATCH 1/4] dxva2: make ff_dxva2_get_surface() static and rename it

2017-01-05 Thread Michael Niedermayer
On Wed, Jan 04, 2017 at 02:36:41PM +0100, Steve Lhomme wrote: > --- > libavcodec/dxva2.c | 8 > libavcodec/dxva2_internal.h | 2 -- > 2 files changed, 4 insertions(+), 6 deletions(-) applied PS: it seems this was posted over a month ago with no reviewes or comments, is some

[FFmpeg-devel] [PATCH] avcodec: add QDMC decoder

2017-01-05 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- Changelog | 1 + doc/general.texi | 1 + libavcodec/Makefile| 1 + libavcodec/allcodecs.c | 1 + libavcodec/qdmc.c | 789 + 5 files changed, 793 insertions(+)

Re: [FFmpeg-devel] [PATCH] avcodec: add QDMC decoder

2017-01-05 Thread James Almer
On 1/5/2017 4:13 PM, Paul B Mahol wrote: >>> +static float sin_table[512]; >>> +static VLC vtable[6]; >> Why are these not part of QDMCContext? > They are static, never change, so having it part of context wastes > memory, with duplicate tables for each instance of decoder. Then i think it might

[FFmpeg-devel] mov: support for multiple edits and cenc decryption

2017-01-05 Thread Eran Kornblau
Hi all, We found today that this change: https://github.com/FFmpeg/FFmpeg/commit/ca6cae73db207f17a0d5507609de12842d8f0ca3 can break the decryption of MP4s encrypted with common encryption. I would like to submit a patch for that, but wanted to consult first about the best approach. The problem

Re: [FFmpeg-devel] [PATCH] avcodec: add QDMC decoder

2017-01-05 Thread Lou Logan
On Thu, 5 Jan 2017 19:34:44 +0100, Paul B Mahol wrote: [...] > +++ b/libavcodec/qdmc.c [...] libavcodec/qdmc.c: In function ‘qdmc_decode_frame’: libavcodec/qdmc.c:463:39: warning: array subscript is above array bounds [-Warray-bounds] s->noise[ch][band][idx] = lastval + k

Re: [FFmpeg-devel] [PATCH 17/17] lavfi/buffersink: move to the new design.

2017-01-05 Thread Nicolas George
Le sextidi 16 nivôse, an CCXXV, Rostislav Pehlivanov a écrit : > Thanks, the excerpt looks fine to me. Thanks for the review. One to go. > btw, there's apparently been a regression w.r.t. EOF handling which wm4 > hasn't told you about yet, not sure which commit introduced it though: > >

Re: [FFmpeg-devel] [PATCH] avcodec: add QDMC decoder

2017-01-05 Thread Paul B Mahol
On 1/5/17, James Almer wrote: > On 1/5/2017 3:34 PM, Paul B Mahol wrote: >> diff --git a/libavcodec/qdmc.c b/libavcodec/qdmc.c >> new file mode 100644 >> index 000..5559db3 >> --- /dev/null >> +++ b/libavcodec/qdmc.c >> @@ -0,0 +1,817 @@ >> +/* >> + * QDMC compatible

Re: [FFmpeg-devel] [PATCH] avcodec: add QDMC decoder

2017-01-05 Thread James Almer
On 1/5/2017 3:34 PM, Paul B Mahol wrote: > diff --git a/libavcodec/qdmc.c b/libavcodec/qdmc.c > new file mode 100644 > index 000..5559db3 > --- /dev/null > +++ b/libavcodec/qdmc.c > @@ -0,0 +1,817 @@ > +/* > + * QDMC compatible decoder > + * Copyright (c) 2017 Paul B Mahol > + * > + * This

[FFmpeg-devel] [PATCH] avcodec: add QDMC decoder

2017-01-05 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- Changelog | 1 + doc/general.texi | 1 + libavcodec/Makefile| 1 + libavcodec/allcodecs.c | 1 + libavcodec/qdmc.c | 817 + 5 files changed, 821 insertions(+)

[FFmpeg-devel] [PATCH 2/2] avformat/matroskaenc: Regression fix for invalid MKV headers

2017-01-05 Thread Soft Works
The following three commits created a regression by writing initially invalid mkv headers: 650e17d88b63b5aca6e0a43483e89e64b0f7d2dd avformat/matroskaenc: write a CRC32 element on Tags 3bcadf822711720ff0f8d14db71ae47cdf97e652 avformat/matroskaenc: write a CRC32 element on Info

[FFmpeg-devel] [PATCH 1/2] libavformat/avio: Add avio_get_dyn_buf function

2017-01-05 Thread Soft Works
This commit adds the avio_get_dyn_buf function which allows accessing the content of a DynBuffer without destroying it. This is required in matroskaenc for preliminary writing (correct) mkv headers. Context for this change is fixing regression bug #5977. --- libavformat/avio.h| 12

Re: [FFmpeg-devel] [PATCH] avformat/test/fifo_muxer: add check for FailingMuxerPacketData alloc

2017-01-05 Thread Michael Niedermayer
On Thu, Jan 05, 2017 at 10:45:23AM +0800, Steven Liu wrote: > CID: 1396257 > Signed-off-by: Steven Liu > --- > libavformat/tests/fifo_muxer.c | 3 +++ > 1 file changed, 3 insertions(+) LGTM thx [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: fix Explicit null dereferenced

2017-01-05 Thread Michael Niedermayer
On Thu, Jan 05, 2017 at 08:40:56AM +0800, Steven Liu wrote: > CID: 1323076 > Passing null pointer loc to avformat_new_stream, which dereferences it > fix: because the vtt_oc maybe have not value, so fix it. hmm i think: this is wrong: hls->vtt_oformat = av_guess_format("webvtt", NULL,

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: fix Explicit null dereferenced in hlsenc

2017-01-05 Thread Michael Niedermayer
On Thu, Jan 05, 2017 at 08:28:56AM +0800, Steven Liu wrote: > CID: 1398228 > Passing null pointer dirname to strlen, which dereferences it. > > Signed-off-by: Steven Liu > --- > libavformat/hlsenc.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff

Re: [FFmpeg-devel] [PATCH 3/5] lavfi/framepool: re-indent after previous commit

2017-01-05 Thread Michael Niedermayer
On Thu, Jan 05, 2017 at 12:11:34PM +0100, Matthieu Bouron wrote: > --- > libavfilter/framepool.c | 54 > - > 1 file changed, 27 insertions(+), 27 deletions(-) LGTM thx [...] -- Michael GnuPG fingerprint:

Re: [FFmpeg-devel] [PATCH 4/5] lavfi/framepool: cosmetic style fixes

2017-01-05 Thread Michael Niedermayer
On Thu, Jan 05, 2017 at 12:11:35PM +0100, Matthieu Bouron wrote: > --- > libavfilter/framepool.c | 6 ++ > 1 file changed, 2 insertions(+), 4 deletions(-) LGTM thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Democracy is the form of government in

Re: [FFmpeg-devel] Add audio support to lavfi/framepool

2017-01-05 Thread Michael Niedermayer
On Thu, Jan 05, 2017 at 12:11:31PM +0100, Matthieu Bouron wrote: > Patchset updated and rebased on master with the following minor changes: > * add av_assert0 in ff_frame_pool_get_{audio,video}_config > * FFFramePool.type declared as enum AVMediaType instead of int > > I was not able to

Re: [FFmpeg-devel] [PATCH 1/5] lavfi/framepool: rename FFVideoFramePool to FFFramePool

2017-01-05 Thread Michael Niedermayer
On Thu, Jan 05, 2017 at 12:11:32PM +0100, Matthieu Bouron wrote: > --- > libavfilter/avfilter.c | 2 +- > libavfilter/avfilter.h | 4 ++-- > libavfilter/framepool.c | 24 > libavfilter/framepool.h | 32 > libavfilter/video.c | 20

Re: [FFmpeg-devel] [PATCH] doc/muxers/hlsenc: typo hls_flag: discont_starts => discont_start

2017-01-05 Thread Steven Liu
2017-01-05 20:25 GMT+08:00 Bodecs Bela : > Dear All, > > in doc/muxers/hlsenc, there is a typo in hls_flags section: > discont_starts instead of discont_start > > > best regards, > > Bela Bodecs > > > ___ > ffmpeg-devel mailing

[FFmpeg-devel] [PATCH] doc/muxers/hlsenc: typo hls_flag: discont_starts => discont_start

2017-01-05 Thread Bodecs Bela
Dear All, in doc/muxers/hlsenc, there is a typo in hls_flags section: discont_starts instead of discont_start best regards, Bela Bodecs >From d6c3375508750886bc7edf82b5282c93e70a8e62 Mon Sep 17 00:00:00 2001 From: Bela Bodecs Date: Thu, 5 Jan 2017 13:21:13 +0100

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: typo in default localtime pattern

2017-01-05 Thread Steven Liu
2017-01-05 18:39 GMT+08:00 Bodecs Bela : > Dear All, > > in hlsenc.c at get_default_pattern_localtime_fmt function the default > pattern contains > %Y%m%d%H%I%S but the original intention was %Y%m%d%H%M%S > Please apply this patch. > > > thank you, > > Bela Bodecs > > > >

[FFmpeg-devel] [PATCH 3/5] lavfi/framepool: re-indent after previous commit

2017-01-05 Thread Matthieu Bouron
--- libavfilter/framepool.c | 54 - 1 file changed, 27 insertions(+), 27 deletions(-) diff --git a/libavfilter/framepool.c b/libavfilter/framepool.c index 588dcb6da2..74235d4886 100644 --- a/libavfilter/framepool.c +++ b/libavfilter/framepool.c @@

[FFmpeg-devel] [PATCH 1/5] lavfi/framepool: rename FFVideoFramePool to FFFramePool

2017-01-05 Thread Matthieu Bouron
--- libavfilter/avfilter.c | 2 +- libavfilter/avfilter.h | 4 ++-- libavfilter/framepool.c | 24 libavfilter/framepool.h | 32 libavfilter/video.c | 20 ++-- 5 files changed, 41 insertions(+), 41 deletions(-) diff

[FFmpeg-devel] [PATCH 2/5] lavfi/framepool: add audio support

2017-01-05 Thread Matthieu Bouron
--- libavfilter/framepool.c | 109 libavfilter/framepool.h | 36 +++- 2 files changed, 144 insertions(+), 1 deletion(-) diff --git a/libavfilter/framepool.c b/libavfilter/framepool.c index 36c6e8fd04..588dcb6da2 100644 ---

[FFmpeg-devel] [PATCH 4/5] lavfi/framepool: cosmetic style fixes

2017-01-05 Thread Matthieu Bouron
--- libavfilter/framepool.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/libavfilter/framepool.c b/libavfilter/framepool.c index 74235d4886..e1f1e2cc41 100644 --- a/libavfilter/framepool.c +++ b/libavfilter/framepool.c @@ -220,9 +220,8 @@ AVFrame

[FFmpeg-devel] Add audio support to lavfi/framepool

2017-01-05 Thread Matthieu Bouron
Patchset updated and rebased on master with the following minor changes: * add av_assert0 in ff_frame_pool_get_{audio,video}_config * FFFramePool.type declared as enum AVMediaType instead of int I was not able to reproduce the issue Michael had with the following command line: ffmpeg -f

[FFmpeg-devel] [PATCH 5/5] lavfi: use an audio frame pool for each link of the filtergraph

2017-01-05 Thread Matthieu Bouron
--- libavfilter/audio.c | 50 -- 1 file changed, 36 insertions(+), 14 deletions(-) diff --git a/libavfilter/audio.c b/libavfilter/audio.c index a18be89293..5996f607b2 100644 --- a/libavfilter/audio.c +++ b/libavfilter/audio.c @@ -27,6 +27,9 @@

Re: [FFmpeg-devel] [PATCH 17/17] lavfi/buffersink: move to the new design.

2017-01-05 Thread Rostislav Pehlivanov
On 5 January 2017 at 10:27, Nicolas George wrote: > Le nonidi 9 nivôse, an CCXXV, Nicolas George a écrit : > > Signed-off-by: Nicolas George > > --- > > libavfilter/buffersink.c | 214 +- > - > > 1 file changed, 56

[FFmpeg-devel] [PATCH] avformat/hlsenc: typo in default localtime pattern

2017-01-05 Thread Bodecs Bela
Dear All, in hlsenc.c at get_default_pattern_localtime_fmt function the default pattern contains %Y%m%d%H%I%S but the original intention was %Y%m%d%H%M%S Please apply this patch. thank you, Bela Bodecs >From 9a11e116ad50bae635ce7c16821975b3d584ddc7 Mon Sep 17 00:00:00 2001 From: Bela

Re: [FFmpeg-devel] [PATCH] nvenc: Make AUD optional for h264_nvenc and hevc_nvenc

2017-01-05 Thread Yogender Gupta
>> There is BUG in Nvidia NVENC when you use AUD for H264 with B-frames, it >> will return corrupted stream, because NVIDIA is inserting AUD type 0 >> (I-frame) before B-frames instead of AUD type 7 (any-frame). Thanks for bringing this to notice. We have added a fix in the Nvidia driver and

Re: [FFmpeg-devel] [PATCH 17/17] lavfi/buffersink: move to the new design.

2017-01-05 Thread Nicolas George
Le nonidi 9 nivôse, an CCXXV, Nicolas George a écrit : > Signed-off-by: Nicolas George > --- > libavfilter/buffersink.c | 214 > +-- > 1 file changed, 56 insertions(+), 158 deletions(-) > > > Minor changes to accomodate the previous

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: fix memleak in hlsenc

2017-01-05 Thread Steven Liu
2017-01-05 16:58 GMT+08:00 Bodecs Bela : > > > 2017.01.05. 9:43 keltezéssel, Steven Liu írta: > >> 2017-01-05 15:47 GMT+08:00 Bodecs Bela : >> >> >>> 2017.01.05. 1:20 keltezéssel, Steven Liu írta: >>> >>> fix CID: 1398364 Resource leak refine the code

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: fix memleak in hlsenc

2017-01-05 Thread Bodecs Bela
2017.01.05. 9:43 keltezéssel, Steven Liu írta: 2017-01-05 15:47 GMT+08:00 Bodecs Bela : 2017.01.05. 1:20 keltezéssel, Steven Liu írta: fix CID: 1398364 Resource leak refine the code of the new options Signed-off-by: Steven Liu ---

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: fix memleak in hlsenc

2017-01-05 Thread Steven Liu
2017-01-05 15:47 GMT+08:00 Bodecs Bela : > > > 2017.01.05. 1:20 keltezéssel, Steven Liu írta: > >> fix CID: 1398364 Resource leak >> refine the code of the new options >> >> Signed-off-by: Steven Liu >> --- >> libavformat/hlsenc.c | 21