[FFmpeg-devel] [PATCH] Check return value of read in ff_rtsp_read_reply()

2017-04-16 Thread Daniel Richard G.
In the course of testing RTSP streaming of CCTV video via the FFmpeg API, I have found some Valgrind uninitialized-memory errors due to what appear to be short/failed reads in ffurl_read_complete(). The calling function ff_rtsp_read_reply() was not checking the return value, and so the library wen

[FFmpeg-devel] Fwd: [PATCH] added expr evaluation to drawtext - fontsize

2017-04-16 Thread Brett Harrison
Any comments on this patch? -- Forwarded message -- From: Brett Harrison Date: Tue, Apr 11, 2017 at 1:37 PM Subject: Re: [FFmpeg-devel] [PATCH] added expr evaluation to drawtext - fontsize To: FFmpeg development discussions and patches Pinging for comments / review... On Tue,

Re: [FFmpeg-devel] [PATCH] this is the gsos qualification task which use psychoacoustic system to detect transients in vorbis encoder.

2017-04-16 Thread 汲如意
Hi, I am sorry for my mistakes. I have fixed them. And attachment is the patch which fix error in first one. Thanks. 2017-04-16 15:25 GMT-04:00 Michael Niedermayer : > On Sun, Apr 16, 2017 at 09:19:33AM -0400, Ruyi Ji wrote: > > Signed-off-by: Ruyi Ji > > --- > > libavcodec/psymodel.c|

[FFmpeg-devel] [PATCH 1/2] cmdutils_opencl: Fix read of uninitialized pointer

2017-04-16 Thread Michael Niedermayer
Fixes: CID1396856 Signed-off-by: Michael Niedermayer --- cmdutils_opencl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cmdutils_opencl.c b/cmdutils_opencl.c index 655d1c9546..f141d1e243 100644 --- a/cmdutils_opencl.c +++ b/cmdutils_opencl.c @@ -129,7 +129,7 @@ static int6

[FFmpeg-devel] [PATCH 2/2] cmdutils_opencl: Fix read of uinitialized cl_mem

2017-04-16 Thread Michael Niedermayer
Fixes CID1396856, CID1396860, CID1396861 Signed-off-by: Michael Niedermayer --- cmdutils_opencl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cmdutils_opencl.c b/cmdutils_opencl.c index f141d1e243..906aef4836 100644 --- a/cmdutils_opencl.c +++ b/cmdutils_opencl.c @@ -133,

[FFmpeg-devel] [PATCH] libavutil/thread.h: Fixed g++ build error when ASSERT_LEVEL is greater than 1

2017-04-16 Thread Aaron Levinson
From 9e6a9e2b8d58f17c661a3f455e03c95587ec7b18 Mon Sep 17 00:00:00 2001 From: Aaron Levinson Date: Sun, 16 Apr 2017 17:13:31 -0700 Subject: [PATCH] libavutil/thread.h: Fixed g++ build error when ASSERT_LEVEL is greater than 1 Purpose: libavutil/thread.h: Fixed g++ build error when ASSERT_LEVEL i

Re: [FFmpeg-devel] libavutil/thread.h: Fixed g++ build error when ASSERT_LEVEL is greater than 1

2017-04-16 Thread Aaron Levinson
I'll try that again with [PATCH] in the subject line. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] libavutil/thread.h: Fixed g++ build error when ASSERT_LEVEL is greater than 1

2017-04-16 Thread Aaron Levinson
From 9e6a9e2b8d58f17c661a3f455e03c95587ec7b18 Mon Sep 17 00:00:00 2001 From: Aaron Levinson Date: Sun, 16 Apr 2017 17:13:31 -0700 Subject: [PATCH] libavutil/thread.h: Fixed g++ build error when ASSERT_LEVEL is greater than 1 Purpose: libavutil/thread.h: Fixed g++ build error when ASSERT_LEVEL i

Re: [FFmpeg-devel] [PATCH] avdevice/decklink: Removed pthread dependency

2017-04-16 Thread Aaron Levinson
On 4/16/2017 4:25 PM, Aaron Levinson wrote: On 4/16/2017 3:31 PM, Hendrik Leppkes wrote: On Mon, Apr 17, 2017 at 12:26 AM, Aaron Levinson wrote: On 4/16/2017 1:33 PM, Timo Rothenpieler wrote: Thanks, applied. Regards, Marton This seems to have broken the coverity builds: https://travis-c

Re: [FFmpeg-devel] [PATCH] avdevice/decklink: Removed pthread dependency

2017-04-16 Thread Aaron Levinson
On 4/16/2017 3:31 PM, Hendrik Leppkes wrote: On Mon, Apr 17, 2017 at 12:26 AM, Aaron Levinson wrote: On 4/16/2017 1:33 PM, Timo Rothenpieler wrote: Thanks, applied. Regards, Marton This seems to have broken the coverity builds: https://travis-ci.org/FFmpeg/FFmpeg-Coverity/builds/222597943

Re: [FFmpeg-devel] [PATCH] Made minor changes to get the decklink avdevice code to build using Visual C++

2017-04-16 Thread Aaron Levinson
On 4/15/2017 6:13 AM, Aaron Levinson wrote: > On 4/15/2017 4:19 AM, Marton Balint wrote: >> >> On Thu, 13 Apr 2017, Aaron Levinson wrote: >> >>> On 4/13/2017 1:23 PM, Hendrik Leppkes wrote: >> [...] >> >>>

Re: [FFmpeg-devel] [PATCH] avdevice/decklink: Removed pthread dependency

2017-04-16 Thread Hendrik Leppkes
On Mon, Apr 17, 2017 at 12:26 AM, Aaron Levinson wrote: > On 4/16/2017 1:33 PM, Timo Rothenpieler wrote: >>> >>> Thanks, applied. >>> >>> Regards, >>> Marton >> >> >> This seems to have broken the coverity builds: >> https://travis-ci.org/FFmpeg/FFmpeg-Coverity/builds/222597943#L1103 > > > It was

Re: [FFmpeg-devel] [PATCH] avdevice/decklink: Removed pthread dependency

2017-04-16 Thread Aaron Levinson
On 4/16/2017 1:33 PM, Timo Rothenpieler wrote: Thanks, applied. Regards, Marton This seems to have broken the coverity builds: https://travis-ci.org/FFmpeg/FFmpeg-Coverity/builds/222597943#L1103 It was suggested on IRC by James Almer that "it seems to complain about ubitux's 'strict' pthrea

Re: [FFmpeg-devel] [PATCH] avdevice/decklink: Removed pthread dependency

2017-04-16 Thread Timo Rothenpieler
Thanks, applied. Regards, Marton This seems to have broken the coverity builds: https://travis-ci.org/FFmpeg/FFmpeg-Coverity/builds/222597943#L1103 ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] avdevice/decklink: Removed pthread dependency

2017-04-16 Thread Aaron Levinson
On 4/16/2017 12:15 PM, wm4 wrote: On Sun, 16 Apr 2017 15:23:15 -0300 James Almer wrote: On 4/16/2017 2:17 PM, Aaron Levinson wrote: On 4/15/2017 9:32 PM, James Almer wrote: On 4/15/2017 7:41 AM, Marton Balint wrote: On Thu, 13 Apr 2017, Aaron Levinson wrote: On 4/13/2017 3:40 PM, Marton

Re: [FFmpeg-devel] [PATCH] this is the gsos qualification task which use psychoacoustic system to detect transients in vorbis encoder.

2017-04-16 Thread Michael Niedermayer
On Sun, Apr 16, 2017 at 09:19:33AM -0400, Ruyi Ji wrote: > Signed-off-by: Ruyi Ji > --- > libavcodec/psymodel.c| 1 + > libavcodec/vorbis_enc_data.h | 111 > +++ > libavcodec/vorbisenc.c | 60 +++ > 3 files changed, 172

Re: [FFmpeg-devel] [PATCH] avdevice/decklink: Removed pthread dependency

2017-04-16 Thread wm4
On Sun, 16 Apr 2017 15:23:15 -0300 James Almer wrote: > On 4/16/2017 2:17 PM, Aaron Levinson wrote: > > On 4/15/2017 9:32 PM, James Almer wrote: > >> On 4/15/2017 7:41 AM, Marton Balint wrote: > >>> > >>> On Thu, 13 Apr 2017, Aaron Levinson wrote: > >>> > On 4/13/2017 3:40 PM, Marton B

Re: [FFmpeg-devel] [PATCH] avdevice/decklink: Removed pthread dependency

2017-04-16 Thread James Almer
On 4/16/2017 2:17 PM, Aaron Levinson wrote: > On 4/15/2017 9:32 PM, James Almer wrote: >> On 4/15/2017 7:41 AM, Marton Balint wrote: >>> >>> On Thu, 13 Apr 2017, Aaron Levinson wrote: >>> On 4/13/2017 3:40 PM, Marton Balint wrote: > > On Thu, 13 Apr 2017, Aaron Levinson wrote: > >>

[FFmpeg-devel] [PATCH] wmv2: NEON intrinsics wmv2dsp

2017-04-16 Thread zeas
0001-wmv2-NEON-intrinsics-wmv2dsp.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Re: [FFmpeg-devel] Debug builds and if (ARCH_...) linking issues

2017-04-16 Thread Aaron Levinson
On 4/16/2017 10:08 AM, Ronald S. Bultje wrote: Hi, On Sun, Apr 16, 2017 at 12:59 PM, Aaron Levinson wrote: I like using this approach, but I think such files should only be added to the build if --disable-optimizations is passed into configure. Or you could detect in configure if the compi

Re: [FFmpeg-devel] [PATCH] avdevice/decklink: Removed pthread dependency

2017-04-16 Thread Aaron Levinson
On 4/15/2017 9:32 PM, James Almer wrote: On 4/15/2017 7:41 AM, Marton Balint wrote: On Thu, 13 Apr 2017, Aaron Levinson wrote: On 4/13/2017 3:40 PM, Marton Balint wrote: On Thu, 13 Apr 2017, Aaron Levinson wrote: On 4/13/2017 2:12 AM, Hendrik Leppkes wrote: On Thu, Apr 13, 2017 at 10:36

Re: [FFmpeg-devel] Debug builds and if (ARCH_...) linking issues

2017-04-16 Thread Ronald S. Bultje
Hi, On Sun, Apr 16, 2017 at 12:59 PM, Aaron Levinson wrote: > I like using this approach, but I think such files should only be added to > the build if --disable-optimizations is passed into configure. > Or you could detect in configure if the compiler supports DCE? Ronald

Re: [FFmpeg-devel] Debug builds and if (ARCH_...) linking issues

2017-04-16 Thread Aaron Levinson
On 4/16/2017 8:57 AM, Matt Oliver wrote: On 14 April 2017 at 08:05, Hendrik Leppkes wrote: On Fri, Apr 14, 2017 at 12:00 AM, Ronald S. Bultje wrote: Hi Aaron, On Thu, Apr 13, 2017 at 4:47 PM, Aaron Levinson wrote: #if ARCH_AARCH64 if (ARCH_AARCH64) return ff_get_cpu_flags_aar

[FFmpeg-devel] [PATCH 2/2] avcodec: add Screen Recorder Gold Codec decoder

2017-04-16 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- configure | 1 + libavcodec/Makefile | 1 + libavcodec/allcodecs.c | 1 + libavcodec/avcodec.h| 1 + libavcodec/codec_desc.c | 7 +++ libavcodec/mscc.c | 24 +--- libavformat/riff.c | 1 + 7 files chan

[FFmpeg-devel] [PATCH 1/2] avcodec: add Mandsoft Screen Capture Codec decoder

2017-04-16 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- configure | 1 + doc/general.texi| 1 + libavcodec/Makefile | 1 + libavcodec/allcodecs.c | 1 + libavcodec/avcodec.h| 1 + libavcodec/codec_desc.c | 7 ++ libavcodec/mscc.c | 231

[FFmpeg-devel] [PATCH] doc/fftools-common-opts: Fixed a typo in the common arguments list

2017-04-16 Thread Mickael Maison
- Fixed a typo for the -sources argument Signed-off-by: Mickael Maison --- doc/fftools-common-opts.texi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/doc/fftools-common-opts.texi b/doc/fftools-common-opts.texi index 7269aeece9..2eff33a36f 100644 --- a/doc/fftools-common

[FFmpeg-devel] [PATCH] this is the gsos qualification task which use psychoacoustic system to detect transients in vorbis encoder.

2017-04-16 Thread Ruyi Ji
Signed-off-by: Ruyi Ji --- libavcodec/psymodel.c| 1 + libavcodec/vorbis_enc_data.h | 111 +++ libavcodec/vorbisenc.c | 60 +++ 3 files changed, 172 insertions(+) diff --git a/libavcodec/psymodel.c b/libavcodec/psymodel

Re: [FFmpeg-devel] [PATCH] ffserver: fix memory leaks pointed out by valgrind.

2017-04-16 Thread Zalewa
Hello, Approach 2. This one reverses the shared state created by unlayer_stream() by nullifying the pointers to shared objects and then passes the "cleaned" AVFormatContext to avformat_free_context(). In result we have less code and less meddling with internals. See close_unlayered_format_con