Re: [FFmpeg-devel] [PATCH 2/2] avcodec/pngdec: Clean up on av_frame_ref() failure

2017-09-18 Thread James Almer
On 9/16/2017 9:42 PM, Michael Niedermayer wrote: > Fixes: memleak > Fixes: 3203/clusterfuzz-testcase-minimized-4514553595428864 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg > Signed-off-by: Michael Niedermayer > --- > libavcodec/pngde

Re: [FFmpeg-devel] [PATCH 3/3 v2] avfilter/interlace: add support for 10 and 12 bit

2017-09-18 Thread James Almer
On 9/18/2017 10:41 PM, Thomas Mundt wrote: > I tried to set up MIPS compiler for two days on windows and linux without > success. > Now I try it blind. This solution is based on the first suggestion James > gave me at IRC. > There might be room for improvement and an alternative solution with > AV_

[FFmpeg-devel] [PATCH 3/3 v2] avfilter/interlace: add support for 10 and 12 bit

2017-09-18 Thread Thomas Mundt
I tried to set up MIPS compiler for two days on windows and linux without success. Now I try it blind. This solution is based on the first suggestion James gave me at IRC. There might be room for improvement and an alternative solution with AV_RL16() / AV_WL16(). I used av_le2ne16() because it will

[FFmpeg-devel] [PATCH 2/3 v2] avfilter/tinterlace: use drawutils for pad mode

2017-09-18 Thread Thomas Mundt
Patch 1/3 has already been applied. Patch attached. 0002-avfilter-tinterlace-use-drawutils-for-pad-mode.patch Description: Binary data ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH 2/2] avcodec/ffv1dec: Fix integer overflow in read_quant_table()

2017-09-18 Thread Michael Niedermayer
Fixes: runtime error: signed integer overflow: 2147483647 + 1 cannot be represented in type 'int' Fixes: 3361/clusterfuzz-testcase-minimized-5065842955911168 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermayer --

[FFmpeg-devel] [PATCH 1/2] avcodec/svq3: Fix overflow in svq3_add_idct_c()

2017-09-18 Thread Michael Niedermayer
Fixes: runtime error: signed integer overflow: 2147392585 + 524288 cannot be represented in type 'int' Fixes: 3348/clusterfuzz-testcase-minimized-4809500517203968 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niedermaye

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/wmv2dec: Check end of bitstream in parse_mb_skip() and ff_wmv2_decode_mb()

2017-09-18 Thread Paul B Mahol
On 9/18/17, Carl Eugen Hoyos wrote: > 2017-09-18 15:11 GMT+02:00 Ronald S. Bultje : >> Hi Michael, >> >> On Sun, Sep 17, 2017 at 8:15 PM, Michael Niedermayer >> >> wrote: >> >>> Iam happy to follow what the community prefers. >>> >> >> Some don't like it, some don't care. I think everyone would be

Re: [FFmpeg-devel] [PATCH]lavfi/stereo3d: Set SAR for every output frame

2017-09-18 Thread Paul B Mahol
On 9/18/17, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes ticket #6672. > > Please comment, Carl Eugen > ok ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH]lavfi/stereo3d: Set SAR for every output frame

2017-09-18 Thread Carl Eugen Hoyos
Hi! Attached patch fixes ticket #6672. Please comment, Carl Eugen From c4d3ba1d69e7dd7a60ea7150eb1fc545a86697fe Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Mon, 18 Sep 2017 23:10:06 +0200 Subject: [PATCH] lavfi/stereo3d: Set SAR for every output frame. Fixes ticket #6672. --- libavfi

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/wmv2dec: Check end of bitstream in parse_mb_skip() and ff_wmv2_decode_mb()

2017-09-18 Thread Carl Eugen Hoyos
2017-09-18 15:11 GMT+02:00 Ronald S. Bultje : > Hi Michael, > > On Sun, Sep 17, 2017 at 8:15 PM, Michael Niedermayer > wrote: > >> Iam happy to follow what the community prefers. >> > > Some don't like it, some don't care. I think everyone would be happy (and > thus the sum of happiness would incre

Re: [FFmpeg-devel] [PATCH 2/2] MAINTAINERS: modify the hlsenc description

2017-09-18 Thread Lou Logan
On Sun, Sep 17, 2017, at 06:08 PM, Steven Liu wrote: > change the hlsenc from hls encryption to hlsenc > > Suggested-by: Aman Gupta > Signed-off-by: Steven Liu > --- > MAINTAINERS | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/MAINTAINERS b/MAINTAINERS > index 0b0f7fa

Re: [FFmpeg-devel] [PATCH] doc/faq: replace "Mo" with Bytes

2017-09-18 Thread Werner Robitza
On Mon, Sep 18, 2017 at 5:06 PM, Nicolas George wrote: > > Le jour du Génie, an CCXXV, Werner Robitza a écrit : > > Replaces French "Mo" with "Bytes". > > > > Signed-off-by: Werner Robitza > > --- > > doc/faq.texi | 5 +++-- > > 1 file changed, 3 insertions(+), 2 deletions(-) > > No. "Octet" ori

Re: [FFmpeg-devel] [PATCH] doc/faq: replace "Mo" with Bytes

2017-09-18 Thread Nicolas George
Le jour du Génie, an CCXXV, Werner Robitza a écrit : > Hum, okay. Didn't think this was a conscious decision. I frequently > see speakers of French using "Mo" without knowing that (most of) the > rest of the world barely has an idea what this is. And I do understand > that octets are used in networ

Re: [FFmpeg-devel] [PATCH]configure: Add -Wno-main

2017-09-18 Thread Nicolas George
Le jour du Génie, an CCXXV, Clement Boesch a écrit : > I'm with James on this one, it's easy and harmless to fix, so I think > we should do that instead. I am also for changing the variable names. But it should also be reported to gcc, because this warning is utterly braindead for local symbols.

Re: [FFmpeg-devel] [PATCH] doc/faq: replace "Mo" with Bytes

2017-09-18 Thread Nicolas George
Le jour du Génie, an CCXXV, Werner Robitza a écrit : > Replaces French "Mo" with "Bytes". > > Signed-off-by: Werner Robitza > --- > doc/faq.texi | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) No. "Octet" originated from French but has been imported to English because "byte" causes

Re: [FFmpeg-devel] [PATCH]configure: Add -Wno-main

2017-09-18 Thread Carl Eugen Hoyos
2017-09-18 7:54 GMT+02:00 Clément Bœsch : > On Mon, Sep 18, 2017 at 03:55:12AM +0200, Carl Eugen Hoyos wrote: >> 2017-09-18 3:47 GMT+02:00 James Almer : >> > On 9/17/2017 10:37 PM, Carl Eugen Hoyos wrote: >> >> Hi! >> >> >> >> Attached patch fixes several warnings when compiling libavfilter with >>

Re: [FFmpeg-devel] [PATCH] x86/exrdsp: optimize ff_reorder_pixels_avx2()

2017-09-18 Thread James Almer
On 9/18/2017 4:35 AM, Martin Vignali wrote: > 2017-09-18 3:52 GMT+02:00 James Almer : > >> From: Henrik Gramner >> >> Tested with "checkasm --test=exrdsp -bench" >> >> Before: >> reorder_pixels_c: 5187.8 >> reorder_pixels_sse2: 377.0 >> reorder_pixels_avx2: 331.3 >> >> After: >> reorder_pixels_c:

Re: [FFmpeg-devel] [PATCH] avfilter: add vmafmotion filter

2017-09-18 Thread Ashish Pratap Singh
Hi, On Mon, Sep 18, 2017 at 6:46 AM, James Almer wrote: > On 9/15/2017 5:47 PM, Ashish Pratap Singh wrote: > > From: Ashish Singh > > > > Hi, this patch addresses the previous issues and changes it to a single > > input filter. > > > > Signed-off-by: Ashish Singh > > --- > > Changelog

Re: [FFmpeg-devel] [PATCH] doc/faq: replace "Mo" with Bytes

2017-09-18 Thread Werner Robitza
On Mon, Sep 18, 2017 at 3:45 PM, Ricardo Constantino wrote: > Why not MB instead? It's still more readable than Bytes/B. Because -probesize takes the number of Bytes by default. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mai

Re: [FFmpeg-devel] [PATCH] doc/faq: replace "Mo" with Bytes

2017-09-18 Thread Ricardo Constantino
On 18 September 2017 at 14:27, Werner Robitza wrote: > Replaces French "Mo" with "Bytes". > > Signed-off-by: Werner Robitza > --- > doc/faq.texi | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/doc/faq.texi b/doc/faq.texi > index ff35c89..07af0d9 100644 > --- a/doc/

[FFmpeg-devel] [PATCH] doc/faq: replace "Mo" with Bytes

2017-09-18 Thread Werner Robitza
Replaces French "Mo" with "Bytes". Signed-off-by: Werner Robitza --- doc/faq.texi | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/doc/faq.texi b/doc/faq.texi index ff35c89..07af0d9 100644 --- a/doc/faq.texi +++ b/doc/faq.texi @@ -450,8 +450,9 @@ work with streams that we

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/wmv2dec: Check end of bitstream in parse_mb_skip() and ff_wmv2_decode_mb()

2017-09-18 Thread Ronald S. Bultje
Hi Michael, On Sun, Sep 17, 2017 at 8:15 PM, Michael Niedermayer wrote: > Iam happy to follow what the community prefers. > Some don't like it, some don't care. I think everyone would be happy (and thus the sum of happiness would increase) if you changed this to ff_dlog() or something along tho

Re: [FFmpeg-devel] [PATCH] avcodec/mips: Fixed rnd_val variable to 6 in hevc uni mc msa functions

2017-09-18 Thread Manojkumar Bhosale
LGTM -Original Message- From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of kaustubh.ra...@imgtec.com Sent: Monday, September 18, 2017 1:49 PM To: ffmpeg-devel@ffmpeg.org Cc: Kaustubh Raste Subject: [FFmpeg-devel] [PATCH] avcodec/mips: Fixed rnd_val variable to 6 in

Re: [FFmpeg-devel] [PATCH] avcodec/mips: preload data in hevc sao edge 0 degree filter msa functions

2017-09-18 Thread Manojkumar Bhosale
LGTM -Original Message- From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of kaustubh.ra...@imgtec.com Sent: Monday, September 18, 2017 1:55 PM To: ffmpeg-devel@ffmpeg.org Cc: Kaustubh Raste Subject: [FFmpeg-devel] [PATCH] avcodec/mips: preload data in hevc sao edge 0

Re: [FFmpeg-devel] [PATCH] configure: support static libnpp [v3]

2017-09-18 Thread Timo Rothenpieler
I don't like the complexity of this. It also seems strange to me that you put cudart_static and other culibos in there, as well as $ldl, stdc++ and so on. That would most likely link against shared cuda and static cuda at the same time, which seems highly redundant. Also, no other library use

[FFmpeg-devel] [PATCH] avcodec/mips: Reduced conditional cases in avc inter lpf msa functions

2017-09-18 Thread kaustubh.raste
From: Kaustubh Raste Signed-off-by: Kaustubh Raste --- libavcodec/mips/h264dsp_msa.c | 274 + 1 file changed, 110 insertions(+), 164 deletions(-) diff --git a/libavcodec/mips/h264dsp_msa.c b/libavcodec/mips/h264dsp_msa.c index a17eacb..422703d 100644 --

[FFmpeg-devel] [PATCH] avcodec/mips: Unrolled loops and expanded functions in avc put mc 10 & 30 msa functions

2017-09-18 Thread kaustubh.raste
From: Kaustubh Raste Signed-off-by: Kaustubh Raste --- libavcodec/mips/h264qpel_msa.c | 284 +++- 1 file changed, 278 insertions(+), 6 deletions(-) diff --git a/libavcodec/mips/h264qpel_msa.c b/libavcodec/mips/h264qpel_msa.c index 05dffea..b7f6c3d 100644 --

[FFmpeg-devel] [PATCH] avcodec/mips: preload data in hevc sao edge 0 degree filter msa functions

2017-09-18 Thread kaustubh.raste
From: Kaustubh Raste Signed-off-by: Kaustubh Raste --- libavcodec/mips/hevc_lpf_sao_msa.c | 232 +--- 1 file changed, 138 insertions(+), 94 deletions(-) diff --git a/libavcodec/mips/hevc_lpf_sao_msa.c b/libavcodec/mips/hevc_lpf_sao_msa.c index 1d77432..3472d32

[FFmpeg-devel] [PATCH] avcodec/mips: Fixed rnd_val variable to 6 in hevc uni mc msa functions

2017-09-18 Thread kaustubh.raste
From: Kaustubh Raste Signed-off-by: Kaustubh Raste --- libavcodec/mips/hevc_mc_uni_msa.c | 372 + 1 file changed, 133 insertions(+), 239 deletions(-) diff --git a/libavcodec/mips/hevc_mc_uni_msa.c b/libavcodec/mips/hevc_mc_uni_msa.c index 754fbdb..cf22e7f

Re: [FFmpeg-devel] [PATCH] x86/exrdsp: optimize ff_reorder_pixels_avx2()

2017-09-18 Thread Martin Vignali
2017-09-18 3:52 GMT+02:00 James Almer : > From: Henrik Gramner > > Tested with "checkasm --test=exrdsp -bench" > > Before: > reorder_pixels_c: 5187.8 > reorder_pixels_sse2: 377.0 > reorder_pixels_avx2: 331.3 > > After: > reorder_pixels_c: 5181.5 > reorder_pixels_sse2: 377.0 > reorder_pixels_avx2:

Re: [FFmpeg-devel] [PATCH 2/2] fate: add mxf_dv25/dvcpro50 regression tests

2017-09-18 Thread Tobias Rapp
On 15.09.2017 22:43, Michael Niedermayer wrote: On Thu, Sep 14, 2017 at 03:44:42PM +0200, Tobias Rapp wrote: Signed-off-by: Tobias Rapp --- tests/fate/avformat.mak | 2 ++ tests/fate/seek.mak | 4 +++ tests/lavf-regression.sh | 8 ++ tests/ref/lavf/mxf_

Re: [FFmpeg-devel] [PATCH 1/2] avformat/mxfenc: fix aspect ratio when writing 16:9 DV frames

2017-09-18 Thread Tobias Rapp
On 15.09.2017 20:44, Michael Niedermayer wrote: On Thu, Sep 14, 2017 at 03:44:41PM +0200, Tobias Rapp wrote: Signed-off-by: Tobias Rapp --- libavformat/mxfenc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/mxfenc.c b/libavformat/mxfenc.c index 7289e0b..da4d