Re: [FFmpeg-devel] [PATCH 2/2] avcodec/gifdec: don't set AVFrame.pkt_dts

2017-05-01 Thread Paul B Mahol
On 5/2/17, James Almer wrote: > decode.c already sets it to the default value. > > Signed-off-by: James Almer > --- > libavcodec/gifdec.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/libavcodec/gifdec.c b/libavcodec/gifdec.c > index 2eeed4c4c7..3d90210737 100644 > --- a/libavcodec/gif

[FFmpeg-devel] [PATCH] avformat/nut: add DSDs support

2017-05-01 Thread Takayuki 'January June' Suwa
codec tags for DSDs are similar to PCMs: 'D', where 'D' means DSD, or pulse-Density-modulation, is 'L' for LSB-first, 'M' for MSB-first is 'D' for default, 'P' for planar is always binary one :) --- libavformat/nut.c | 4 1 file changed, 4 insertions(+) diff --git a/libavformat/nut.c

[FFmpeg-devel] [PATCH] lavc/vaapi_encode_h264: add option to insert AUD nal to AVC stream

2017-05-01 Thread Jun Zhao
From e39078e9fe02c8d77c5e28436aef4d80a2d7b3a0 Mon Sep 17 00:00:00 2001 From: Jun Zhao Date: Tue, 2 May 2017 10:36:55 +0800 Subject: [PATCH] lavc/vaapi_encode_h264: add option to insert AUD nal to AVC stream. Add aud option to support insert AUD nal in AVC stream. Default is disable. Signed-off-

Re: [FFmpeg-devel] [PATCH] ffmpeg: count packets when queued

2017-05-01 Thread James Almer
On 5/1/2017 9:51 PM, Muhammad Faiz wrote: > Because write_packet() fakely writes packets to muxer by queueing > them when muxer hasn't been initialized, it should also increment > frame_number fakely. > This is required because code in do_streamcopy() rely on > frame_number. > > Should fix Ticket6

[FFmpeg-devel] [PATCH 2/2] lavf/utils: bail early if we don't see any packets in an MPEGTS stream

2017-05-01 Thread Rodger Combs
--- libavformat/utils.c | 18 +- 1 file changed, 17 insertions(+), 1 deletion(-) diff --git a/libavformat/utils.c b/libavformat/utils.c index ba82a766dc..4028d8dbcb 100644 --- a/libavformat/utils.c +++ b/libavformat/utils.c @@ -3505,6 +3505,8 @@ int avformat_find_stream_info(AVFor

[FFmpeg-devel] [PATCH 1/2] ffmpeg: treat audio streams with no parsed packets like unknown streams

2017-05-01 Thread Rodger Combs
--- ffmpeg_opt.c | 8 1 file changed, 8 insertions(+) diff --git a/ffmpeg_opt.c b/ffmpeg_opt.c index d1fe8742ff..5ed29d717e 100644 --- a/ffmpeg_opt.c +++ b/ffmpeg_opt.c @@ -2255,6 +2255,14 @@ loop_end: if(o->data_disable && ist->st->codecpar->codec_type == AVMEDIA_T

Re: [FFmpeg-devel] [PATCH] libavformat/http: Ignore expired cookies

2017-05-01 Thread Michael Niedermayer
On Mon, May 01, 2017 at 09:11:11PM -0400, Micah Galizia wrote: > On 2017-05-01 12:09 PM, Michael Niedermayer wrote: > >On Sun, Apr 30, 2017 at 02:25:29PM -0400, Micah Galizia wrote: > >>Signed-off-by: Micah Galizia > >>--- > >> libavformat/http.c | 213 > >> ++

[FFmpeg-devel] [PATCH] ffmpeg: count packets when queued

2017-05-01 Thread Muhammad Faiz
Because write_packet() fakely writes packets to muxer by queueing them when muxer hasn't been initialized, it should also increment frame_number fakely. This is required because code in do_streamcopy() rely on frame_number. Should fix Ticket6227 Signed-off-by: Muhammad Faiz --- ffmpeg.c | 39 ++

Re: [FFmpeg-devel] [PATCH] libavformat/http: Ignore expired cookies

2017-05-01 Thread Micah Galizia
On 2017-05-01 12:09 PM, Michael Niedermayer wrote: On Sun, Apr 30, 2017 at 02:25:29PM -0400, Micah Galizia wrote: Signed-off-by: Micah Galizia --- libavformat/http.c | 213 +++-- 1 file changed, 156 insertions(+), 57 deletions(-) applied thx

Re: [FFmpeg-devel] [PATCH] lavc/vaapi_encode_h265: Suppress the "above array bounds" warning.

2017-05-01 Thread Jun Zhao
On 2017/4/30 20:47, Mark Thompson wrote: > On 26/04/17 07:44, Jun Zhao wrote: >> From f3678e0ceb691b6df5957a2c3d26d4f0d91d4ff5 Mon Sep 17 00:00:00 2001 >> From: Jun Zhao >> Date: Wed, 26 Apr 2017 14:00:56 +0800 >> Subject: [PATCH] lavc/vaapi_encode_h265: Suppress the "above array bounds" >> war

[FFmpeg-devel] [PATCH 1/2] avcodec/vp9: don't set AVFrame.pkt_dts

2017-05-01 Thread James Almer
decode.c already sets it to the default value. Signed-off-by: James Almer --- libavcodec/vp9.c | 1 - 1 file changed, 1 deletion(-) diff --git a/libavcodec/vp9.c b/libavcodec/vp9.c index 4d7310f6d4..3147ffa0db 100644 --- a/libavcodec/vp9.c +++ b/libavcodec/vp9.c @@ -1125,7 +1125,6 @@ FF_DISABLE

[FFmpeg-devel] [PATCH 2/2] avcodec/gifdec: don't set AVFrame.pkt_dts

2017-05-01 Thread James Almer
decode.c already sets it to the default value. Signed-off-by: James Almer --- libavcodec/gifdec.c | 1 - 1 file changed, 1 deletion(-) diff --git a/libavcodec/gifdec.c b/libavcodec/gifdec.c index 2eeed4c4c7..3d90210737 100644 --- a/libavcodec/gifdec.c +++ b/libavcodec/gifdec.c @@ -467,7 +467,6

[FFmpeg-devel] [PATCH] avcodec/decode: also check for FF_CODEC_CAP_SETS_PKT_DTS in audio decoders

2017-05-01 Thread James Almer
Signed-off-by: James Almer --- libavcodec/decode.c | 6 ++ 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/libavcodec/decode.c b/libavcodec/decode.c index edfae5583c..a54653f5a0 100644 --- a/libavcodec/decode.c +++ b/libavcodec/decode.c @@ -413,9 +413,9 @@ FF_ENABLE_DEPRECATION_

[FFmpeg-devel] [PATCH] avcodec/flicvideo: Check for chunk overread

2017-05-01 Thread Michael Niedermayer
Fixes integer overflow Fixes: 1292/clusterfuzz-testcase-minimized-5795512143839232 Signed-off-by: Michael Niedermayer --- libavcodec/flicvideo.c | 20 +++- 1 file changed, 19 insertions(+), 1 deletion(-) diff --git a/libavcodec/flicvideo.c b/libavcodec/flicvideo.c index b1b7b5a4

[FFmpeg-devel] [PATCH 9/9] hevc_parser: move slice header parsing to its own function

2017-05-01 Thread James Almer
Signed-off-by: James Almer --- libavcodec/hevc_parser.c | 227 +-- 1 file changed, 120 insertions(+), 107 deletions(-) diff --git a/libavcodec/hevc_parser.c b/libavcodec/hevc_parser.c index 57f2b2bb3e..4f41b78f66 100644 --- a/libavcodec/hevc_parser.c +

[FFmpeg-devel] [PATCH 8/9] hevc_parse: decode SEI message NALUs in extradata

2017-05-01 Thread James Almer
They may be present in hvcc style extradata. Based on a patch by Hendrik Leppkes. Signed-off-by: James Almer --- libavcodec/hevc_parse.c| 21 ++--- libavcodec/hevc_parse.h| 4 ++-- libavcodec/hevcdec.c | 2 +- libavcodec/mediacodecdec.c | 4 +++- 4 files changed

[FFmpeg-devel] [PATCH v2 6/9] hevc_parser: remove HEVCContext usage

2017-05-01 Thread James Almer
This gets rid of the duplicate, limited parser. Signed-off-by: James Almer --- Cosmetic changes only, needed to apply patch 9/9 on top of this. libavcodec/hevc_parser.c | 177 +-- 1 file changed, 32 insertions(+), 145 deletions(-) diff --git a/libavc

Re: [FFmpeg-devel] [PATCH]lavc/mips: Add missing const qualifiers

2017-05-01 Thread Carl Eugen Hoyos
2017-05-01 15:10 GMT+02:00 Michael Niedermayer : > On Mon, May 01, 2017 at 10:34:23AM +0200, Carl Eugen Hoyos wrote: >> Hi! >> >> Attached patch fixes many warnings when compiling for mips. >> >> Please comment, Carl Eugen > > LGTM, untested though Patch applied (tested with Android NDK). Thank y

Re: [FFmpeg-devel] [PATCH]lavc/mips/iirfilter_mips: Include config.h

2017-05-01 Thread Carl Eugen Hoyos
2017-05-01 16:41 GMT+02:00 Michael Niedermayer : > On Mon, May 01, 2017 at 10:37:51AM +0200, Carl Eugen Hoyos wrote: >> Hi! >> >> Attached patch fixes a warning when compiling for mips, I suspect it also >> fixes usage of inline asm. >> >> Please comment, Carl Eugen > >> iirfilter_mips.c |1 +

Re: [FFmpeg-devel] [RFC] Reviewing merges

2017-05-01 Thread James Almer
On 5/1/2017 3:13 PM, Marton Balint wrote: > > On Mon, 24 Apr 2017, wm4 wrote: > >> On Mon, 24 Apr 2017 21:14:15 +0200 (CEST) >> Marton Balint wrote: >> >>> On Mon, 24 Apr 2017, Michael Niedermayer wrote: >>> > On Mon, Apr 24, 2017 at 11:23:16AM -0300, James Almer wrote: >>> >> We have recently b

Re: [FFmpeg-devel] [PATCH] avcodec/libmp3lame: properly handle unaligned frame data

2017-05-01 Thread Paul B Mahol
On 5/1/17, Muhammad Faiz wrote: > On Tue, May 2, 2017 at 1:05 AM, Paul B Mahol wrote: >> On 5/1/17, Muhammad Faiz wrote: >>> On Tue, May 2, 2017 at 12:45 AM, Paul B Mahol wrote: On 5/1/17, Muhammad Faiz wrote: > On Mon, May 1, 2017 at 11:22 PM, Kyle Swanson wrote: >> Hi, >> >

Re: [FFmpeg-devel] [PATCH] avcodec/libmp3lame: properly handle unaligned frame data

2017-05-01 Thread Muhammad Faiz
On Tue, May 2, 2017 at 1:05 AM, Paul B Mahol wrote: > On 5/1/17, Muhammad Faiz wrote: >> On Tue, May 2, 2017 at 12:45 AM, Paul B Mahol wrote: >>> On 5/1/17, Muhammad Faiz wrote: On Mon, May 1, 2017 at 11:22 PM, Kyle Swanson wrote: > Hi, > > On Mon, May 1, 2017 at 3:18 AM, Paul

Re: [FFmpeg-devel] [RFC] Reviewing merges

2017-05-01 Thread Marton Balint
On Mon, 24 Apr 2017, wm4 wrote: On Mon, 24 Apr 2017 21:14:15 +0200 (CEST) Marton Balint wrote: On Mon, 24 Apr 2017, Michael Niedermayer wrote: > On Mon, Apr 24, 2017 at 11:23:16AM -0300, James Almer wrote: >> We have recently been able to go through six hundred or so commits in a >> month

Re: [FFmpeg-devel] [PATCH] avcodec/libmp3lame: properly handle unaligned frame data

2017-05-01 Thread Paul B Mahol
On 5/1/17, Muhammad Faiz wrote: > On Tue, May 2, 2017 at 12:45 AM, Paul B Mahol wrote: >> On 5/1/17, Muhammad Faiz wrote: >>> On Mon, May 1, 2017 at 11:22 PM, Kyle Swanson wrote: Hi, On Mon, May 1, 2017 at 3:18 AM, Paul B Mahol wrote: > On 4/30/17, Nicolas George wrote: >>>

Re: [FFmpeg-devel] [PATCH] avcodec/libmp3lame: properly handle unaligned frame data

2017-05-01 Thread Muhammad Faiz
On Tue, May 2, 2017 at 12:45 AM, Paul B Mahol wrote: > On 5/1/17, Muhammad Faiz wrote: >> On Mon, May 1, 2017 at 11:22 PM, Kyle Swanson wrote: >>> Hi, >>> >>> On Mon, May 1, 2017 at 3:18 AM, Paul B Mahol wrote: On 4/30/17, Nicolas George wrote: > Le primidi 11 floreal, an CCXXV, Muham

Re: [FFmpeg-devel] [PATCH] avcodec/libmp3lame: properly handle unaligned frame data

2017-05-01 Thread Paul B Mahol
On 5/1/17, Muhammad Faiz wrote: > On Mon, May 1, 2017 at 11:22 PM, Kyle Swanson wrote: >> Hi, >> >> On Mon, May 1, 2017 at 3:18 AM, Paul B Mahol wrote: >>> On 4/30/17, Nicolas George wrote: Le primidi 11 floreal, an CCXXV, Muhammad Faiz a ecrit : > Are you working on these? Because cur

Re: [FFmpeg-devel] [PATCH] avcodec/libmp3lame: properly handle unaligned frame data

2017-05-01 Thread Muhammad Faiz
On Mon, May 1, 2017 at 11:22 PM, Kyle Swanson wrote: > Hi, > > On Mon, May 1, 2017 at 3:18 AM, Paul B Mahol wrote: >> On 4/30/17, Nicolas George wrote: >>> Le primidi 11 floreal, an CCXXV, Muhammad Faiz a ecrit : Are you working on these? Because currently I'm not. >>> >>> There is nothing

Re: [FFmpeg-devel] [PATCH] avcodec/libmp3lame: properly handle unaligned frame data

2017-05-01 Thread Ronald S. Bultje
Hi, On Mon, May 1, 2017 at 12:44 PM, Ronald S. Bultje wrote: > Hi, > > On Sun, Apr 30, 2017 at 1:16 PM, wm4 wrote: > >> I think it's probably ok to push your current patch, since the original >> author of the patch who broke this probably won't fix it. > > > I don't like the patch because it es

Re: [FFmpeg-devel] [PATCH] avcodec/libmp3lame: properly handle unaligned frame data

2017-05-01 Thread Ronald S. Bultje
Hi, On Sun, Apr 30, 2017 at 1:16 PM, wm4 wrote: > I think it's probably ok to push your current patch, since the original > author of the patch who broke this probably won't fix it. I don't like the patch because it essentially duplicates the C implementation of the DSP function. If we do that

Re: [FFmpeg-devel] [PATCH] avcodec/libmp3lame: properly handle unaligned frame data

2017-05-01 Thread Kyle Swanson
Hi, On Mon, May 1, 2017 at 3:18 AM, Paul B Mahol wrote: > On 4/30/17, Nicolas George wrote: >> Le primidi 11 floreal, an CCXXV, Muhammad Faiz a ecrit : >>> Are you working on these? Because currently I'm not. >> >> There is nothing to work on yet: the message you answer to is raising a >> questi

Re: [FFmpeg-devel] [PATCH] libavformat/http: Ignore expired cookies

2017-05-01 Thread Michael Niedermayer
On Sun, Apr 30, 2017 at 02:25:29PM -0400, Micah Galizia wrote: > Signed-off-by: Micah Galizia > --- > libavformat/http.c | 213 > +++-- > 1 file changed, 156 insertions(+), 57 deletions(-) applied thx [...] -- Michael GnuPG fingerprint: 9FF

Re: [FFmpeg-devel] [PATCH] doc/developer: Add terse documentation of assumed C implementation defined behavior

2017-05-01 Thread Michael Niedermayer
On Fri, Apr 28, 2017 at 12:52:45PM +0200, wm4 wrote: > On Fri, 28 Apr 2017 02:50:42 +0200 > Michael Niedermayer wrote: > > > Suggested-by: "Ronald S. Bultje" > > Signed-off-by: Michael Niedermayer > > --- > > doc/developer.texi | 5 + > > 1 file changed, 5 insertions(+) > > > > diff --git

Re: [FFmpeg-devel] [PATCH]lavc/mips/iirfilter_mips: Include config.h

2017-05-01 Thread Michael Niedermayer
On Mon, May 01, 2017 at 10:37:51AM +0200, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes a warning when compiling for mips, I suspect it also > fixes usage of inline asm. > > Please comment, Carl Eugen > iirfilter_mips.c |1 + > 1 file changed, 1 insertion(+) > 2c9416e78c7b93db40c7

Re: [FFmpeg-devel] [PATCH v2] avcodec/decode: do not treat discarded frames as eof when draining

2017-05-01 Thread wm4
On Mon, 1 May 2017 07:36:35 +0700 Muhammad Faiz wrote: > Fix fate failures: > make fate-mov THREADS=32 > > Signed-off-by: Muhammad Faiz > --- > libavcodec/decode.c | 8 +--- > 1 file changed, 5 insertions(+), 3 deletions(-) > > diff --git a/libavcodec/decode.c b/libavcodec/decode.c >

Re: [FFmpeg-devel] [PATCH]lavc/mips: Add missing const qualifiers

2017-05-01 Thread Michael Niedermayer
On Mon, May 01, 2017 at 10:34:23AM +0200, Carl Eugen Hoyos wrote: > Hi! > > Attached patch fixes many warnings when compiling for mips. > > Please comment, Carl Eugen LGTM, untested though thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB I have never wish

[FFmpeg-devel] fate/exr : add test for Y, B44A negative, datawindow != display window

2017-05-01 Thread Martin Vignali
Hello, In attach a patch to add fate tests for exr samples can be found here https://we.tl/ItuIX0BMfk and need to be put inside fate-suite/exr can be test with make fate-exr SAMPLES=fate-suite/ Theses tests increase coverage of exr.c : - rgb_b44a_half_negative_4x4 : test negative half value i

Re: [FFmpeg-devel] libavcodec/exr : fix piz uncompress on big endian

2017-05-01 Thread Michael Niedermayer
On Sun, Apr 30, 2017 at 05:11:00PM +0200, Martin Vignali wrote: > > > > Tested-by: michael on qemu mips > > > > Ping for apply this patch (and the two fate tests) applied thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB When you are offended at any man's fau

[FFmpeg-devel] [PATCH]compat/strtod: Add missing const qualifiers

2017-05-01 Thread Carl Eugen Hoyos
Hi! Even without the casts, the patch reduces the number of warnings shown when compiling compat/strtod from seven to three. Please comment, Carl Eugen From f376877bfabb6fba8f83acab7bd7fb76388d88fd Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Mon, 1 May 2017 10:49:31 +0200 Subject: [PA

[FFmpeg-devel] [PATCH]lavc/mips/iirfilter_mips: Include config.h

2017-05-01 Thread Carl Eugen Hoyos
Hi! Attached patch fixes a warning when compiling for mips, I suspect it also fixes usage of inline asm. Please comment, Carl Eugen From bbb8a20926b679d05af3b668a2822e6e1a97efa7 Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Mon, 1 May 2017 10:35:28 +0200 Subject: [PATCH] lavc/mips/iirfi

[FFmpeg-devel] [PATCH]lavc/mips: Add missing const qualifiers

2017-05-01 Thread Carl Eugen Hoyos
Hi! Attached patch fixes many warnings when compiling for mips. Please comment, Carl Eugen From a7d2e32806adef148496454b23b5f8fe12cbf396 Mon Sep 17 00:00:00 2001 From: Carl Eugen Hoyos Date: Mon, 1 May 2017 10:31:35 +0200 Subject: [PATCH] lavc/mips/hevc_idct_msa: Add missing const qualifier. Fi

Re: [FFmpeg-devel] [PATCH] avfilter: add arbitrary audio FIR filter

2017-05-01 Thread Paul B Mahol
On 5/1/17, Muhammad Faiz wrote: > On Mon, May 1, 2017 at 5:02 AM, Paul B Mahol wrote: [...] >> + >> +for (ch = 0; ch < s->nb_channels; ch++) { >> +dst = (float *)out->extended_data[ch]; >> +buf = (float *)s->buffer->extended_data[ch]; >> + >> +for (n

[FFmpeg-devel] [PATCH] avfilter: add arbitrary audio FIR filter

2017-05-01 Thread Paul B Mahol
Signed-off-by: Paul B Mahol --- configure | 2 + doc/filters.texi| 10 ++ libavfilter/Makefile| 1 + libavfilter/af_afirfilter.c | 409 libavfilter/allfilters.c| 1 + 5 files changed, 423 insertions(+)

Re: [FFmpeg-devel] [PATCH] avcodec/libmp3lame: properly handle unaligned frame data

2017-05-01 Thread Paul B Mahol
On 4/30/17, Nicolas George wrote: > Le primidi 11 floreal, an CCXXV, Muhammad Faiz a ecrit : >> Are you working on these? Because currently I'm not. > > There is nothing to work on yet: the message you answer to is raising a > question about the global design of the internal API. That question > n