> -Original Message-
> From: ffmpeg-devel On Behalf Of
> Lynne
> Sent: Wednesday, May 13, 2020 7:19 AM
> To: FFmpeg development discussions and patches de...@ffmpeg.org>
> Subject: Re: [FFmpeg-devel] [PATCH v2 1/3] lavu/pix_fmt: add new pixel
> format x2rgb10
>
> May 12, 2020, 23:42 by
Marton Balint 于2020年5月15日周五 上午2:33写道：
>
>
>
> On Thu, 14 May 2020, Tao Zhang wrote:
>
> > Hi,
> > FFmpeg supports multiple outputs created out of the same input in the
> > same process like
> > ffmpeg -i input -filter_complex '[0:v]yadif,split=3[out1][out2][out3]' \
> >-map '[out1]' -s 128
From: Limin Wang
This bug was introduced in commit: 339593ca90cb3e05d659ec99a1479904ec742294 and
339593ca90cb3e05d659ec99a1479904ec742294.
Reported-by: Marton Balint
Signed-off-by: Limin Wang
---
libavfilter/vf_framerate.c| 2 +-
libavfilter/vf_minterpolate.c | 2 +-
2 files changed, 2 i
On Thu, May 14, 2020 at 11:03:49PM +0200, Marton Balint wrote:
> This reverts commit 339593ca90cb3e05d659ec99a1479904ec742294.
>
> Fixes null pointer dereference.
>
> Signed-off-by: Marton Balint
> ---
> libavfilter/vf_framerate.c | 15 ---
> 1 file changed, 4 insertions(+), 11 del
On Thu, May 14, 2020 at 09:00:21PM +0200, Marton Balint wrote:
>
>
> On Thu, 14 May 2020, Marton Balint wrote:
>
> >
> >
> > On Thu, 14 May 2020, Nicolas George wrote:
> >
> > > Marton Balint (12020-05-14):
> > > > I am not a huge fan of this patch, mafd refers to a score
> > > > between this
On Thu, May 14, 2020 at 08:53:58PM +0200, Marton Balint wrote:
>
>
> On Thu, 14 May 2020, Nicolas George wrote:
>
> > Marton Balint (12020-05-14):
> > > I am not a huge fan of this patch, mafd refers to a score between this
> > > frame
> > > and the previous frame, we cannot ensure that there w
On Thu, May 14, 2020 at 11:51:29PM +0200, Marton Balint wrote:
>
>
> On Thu, 14 May 2020, Carl Eugen Hoyos wrote:
>
> > Am Do., 14. Mai 2020 um 23:04 Uhr schrieb Marton Balint :
> > >
> > > This reverts commit 339593ca90cb3e05d659ec99a1479904ec742294.
> > >
> > > Fixes null pointer dereference
This tests the hscale 8bpp to 14bpp functions with different filter
sizes.
Signed-off-by: Josh de Kock
---
Adds support for checking corner cases for signed and large
coefficients. Also makes the padded coefficients random data.
Tested on x86_64 and aarch64.
tests/checkasm/Makefile | 2
applied, thanks
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
Fixes one issue in the function itself and one in the dependent
function av_add_stable() which wasn't checking for NaN.
Signed-off-by: Dale Curtis
---
libavformat/utils.c | 2 +-
libavutil/mathematics.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
From fbe9f4552d7153286cfa50d3f03
Signed-off-by: Dale Curtis
---
libavformat/mov.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
From e4b963f890ae37e2a06276a3067daab75013c8f9 Mon Sep 17 00:00:00 2001
From: Dale Curtis
Date: Thu, 14 May 2020 14:38:07 -0700
Subject: [PATCH 3/5] [mov] Check if DTS is AV_NOPTS_VALUE in
mov_
Signed-off-by: Dale Curtis
---
libavformat/mov.c | 4
1 file changed, 4 insertions(+)
From 3cd1ecb83c4b100bef99d9cd23d0066f0ad3cc5c Mon Sep 17 00:00:00 2001
From: Dale Curtis
Date: Thu, 14 May 2020 14:57:08 -0700
Subject: [PATCH 5/5] [mov] Don't allow negative sample sizes.
Signed-off-by:
Signed-off-by: Dale Curtis
---
libavformat/oggparsetheora.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
From 6ae2834612ddc47e4ce40c87a9cc7e76e402bbdc Mon Sep 17 00:00:00 2001
From: Dale Curtis
Date: Thu, 14 May 2020 14:33:55 -0700
Subject: [PATCH 2/5] [oggparsetheora] Use av_sat_sub64(
Avoids overflow from fuzzed skip_samples values.
Signed-off-by: Dale Curtis
---
libavformat/utils.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
From 7482aaef44fa4c6c43efd16b2ed8eb474b1283b0 Mon Sep 17 00:00:00 2001
From: Dale Curtis
Date: Thu, 14 May 2020 14:29:15 -0700
Subject: [
On 5/12/2020 10:47 PM, myp...@gmail.com wrote:
> On Tue, May 12, 2020 at 10:09 PM James Almer wrote:
>>
>> It's not meant for audio or subtitles, or for encoders of any kind.
>>
>> Signed-off-by: James Almer
>> ---
>> libavcodec/options_table.h | 2 +-
>> 1 file changed, 1 insertion(+), 1 deleti
On Thu, 14 May 2020, Carl Eugen Hoyos wrote:
Am Do., 14. Mai 2020 um 23:04 Uhr schrieb Marton Balint :
This reverts commit 339593ca90cb3e05d659ec99a1479904ec742294.
Fixes null pointer dereference.
I did not look into the patches, and maybe you could add how the
null pointer dereferences a
Am Do., 14. Mai 2020 um 23:04 Uhr schrieb Marton Balint :
>
> This reverts commit 339593ca90cb3e05d659ec99a1479904ec742294.
>
> Fixes null pointer dereference.
I did not look into the patches, and maybe you could add how the
null pointer dereferences are reproducible but if these patches fix
crash
On Thu, 14 May 2020, Josh de Kock wrote:
From: Martin Storsjö
This was ported from dav1d (c950e7101bdf5f7117bfca816984a21e550509f0).
Signed-off-by: Josh de Kock
---
tests/checkasm/checkasm.c | 42 +++
tests/checkasm/checkasm.h | 16 +++
2 files c
This reverts commit 339593ca90cb3e05d659ec99a1479904ec742294.
Fixes null pointer dereference.
Signed-off-by: Marton Balint
---
libavfilter/vf_framerate.c | 15 ---
1 file changed, 4 insertions(+), 11 deletions(-)
diff --git a/libavfilter/vf_framerate.c b/libavfilter/vf_framerate.c
This reverts commit d88e1c9838dbcfe29d7835f2705ffc9ee6a36bf3.
Fixes null pointer dereference.
Signed-off-by: Marton Balint
---
libavfilter/vf_minterpolate.c | 16
1 file changed, 4 insertions(+), 12 deletions(-)
diff --git a/libavfilter/vf_minterpolate.c b/libavfilter/vf_minte
On Thu, May 14, 2020 at 11:47 AM Michael Niedermayer
wrote:
> On Fri, May 01, 2020 at 11:42:43AM -0700, Dale Curtis wrote:
> > On Fri, May 1, 2020 at 10:57 AM Carl Eugen Hoyos
> wrote:
> >
> > > Could you confirm that you attached the wrong patch?
> > >
> >
> > No, I sent the patches without com
Bump again for this one too. Thanks. The Arch Linux distro is awaiting
resolution of this issue.
- dale
On Mon, May 4, 2020 at 11:10 AM Dale Curtis wrote:
> Any comments on this? Thanks.
>
> - dale
>
> On Thu, Apr 30, 2020 at 12:42 PM Dale Curtis
> wrote:
>
>> Ping for this patch. Thanks
>>
>>
On 14/05/2020 19:15, Zachariah Brown wrote:
> The h264_nvenc and hevc_nvenc encoders aren't respecting the framerate in the
> codec context. Instead it was using the timebase which in our use-case was
> 1/1000 so the encoder was behaving as if we wanted 1000fps. This resulted in
> poor encoding
Do you have actual file this fixes?
On 5/10/20, Jean-Baptiste Kempf wrote:
> This is similar to field 1 (effd2e72)
> ---
> libavcodec/speedhq.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavcodec/speedhq.c b/libavcodec/speedhq.c
> index 890b8253cd..b834b79f28 1006
On 14/05/2020 18:19, Thierry Foucu wrote:
> Looking at
> https://developer.apple.com/library/archive/documentation/QuickTime/QTFF/Metadata/Metadata.html
> The spec does not seem to say 1 or more. But because the `keys` atom is a
> list of indexes used by the `ilst` and the spec for `keys` said:
> I
On Thu, 14 May 2020, Paul B Mahol wrote:
On 5/14/20, Marton Balint wrote:
On Thu, 14 May 2020, Marton Balint wrote:
On Thu, 14 May 2020, Nicolas George wrote:
Marton Balint (12020-05-14):
I am not a huge fan of this patch, mafd refers to a score between this
frame
and the previous
On 5/14/20, Marton Balint wrote:
>
>
> On Thu, 14 May 2020, Marton Balint wrote:
>
>>
>>
>> On Thu, 14 May 2020, Nicolas George wrote:
>>
>>> Marton Balint (12020-05-14):
I am not a huge fan of this patch, mafd refers to a score between this
>> frame
and the previous frame, we cannot ens
On Wed, May 13, 2020 at 10:42:15PM +0200, Michael Niedermayer wrote:
> This causes indexes into scale_conversion_table to wrap around, alternatively
> they
> could be clipped, the table be enlarged or we can error out. I have not found
> a document that specifies
> what is the correct way to hand
Signed-off-by: Paul B Mahol
---
libavcodec/Makefile | 1 +
libavcodec/allcodecs.c | 1 +
libavcodec/codec_desc.c | 7 +
libavcodec/codec_id.h | 1 +
libavcodec/notchlc.c| 461
libavformat/isom.c | 2 +
libavformat/riff.c |
On Thu, 14 May 2020, Marton Balint wrote:
On Thu, 14 May 2020, Nicolas George wrote:
Marton Balint (12020-05-14):
I am not a huge fan of this patch, mafd refers to a score between this
frame
and the previous frame, we cannot ensure that there were no additional
frame
processing betwee
On Thu, 14 May 2020, Nicolas George wrote:
Marton Balint (12020-05-14):
I am not a huge fan of this patch, mafd refers to a score between this frame
and the previous frame, we cannot ensure that there were no additional frame
processing between scdet and this filter which may have duplicated
Marton Balint (12020-05-14):
> I am not a huge fan of this patch, mafd refers to a score between this frame
> and the previous frame, we cannot ensure that there were no additional frame
> processing between scdet and this filter which may have duplicated or
> removed frames. So I'd rather not add
On Fri, May 01, 2020 at 11:42:43AM -0700, Dale Curtis wrote:
> On Fri, May 1, 2020 at 10:57 AM Carl Eugen Hoyos wrote:
>
> > Could you confirm that you attached the wrong patch?
> >
>
> No, I sent the patches without completing the rebase. Sorry.
>
> - dale
> common.h | 36 +
On Thu, 14 May 2020, lance.lmw...@gmail.com wrote:
From: Limin Wang
Reviewed-by: Paul B Mahol
Signed-off-by: Limin Wang
---
libavfilter/vf_minterpolate.c | 16
1 file changed, 12 insertions(+), 4 deletions(-)
diff --git a/libavfilter/vf_minterpolate.c b/libavfilter/vf_mint
On Thu, 14 May 2020, lance.lmw...@gmail.com wrote:
From: Limin Wang
Reviewed-by: Paul B Mahol
Signed-off-by: Limin Wang
---
libavfilter/vf_framerate.c | 15 +++
1 file changed, 11 insertions(+), 4 deletions(-)
diff --git a/libavfilter/vf_framerate.c b/libavfilter/vf_framerate.c
On Thu, 14 May 2020, Tao Zhang wrote:
Hi,
FFmpeg supports multiple outputs created out of the same input in the
same process like
ffmpeg -i input -filter_complex '[0:v]yadif,split=3[out1][out2][out3]' \
-map '[out1]' -s 1280x720 -acodec … -vcodec … output1 \
-map '[out2]' -s 640x
The h264_nvenc and hevc_nvenc encoders aren't respecting the framerate in the
codec context. Instead it was using the timebase which in our use-case was
1/1000 so the encoder was behaving as if we wanted 1000fps. This resulted in
poor encoding results due to an extremely low bitrate.
Both the a
On Thu, May 14, 2020 at 5:09 AM Derek Buitenhuis
wrote:
> On 11/05/2020 17:35, Thierry Foucu wrote:
> > If you have a file with multiple Metadata Keys, the second time you parse
> > the keys, you will re-alloc c->meta_keys without freeing the old one.
> > This change will avoid parsing all the co
On Tue, 12. May 10:37, Marton Balint wrote:
>
>
> On Mon, 11 May 2020, Andriy Gelman wrote:
>
> > On Fri, 01. May 17:54, Marton Balint wrote:
> > >
> > >
> > > On Thu, 30 Apr 2020, Andriy Gelman wrote:
> > >
> > > > From: Andriy Gelman
> > > > > A temporary heap array currently stores pids f
From: Martin Storsjö
This was ported from dav1d (c950e7101bdf5f7117bfca816984a21e550509f0).
Signed-off-by: Josh de Kock
---
tests/checkasm/checkasm.c | 42 +++
tests/checkasm/checkasm.h | 16 +++
2 files changed, 58 insertions(+)
diff --git a/te
This makes it easier to share code with e.g. the dav1d implementation
of checkasm.
---
tests/checkasm/aarch64/checkasm.S | 196 +++---
tests/checkasm/arm/checkasm.S | 144 +++---
2 files changed, 170 insertions(+), 170 deletions(-)
diff --git a/tests/ch
On windows and darwin (and modern android), the x18 register is reserved
and shouldn't be modified by user code, while it is freely available on
linux. Strictly avoid it, to keep the assembly code portable.
This would have helped catch the issue fixed in 872790b1f9be6
immediately.
---
libavutil/a
Also fill x8-x17 with garbage before calling the function.
Figure out the number of stack parameters and make sure that the
value on the stack after those is untouched.
---
tests/checkasm/aarch64/checkasm.S | 47 +--
tests/checkasm/checkasm.h | 7 +++--
2 file
Figure out the number of stack parameters and make sure that the
value on the stack after those is untouched.
---
tests/checkasm/arm/checkasm.S | 35 ---
tests/checkasm/checkasm.h | 6 --
2 files changed, 36 insertions(+), 5 deletions(-)
diff --git a/tests
We should just use a normal bl here, and the linker will add the 'x'
bit if necessary.
This fixes calling the checkasm_fail_func on windows, where the
code is built in thumb mode (and the linker doesn't clear the 'x'
bit in the blx instruction).
---
tests/checkasm/arm/checkasm.S | 2 +-
1 file ch
On 11/05/2020 17:35, Thierry Foucu wrote:
> If you have a file with multiple Metadata Keys, the second time you parse
> the keys, you will re-alloc c->meta_keys without freeing the old one.
> This change will avoid parsing all the consecutive Metadata keys.
> ---
> libavformat/mov.c | 3 ++-
> 1 f
On Fri, Sep 20, 2019 at 11:08:36AM +0800, lance.lmw...@gmail.com wrote:
> From: Limin Wang
>
> have tested on linux x86_32/64, mingw32/64 arm & mips qemu
will apply it tomorrow if no comments
>
> Tested-by: Michael Niedermayer
> Signed-off-by: Limin Wang
> ---
> tests/fate/filter-video.mak
From: Limin Wang
Reviewed-by: Paul B Mahol
Signed-off-by: Limin Wang
---
rebase with master and update the version and Changelog
Changelog| 1 +
configure| 1 +
doc/filters.texi | 35
libavfilter/Makefile | 1 +
libavfilter/allfilter
From: Limin Wang
Reviewed-by: Paul B Mahol
Signed-off-by: Limin Wang
---
libavfilter/vf_minterpolate.c | 16
1 file changed, 12 insertions(+), 4 deletions(-)
diff --git a/libavfilter/vf_minterpolate.c b/libavfilter/vf_minterpolate.c
index b0bb238..db3d7d4 100644
--- a/libavfi
From: Limin Wang
Reviewed-by: Paul B Mahol
Signed-off-by: Limin Wang
---
libavfilter/vf_framerate.c | 15 +++
1 file changed, 11 insertions(+), 4 deletions(-)
diff --git a/libavfilter/vf_framerate.c b/libavfilter/vf_framerate.c
index 6c8d01c..8d16998 100644
--- a/libavfilter/vf_fr
From: Limin Wang
Reviewed-by: Paul B Mahol
Signed-off-by: Limin Wang
---
tests/fate/filter-video.mak | 6 ++
tests/ref/fate/filter-metadata-scdet | 11 +++
2 files changed, 17 insertions(+)
create mode 100644 tests/ref/fate/filter-metadata-scdet
diff --git a/tests/fate/
On Thu, May 14, 2020 at 10:33:00AM +0530, Gyan Doshi wrote:
>
>
> On 14-05-2020 06:55 am, lance.lmw...@gmail.com wrote:
> >
> > will rebase and apply it tomorrow if no more comments as it's reviewed long
> > time. I need it
> > for other filter to use.
> What would be helpful is to add options
From: Limin Wang
Signed-off-by: Limin Wang
---
doc/filters.texi | 13 +
libavfilter/vf_drawbox.c | 27 +++
2 files changed, 40 insertions(+)
diff --git a/doc/filters.texi b/doc/filters.texi
index 76e12ef..f838dfc 100644
--- a/doc/filters.texi
+++ b/d
Hi,
FFmpeg supports multiple outputs created out of the same input in the
same process like
ffmpeg -i input -filter_complex '[0:v]yadif,split=3[out1][out2][out3]' \
-map '[out1]' -s 1280x720 -acodec … -vcodec … output1 \
-map '[out2]' -s 640x480 -acodec … -vcodec … output2 \
54 matches
Mail list logo
