[FFmpeg-devel] [PATCH] configure: check for stdatomic.h

2016-11-23 Thread Wan-Teh Chang
From: Anton Khirnov Since this is a C11 feature, it requires -std=c11. Not actually used for anything yet, that will be added in the following commits. This merges libav commit 13f5d2bf75b95a0bfdb9940a5e359a719e242bed. Signed-off-by: Wan-Teh Chang ---

[FFmpeg-devel] [PATCH] avformat/flvenc: fix ticket 5976 and use old commit

2016-11-23 Thread Steven Liu
mythtv have problem with non-seekable dont write duration and filesize and there have problem with some other server and player with 0 value duation and filesize. So add a flv flags to fix the ticket and make a choose for users. Signed-off-by: Steven Liu ---

Re: [FFmpeg-devel] [PATCH] compat/atomics: merge the C11 atomics from Libav

2016-11-23 Thread Michael Niedermayer
On Thu, Nov 24, 2016 at 03:19:33AM +0100, Michael Niedermayer wrote: > On Wed, Nov 23, 2016 at 04:54:51PM -0800, Wan-Teh Chang wrote: > > On Wed, Nov 23, 2016 at 4:08 PM, Michael Niedermayer > > wrote: [...] > > > > > "Testing must be adequate but not excessive. > > > If

Re: [FFmpeg-devel] [PATCH] compat/atomics: merge the C11 atomics from Libav

2016-11-23 Thread Michael Niedermayer
On Wed, Nov 23, 2016 at 04:54:51PM -0800, Wan-Teh Chang wrote: > On Wed, Nov 23, 2016 at 4:08 PM, Michael Niedermayer > wrote: > > > > please read > > http://ffmpeg.org/developer.html > > > > for example it says: > > "Do not commit unrelated changes together." > > > > you

Re: [FFmpeg-devel] [PATCH] mpegpicture: use coded_width/coded_height to allocate frame

2016-11-23 Thread Andreas Cadhalpun
On 23.11.2016 15:01, Michael Niedermayer wrote: > On Mon, Nov 07, 2016 at 10:32:29PM +0100, Andreas Cadhalpun wrote: >> This fixes a heap-buffer-overflow in ff_er_frame_end when decoding mss2 with >> coded_width/coded_height larger than width/height. >> >> Signed-off-by: Andreas Cadhalpun

Re: [FFmpeg-devel] [PATCH] pgssubdec: only set w/h/linesize when allocating data

2016-11-23 Thread Andreas Cadhalpun
On 23.11.2016 14:05, Petri Hintukainen wrote: > ti, 2016-11-22 kello 23:21 +0100, Andreas Cadhalpun kirjoitti: >> On 09.11.2016 23:27, Andreas Cadhalpun wrote: >>> >>> Rects with positive w/h/linesize but no data are invalid. >>> >>> Signed-off-by: Andreas Cadhalpun

Re: [FFmpeg-devel] [PATCH] pgssubdec: only set w/h/linesize when allocating data

2016-11-23 Thread Andreas Cadhalpun
On 23.11.2016 14:48, wm4 wrote: > On Wed, 9 Nov 2016 23:27:04 +0100 > Andreas Cadhalpun wrote: > >> Rects with positive w/h/linesize but no data are invalid. >> >> Signed-off-by: Andreas Cadhalpun >> --- >>

Re: [FFmpeg-devel] [PATCH] flvdec: set need_context_update when changing codec id

2016-11-23 Thread Andreas Cadhalpun
On 23.11.2016 03:26, Michael Niedermayer wrote: > On Fri, Nov 04, 2016 at 10:28:20PM +0100, Andreas Cadhalpun wrote: >> Otherwise the codec context and codecpar might disagree on the codec id, >> triggering asserts in av_parser_parse2. >> >> Signed-off-by: Andreas Cadhalpun

Re: [FFmpeg-devel] [PATCH] aacsbr_fixed: prevent sbr gain from being zero

2016-11-23 Thread Andreas Cadhalpun
On 23.11.2016 03:22, Michael Niedermayer wrote: > On Sun, Nov 13, 2016 at 09:29:11PM +0100, Andreas Cadhalpun wrote: >> It causes division by zero crashes. >> >> Signed-off-by: Andreas Cadhalpun >> --- >> libavcodec/aacsbr_fixed.c | 4 >> 1 file changed, 4

Re: [FFmpeg-devel] [PATCH] compat/atomics: merge the C11 atomics from Libav

2016-11-23 Thread Michael Niedermayer
On Wed, Nov 23, 2016 at 03:44:12PM -0800, Wan-Teh Chang wrote: > From: Anton Khirnov > > Merge the following libav commits: > 13f5d2bf75b95a0bfdb9940a5e359a719e242bed > 4e928ef340ac20325f529d92fcbc51e768085358 > c2755864afadfbaa349e8d583665c86fe99fa90b >

Re: [FFmpeg-devel] [libav-devel] [PATCH] libopusdec: fix out-of-bounds read

2016-11-23 Thread Andreas Cadhalpun
On 23.11.2016 03:07, Michael Niedermayer wrote: > On Mon, Nov 14, 2016 at 09:55:15PM +0100, Andreas Cadhalpun wrote: >> libopusdec.c |6 ++ >> 1 file changed, 6 insertions(+) >> 0b663c14f4a6dae3e1da453239dbe429aef7886e >> 0001-libopusdec-default-to-stereo-for-invalid-number-of-c.patch

Re: [FFmpeg-devel] [PATCH] lavfi/pan: allow negative gain parameters also for other inputs than the first named

2016-11-23 Thread Michael Niedermayer
On Sat, Oct 29, 2016 at 03:03:58PM +0200, Nicolas George wrote: > Le septidi 7 brumaire, an CCXXV, Moritz Barsnick a écrit : > > Expands the parser to also accept the separator '-' in addition to > > '+', and take the negative sign into consideration. > > > > The optional sign for the first

[FFmpeg-devel] [PATCH] compat/atomics: merge the C11 atomics from Libav

2016-11-23 Thread Wan-Teh Chang
From: Anton Khirnov Merge the following libav commits: 13f5d2bf75b95a0bfdb9940a5e359a719e242bed 4e928ef340ac20325f529d92fcbc51e768085358 c2755864afadfbaa349e8d583665c86fe99fa90b bb81ed476569b912a37ed553e756e123b6b13b14 f9a6a80e065cdb95b233978f1d96ec9bc863daa1

[FFmpeg-devel] [PATCH] compat/atomics: merge the C11 atomics from Libav

2016-11-23 Thread Wan-Teh Chang
I found the commits to merge by inspecting the "git log configure" output and the list of Anton Khirnov's commits at http://git.libav.org/?p=libav.git;a=search;h=5cc0057f4910c8c72421b812c8f337ef6c43696c;s=Anton+Khirnov;st=author Wan-Teh Chang Anton Khirnov (1): compat/atomics: merge the C11

[FFmpeg-devel] [PATCH] avutil/tests: add cpu_init to .gitignore and tests/fate

2016-11-23 Thread Wan-Teh Chang
This is a follow-up to commit d84a21207ea83055dc9b6dc1cd6a379f2ea756e7, which added the libavutil/tests/cpu_init.c. Also add |integral| to libavfilter/tests/.gitignore. Signed-off-by: Wan-Teh Chang --- libavfilter/tests/.gitignore | 1 + libavutil/tests/.gitignore | 1 +

[FFmpeg-devel] [PATCH] avutil/tests: add cpu_init to .gitignore and tests/fate

2016-11-23 Thread Wan-Teh Chang
Hi Michael, Please review and apply this patch. I didn't know I also needed to add the new cpu_init test to libavutil/tests/.gitignore and tests/fate/libavutil.mak. I noticed libavfilter/tests/integral being reported as an untracked file when I ran "git status", so I also took care of that in

[FFmpeg-devel] [PATCH] avcodec/nvenc: Remove aspect-ratio decompensation logic

2016-11-23 Thread Philip Langdale
This dubious behaviour in nvenc was finally removed by nvidia, and as we refuse to run on anything older than 7.0, we don't need to keep it around for old versions. Signed-off-by: Philip Langdale --- libavcodec/nvenc.c | 12 1 file changed, 12 deletions(-) diff

Re: [FFmpeg-devel] [PATCH] avutil: fix data race in av_get_cpu_flags().

2016-11-23 Thread Michael Niedermayer
On Wed, Nov 23, 2016 at 11:44:04AM -0800, Wan-Teh Chang wrote: > On Tue, Nov 22, 2016 at 3:32 PM, wm4 wrote: > > > > [libav/compat/atomics/] is emulation code for compilers which don't provide > > C11 atomics. > > All relevant compilers provide them natively (and

Re: [FFmpeg-devel] [PATCH] lavc/vaapi_hevc: fix scaling list duplicate transfer issue.

2016-11-23 Thread Mark Thompson
On 22/11/16 08:16, Jun Zhao wrote: > From 4059bc56cba30b26bc194673968c3a51014e315f Mon Sep 17 00:00:00 2001 > From: Jun Zhao > Date: Tue, 22 Nov 2016 15:58:50 +0800 > Subject: [PATCH] lavc/vaapi_hevc: fix scaling list duplicate transfer issue. > > scaling list is already

Re: [FFmpeg-devel] [PATCH] avutil/cpu: remove the |checked| static variable

2016-11-23 Thread Michael Niedermayer
On Wed, Nov 23, 2016 at 11:21:56AM -0800, Wan-Teh Chang wrote: > Remove the |checked| variable because the invalid value of -1 for > |flags| can be used to indicate the same condition. Also rename |flags| > to |cpu_flags| because there are a local variable and a function > parameter named |flags|

Re: [FFmpeg-devel] Help mixing live audio streams

2016-11-23 Thread Lou Logan
Hi, On Wed, 23 Nov 2016 12:59:09 -0800, Yan Brenman wrote: > All FFMPEG gurus! > > Can somebody please suggest a format of FFMPEG command to mix several audio > live streams on Windows. [...] Wrong mailing list: ffmpeg-devel is only for patch submissions and discussions related to the

[FFmpeg-devel] Help mixing live audio streams

2016-11-23 Thread Yan Brenman
All FFMPEG gurus! Can somebody please suggest a format of FFMPEG command to mix several audio live streams on Windows. Let my just make it very clear - live audio streams and not existing audio files. Just to put it in the context - lets say audio stream from the microphone and Stereo Mix. With

Re: [FFmpeg-devel] [PATCH] web/consulting: add myself

2016-11-23 Thread Lou Logan
On Wed, 23 Nov 2016 10:41:42 +0800, Steven Liu wrote: > Been getting occassional consulting help emails for a while, so I might > as well add myself here. > > Signed-off-by: Steven Liu > --- > src/consulting | 13 + > 1 files changed, 13 insertions(+), 0

Re: [FFmpeg-devel] [PATCH 2/2] ffserver: Remove extract_mpeg4_header()

2016-11-23 Thread Ronald S. Bultje
Hi, On Wed, Nov 23, 2016 at 2:13 PM, Michael Niedermayer wrote: > This should not be needed, our AVParsers should do this > I do not have a testcase though, please help testing this and please > add fate tests if you can. I thought we were going to remove ffserver?

Re: [FFmpeg-devel] [PATCH] lavfi/avfiltergraph: always reduce all_layouts to a single layout

2016-11-23 Thread Marton Balint
On Wed, 23 Nov 2016, Nicolas George wrote: Le duodi 2 frimaire, an CCXXV, Marton Balint a écrit : Wouldn't it make sense to change the default to allow unknown channel layouts as well, if no query_formats is specified? Only if you volunteer to monitor all merges from the fork to check that

Re: [FFmpeg-devel] [PATCH] avutil: fix data race in av_get_cpu_flags().

2016-11-23 Thread Wan-Teh Chang
On Tue, Nov 22, 2016 at 3:30 PM, wm4 wrote: > On Tue, 22 Nov 2016 23:57:15 +0100 > Michael Niedermayer wrote: > >> For example the progress code in the frame threading. > > Which was recently fixed in Libav AFAIR... You're right.

Re: [FFmpeg-devel] [PATCH] avutil: fix data race in av_get_cpu_flags().

2016-11-23 Thread Wan-Teh Chang
On Tue, Nov 22, 2016 at 3:32 PM, wm4 wrote: > > [libav/compat/atomics/] is emulation code for compilers which don't provide > C11 atomics. > All relevant compilers provide them natively (and presumably implement > the weaker semantics). Thank you for pointing this out.

[FFmpeg-devel] [PATCH] avutil/cpu: remove the |checked| static variable

2016-11-23 Thread Wan-Teh Chang
Remove the |checked| variable because the invalid value of -1 for |flags| can be used to indicate the same condition. Also rename |flags| to |cpu_flags| because there are a local variable and a function parameter named |flags| in the same file. Add a test program libavutil/tests/cpu_init.c to

[FFmpeg-devel] [PATCH 1/2] Revert "ffserver: use AVStream.codecpar in open_input_stream()"

2016-11-23 Thread Michael Niedermayer
Fixes null pointer dereference Testcase is simply a ffmpeg instance sending a stream to ffserver while another ffmpeg reads from it This reverts commit 6f0a1710d77dde0d803861506a2157a23f08c14c. --- ffserver.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ffserver.c

[FFmpeg-devel] [PATCH 2/2] ffserver: Remove extract_mpeg4_header()

2016-11-23 Thread Michael Niedermayer
This should not be needed, our AVParsers should do this I do not have a testcase though, please help testing this and please add fate tests if you can. Signed-off-by: Michael Niedermayer --- ffserver.c | 52 1 file

Re: [FFmpeg-devel] [PATCH] swresample/resample: add force_comp argument

2016-11-23 Thread Muhammad Faiz
On 11/24/16, Muhammad Faiz wrote: > On 11/23/16, Michael Niedermayer wrote: >> On Mon, Nov 21, 2016 at 02:21:07AM +0700, Muhammad Faiz wrote: >>> On 11/21/16, Michael Niedermayer wrote: >>> > On Sun, Nov 20, 2016 at 03:56:28PM

Re: [FFmpeg-devel] [PATCH] swresample/resample: add force_comp argument

2016-11-23 Thread Muhammad Faiz
On 11/23/16, Michael Niedermayer wrote: > On Mon, Nov 21, 2016 at 02:21:07AM +0700, Muhammad Faiz wrote: >> On 11/21/16, Michael Niedermayer wrote: >> > On Sun, Nov 20, 2016 at 03:56:28PM +0700, Muhammad Faiz wrote: >> >> this allow to use

Re: [FFmpeg-devel] [PATCH] lavfi/avfiltergraph: always reduce all_layouts to a single layout

2016-11-23 Thread Nicolas George
Le duodi 2 frimaire, an CCXXV, Marton Balint a écrit : > Wouldn't it make sense to change the default to allow unknown channel > layouts as well, if no query_formats is specified? Only if you volunteer to monitor all merges from the fork to check that they work with unknown layouts. Because that

Re: [FFmpeg-devel] [PATCH] mpegpicture: use coded_width/coded_height to allocate frame

2016-11-23 Thread Michael Niedermayer
On Mon, Nov 07, 2016 at 10:32:29PM +0100, Andreas Cadhalpun wrote: > This fixes a heap-buffer-overflow in ff_er_frame_end when decoding mss2 with > coded_width/coded_height larger than width/height. > > Signed-off-by: Andreas Cadhalpun > --- >

Re: [FFmpeg-devel] [PATCH] pgssubdec: only set w/h/linesize when allocating data

2016-11-23 Thread wm4
On Wed, 9 Nov 2016 23:27:04 +0100 Andreas Cadhalpun wrote: > Rects with positive w/h/linesize but no data are invalid. > > Signed-off-by: Andreas Cadhalpun > --- > libavcodec/pgssubdec.c | 9 + > 1 file changed, 5

Re: [FFmpeg-devel] [PATCH] pgssubdec: only set w/h/linesize when allocating data

2016-11-23 Thread Petri Hintukainen
ti, 2016-11-22 kello 23:21 +0100, Andreas Cadhalpun kirjoitti: > On 09.11.2016 23:27, Andreas Cadhalpun wrote: > > > > Rects with positive w/h/linesize but no data are invalid. > > > > Signed-off-by: Andreas Cadhalpun > > --- > >  libavcodec/pgssubdec.c | 9

[FFmpeg-devel] [PATCH] MAINTAINERS: Add Vittorio Giovara to "Developers with git write access who are currently not maintaining any specific part"

2016-11-23 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index e80cd23..e718463 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -546,6 +546,7 @@ Martin Storsjö Panagiotis Issaris Pedro Arthur

Re: [FFmpeg-devel] [PATCH] swresample/resample: add force_comp argument

2016-11-23 Thread Michael Niedermayer
On Mon, Nov 21, 2016 at 02:21:07AM +0700, Muhammad Faiz wrote: > On 11/21/16, Michael Niedermayer wrote: > > On Sun, Nov 20, 2016 at 03:56:28PM +0700, Muhammad Faiz wrote: > >> this allow to use phase_count_compensation on init (rather than > >> rebuild on