On 07-12-2019 12:36 am, Michael Niedermayer wrote:
On Wed, Dec 04, 2019 at 02:14:35PM +0530, Gyan wrote:
Will help reduce code duplication when adding animation support to vf_scale.
See Michael's last comment in https://patchwork.ffmpeg.org/patch/16272/
Gyan
doc/filters.texi |
Hi,
On Fri, Nov 29, 2019 at 2:42 PM Fu, Linjie wrote:
> > -Original Message-
> > From: ffmpeg-devel On Behalf Of
> > Zhong Li
> > Sent: Friday, November 29, 2019 13:13
> > To: FFmpeg development discussions and patches > de...@ffmpeg.org>
> > Subject: Re: [FFmpeg-devel] [PATCH]
From: Limin Wang
Fix CID 1456556
Signed-off-by: Limin Wang
---
libavfilter/vsrc_cellauto.c | 10 ++
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/libavfilter/vsrc_cellauto.c b/libavfilter/vsrc_cellauto.c
index c754603..03bdaba 100644
--- a/libavfilter/vsrc_cellauto.c
From: Limin Wang
Fix CID 1456556
Signed-off-by: Limin Wang
---
libavfilter/vsrc_life.c | 10 ++
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/libavfilter/vsrc_life.c b/libavfilter/vsrc_life.c
index 4c0ea73..01b060a 100644
--- a/libavfilter/vsrc_life.c
+++
On 12/6/2019 8:08 PM, Michael Niedermayer wrote:
> On Fri, Dec 06, 2019 at 11:56:13AM -0300, James Almer wrote:
>> Should increase coverage of some decoders.
>>
>> Signed-off-by: James Almer
>> ---
>> Passing 6 to av_get_default_channel_layout() will return the 5.1 layout but
>> never 5.1(side),
On 12/6/2019 7:59 PM, Michael Niedermayer wrote:
> On Fri, Dec 06, 2019 at 04:16:23PM -0300, James Almer wrote:
>> This should increase coverage on some decoders by executing flushing code.
>>
>> Signed-off-by: James Almer
>> ---
>> tools/target_dec_fuzzer.c | 4
>> 1 file changed, 4
There is no need to recheck this for every frame.
Signed-off-by: Andreas Rheinhardt
---
libavformat/matroskadec.c | 9 +++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/libavformat/matroskadec.c b/libavformat/matroskadec.c
index d1a0a07782..a7226ee810 100644
---
Signed-off-by: Andreas Rheinhardt
---
libavformat/matroskadec.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
diff --git a/libavformat/matroskadec.c b/libavformat/matroskadec.c
index 982dfdec39..5c10862d48 100644
--- a/libavformat/matroskadec.c
+++
rmdec does it, too. And it allows to simplify the check for whether one
should use the Real Audio parsing mode.
Signed-off-by: Andreas Rheinhardt
---
libavformat/matroskadec.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/libavformat/matroskadec.c
ProRes in Matroska is supposed to not contain the first atom header
(containing a size field and the tag "icpf") and therefore the Matroska
demuxer has to recreate it; this involves an allocation and copy, of
course. Whether the old buffer (containing the data without the atom
header) needs to be
The Matroska demuxer has three functions for creating packets out of the
data read: One for certain Real audio codecs (atrac3, cook, sipr,
ra_288), one for WebVTT (actually, the WebM flavour of it) and one for
all the others. Only the last function supported Matroska's
ContentCompression (i.e. it
Signed-off-by: Andreas Rheinhardt
---
libavformat/matroskadec.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/libavformat/matroskadec.c b/libavformat/matroskadec.c
index 162697ff12..908ad097cb 100644
--- a/libavformat/matroskadec.c
+++ b/libavformat/matroskadec.c
@@
On Fri, Dec 06, 2019 at 11:56:13AM -0300, James Almer wrote:
> Should increase coverage of some decoders.
>
> Signed-off-by: James Almer
> ---
> Passing 6 to av_get_default_channel_layout() will return the 5.1 layout but
> never 5.1(side), which is for example what the dts decoder actually looks
On Fri, Dec 06, 2019 at 04:16:23PM -0300, James Almer wrote:
> This should increase coverage on some decoders by executing flushing code.
>
> Signed-off-by: James Almer
> ---
> tools/target_dec_fuzzer.c | 4
"[PATCH] tools/target_dec_bsf:"
this isnt matching
[...]
--
Michael GnuPG
On Fri, Dec 06, 2019 at 04:16:23PM -0300, James Almer wrote:
> This should increase coverage on some decoders by executing flushing code.
>
> Signed-off-by: James Almer
> ---
> tools/target_dec_fuzzer.c | 4
> 1 file changed, 4 insertions(+)
>
> diff --git a/tools/target_dec_fuzzer.c
The old interlace filter worked this way before it was merged with tinterlace.
Signed-off-by: Marton Balint
---
doc/filters.texi| 6 --
libavfilter/tinterlace.h| 1 +
libavfilter/vf_tinterlace.c | 11 +++
3 files changed, 16 insertions(+), 2 deletions(-)
diff
The documentation still mentions numerical constants in addition to textual
ones. It is also wrong to use distinct modes as flags and it disallows us to
actually use the flags field for real flags in the future.
Signed-off-by: Marton Balint
---
libavfilter/tinterlace.h| 7 +++
Fixes crashes in command lines such as:
ffmpeg -f lavfi -i testsrc2=704x576:r=50,interlace,pad=720:576:8 -f null none
Related to ticket #6491.
Signed-off-by: Marton Balint
---
libavfilter/x86/vf_interlace.asm | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git
The filter used to work this way before it was merged into tinterlace.
Signed-off-by: Marton Balint
---
libavfilter/vf_tinterlace.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/libavfilter/vf_tinterlace.c b/libavfilter/vf_tinterlace.c
index 0ee40ffce6..a77753775c 100644
---
On Fri, 06. Dec 18:01, Andreas Rheinhardt wrote:
> Andriy Gelman:
> > From: Andriy Gelman
> >
> > payload_count is used to track the number of SEI payloads. It is also
> > used to free the SEIs in cbs_h265_free_sei().
> >
> > Currently, payload_count is set after for loop is completed. Hence if
On Fri, 06. Dec 17:47, Andreas Rheinhardt wrote:
> Andriy Gelman:
> > From: Andriy Gelman
> >
> > In the worst case the startcode prefix has 4 bytes.
> >
> > This fixes a trigerred assertion:
> > Assertion dp <= max_size failed at libavcodec/cbs_h2645.c:1451
> >
> > Found-by:libFuzzer
> >
From: Andriy Gelman
payload_count is used to track the number of SEI payloads. It is also
used to free the SEIs in cbs_h264_free_sei()/cbs_h265_free_sei().
Currently, payload_count is set after for loop is completed. Hence if
there is an error and the function exits, the payload remains zero
This should increase coverage on some decoders by executing flushing code.
Signed-off-by: James Almer
---
tools/target_dec_fuzzer.c | 4
1 file changed, 4 insertions(+)
diff --git a/tools/target_dec_fuzzer.c b/tools/target_dec_fuzzer.c
index dcf47b0f4d..3c2f9125bb 100644
---
On Wed, Dec 04, 2019 at 02:14:35PM +0530, Gyan wrote:
> Will help reduce code duplication when adding animation support to vf_scale.
> See Michael's last comment in https://patchwork.ffmpeg.org/patch/16272/
>
> Gyan
> doc/filters.texi | 40 +
>
On 12/5/19, Paul B Mahol wrote:
> On 12/5/19, Tao Zhang wrote:
>> Hello everyone,
>> Can I assume this patch is ok if no comments or objections?
>
> Yes, give some time for it to be applied.
> I'm busy with other stuff right now. So this patch LGTM (Note to
> committer to bump minor of
On 12/6/2019 2:47 PM, Andreas Rheinhardt wrote:
> Andriy Gelman:
>> From: Andriy Gelman
>>
>> In the worst case the startcode prefix has 4 bytes.
>>
>> This fixes a trigerred assertion:
>> Assertion dp <= max_size failed at libavcodec/cbs_h2645.c:1451
>>
>> Found-by:libFuzzer
>> Signed-off-by:
Andriy Gelman:
> From: Andriy Gelman
>
> payload_count is used to track the number of SEI payloads. It is also
> used to free the SEIs in cbs_h265_free_sei().
>
> Currently, payload_count is set after for loop is completed. Hence if
> there is an error and the function exits, the
Andriy Gelman:
> From: Andriy Gelman
>
> In the worst case the startcode prefix has 4 bytes.
>
> This fixes a trigerred assertion:
> Assertion dp <= max_size failed at libavcodec/cbs_h2645.c:1451
>
> Found-by:libFuzzer
> Signed-off-by: Andriy Gelman
> ---
> libavcodec/cbs_h2645.c | 2 +-
> 1
From: Andriy Gelman
payload_count is used to track the number of SEI payloads. It is also
used to free the SEIs in cbs_h265_free_sei().
Currently, payload_count is set after for loop is completed. Hence if
there is an error and the function exits, the payload_remains zero
causing a memleak.
This is a micro-optimization. Saving almost 200 reallocations makes it
worth a try.
---
fix commit message typo: relocations -> reallocations
libavfilter/formats.c | 35 +--
1 file changed, 21 insertions(+), 14 deletions(-)
diff --git a/libavfilter/formats.c
This is a micro-optimization. Saving almost 200 relocations makes it
worth a try.
---
Can lavf depends on the sequential properties of AVPixelFormat and
AVSampleFormat? Or should limited to pixdesc.h/samplefmt.h API?
libavfilter/formats.c | 35 +--
1 file changed,
On 04-12-2019 02:14 pm, Gyan wrote:
Will help reduce code duplication when adding animation support to
vf_scale.
See Michael's last comment in https://patchwork.ffmpeg.org/patch/16272/
Ping.
Gyan
___
ffmpeg-devel mailing list
Signed-off-by: Michael Niedermayer
---
libavfilter/vf_geq.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/libavfilter/vf_geq.c b/libavfilter/vf_geq.c
index e3267e331f..c965204a1c 100644
--- a/libavfilter/vf_geq.c
+++ b/libavfilter/vf_geq.c
@@ -33,6 +33,8 @@
#include
This allows integrating box blur style filters in geq.
Without this computing the mean of an area in geq would have been excessivly
slow
Signed-off-by: Michael Niedermayer
---
libavfilter/vf_geq.c | 107 +--
1 file changed, 103 insertions(+), 4
Signed-off-by: Michael Niedermayer
---
libavutil/eval.c | 28
libavutil/eval.h | 11 +++
2 files changed, 31 insertions(+), 8 deletions(-)
diff --git a/libavutil/eval.c b/libavutil/eval.c
index 62d2ae938b..d527f6a9d0 100644
--- a/libavutil/eval.c
+++
From: Andriy Gelman
In the worst case the startcode prefix has 4 bytes.
This fixes a trigerred assertion:
Assertion dp <= max_size failed at libavcodec/cbs_h2645.c:1451
Found-by:libFuzzer
Signed-off-by: Andriy Gelman
---
libavcodec/cbs_h2645.c | 2 +-
1 file changed, 1 insertion(+), 1
Should increase coverage of some decoders.
Signed-off-by: James Almer
---
Passing 6 to av_get_default_channel_layout() will return the 5.1 layout but
never 5.1(side), which is for example what the dts decoder actually looks for.
Any suggestion on how to make the fuzzer pick one of the non
From: Andriy Gelman
Since the nal unit payload is located after the length prefix, there is
no reason to continue reading if there is no nal unit payload remaining.
---
libavcodec/hevc_mp4toannexb_bsf.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
From: Andriy Gelman
Fixes a check where the grow_by packet size cannot exceed INT_MAX.
Although the complete heap allocation check should ensure that
(uint64_t)nalu_size + extra_size + 4 + out->size + AV_INPUT_BUFFER_PADDING_SIZE
<= INT_MAX,
it suffices to only check that the grow_by size
Dec 3, 2019, 21:23 by mich...@niedermayer.cc:
> Fixes: index 8 out of bounds for type 'const uint8_t [8][3]'
> Fixes:
> 19127/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_ATRAC9_fuzzer-5709394985091072
>
> Found-by: continuous fuzzing process
>
Ping.
Thanks,
Huiwen Ren
At 2019-10-12 11:15:55, "Steven Liu" wrote:
>
>
>> 在 2019年10月12日，09:28，hwren 写道：
>>
>> From: hwrenx
>>
>> Signed-off-by: hwrenx
>> ---
>> MAINTAINERS | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 7f60ef0..928f4ee
Signed-off-by: Vittorio Giovara
---
libavutil/frame.c | 113 ++
libavutil/frame.h | 12 -
2 files changed, 104 insertions(+), 21 deletions(-)
diff --git a/libavutil/frame.c b/libavutil/frame.c
index e4038096c2..600f82bbeb 100644
---
From: Vittorio Giovara
Signed-off-by: Vittorio Giovara
---
libavformat/aea.c | 11 ++-
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/libavformat/aea.c b/libavformat/aea.c
index bdeed64254..8acfbce4fc 100644
--- a/libavformat/aea.c
+++ b/libavformat/aea.c
@@ -62,12
---
libavformat/adp.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/libavformat/adp.c b/libavformat/adp.c
index 56f302acfd..ac80c7809f 100644
--- a/libavformat/adp.c
+++ b/libavformat/adp.c
@@ -55,8 +55,7 @@ static int adp_read_header(AVFormatContext *s)
---
libavformat/ads.c | 11 ++-
1 file changed, 6 insertions(+), 5 deletions(-)
diff --git a/libavformat/ads.c b/libavformat/ads.c
index f25141b3c6..20be40f499 100644
--- a/libavformat/ads.c
+++ b/libavformat/ads.c
@@ -47,11 +47,12 @@ static int ads_read_header(AVFormatContext *s)
The new API is more extensible and allows for custom layouts.
More accurate information is exported, eg for decoders that do not
set a channel layout, lavc will not make one up for them.
Deprecate the old API working with just uint64_t bitmasks.
Expanded and completed by Vittorio Giovara .
They are incompatible with the new channel layout scheme and no decoder
uses them.
Signed-off-by: Vittorio Giovara
---
libavcodec/avcodec.h | 5 +
libavcodec/decode.c | 4
libavformat/dump.c | 9 -
libavformat/nutenc.c | 2 ++
libavformat/utils.c | 9 +
5 files
Mediates between old-style (de)muxers and new-style callers. Will be
removed once all the (de)muxers are converted to the new API.
---
libavformat/mux.c | 10 ++
libavformat/utils.c | 12
2 files changed, 22 insertions(+)
diff --git a/libavformat/mux.c b/libavformat/mux.c
Signed-off-by: Vittorio Giovara
---
libavutil/opt.c | 66 -
libavutil/opt.h | 14 ++-
2 files changed, 78 insertions(+), 2 deletions(-)
diff --git a/libavutil/opt.c b/libavutil/opt.c
index 9081a593a1..3e776804a6 100644
---
---
libavformat/3dostr.c | 13 +++--
1 file changed, 7 insertions(+), 6 deletions(-)
diff --git a/libavformat/3dostr.c b/libavformat/3dostr.c
index 6c49f7589c..bfa52c0da5 100644
--- a/libavformat/3dostr.c
+++ b/libavformat/3dostr.c
@@ -63,15 +63,16 @@ static int
From: Vittorio Giovara
Signed-off-by: Vittorio Giovara
---
libavformat/4xm.c | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/libavformat/4xm.c b/libavformat/4xm.c
index a6101a92ec..864e525e09 100644
--- a/libavformat/4xm.c
+++ b/libavformat/4xm.c
@@ -184,13 +184,14
---
libavformat/aadec.c | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/libavformat/aadec.c b/libavformat/aadec.c
index b9dd51ebfc..adf0869a65 100644
--- a/libavformat/aadec.c
+++ b/libavformat/aadec.c
@@ -188,7 +188,8 @@ static int aa_read_header(AVFormatContext *s)
---
libavformat/act.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/libavformat/act.c b/libavformat/act.c
index 26425ca1bb..2ba2673998 100644
--- a/libavformat/act.c
+++ b/libavformat/act.c
@@ -87,7 +87,8 @@ static int read_header(AVFormatContext *s)
}
---
libavformat/acm.c | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/libavformat/acm.c b/libavformat/acm.c
index 5e03cf8bff..30086198dc 100644
--- a/libavformat/acm.c
+++ b/libavformat/acm.c
@@ -48,12 +48,13 @@ static int acm_read_header(AVFormatContext *s)
if
---
libavformat/mux.c | 13 -
libavformat/utils.c | 18 +-
2 files changed, 29 insertions(+), 2 deletions(-)
diff --git a/libavformat/mux.c b/libavformat/mux.c
index ae1f26c3da..dc4dc4518c 100644
--- a/libavformat/mux.c
+++ b/libavformat/mux.c
@@ -333,8 +333,19 @@
From: Vittorio Giovara
Signed-off-by: Vittorio Giovara
Signed-off-by: Anton Khirnov
---
libavformat/adxdec.c | 17 ++---
1 file changed, 10 insertions(+), 7 deletions(-)
diff --git a/libavformat/adxdec.c b/libavformat/adxdec.c
index 1038a0d67e..5bbd6a901f 100644
---
From: Vittorio Giovara
Signed-off-by: Vittorio Giovara
Signed-off-by: Anton Khirnov
---
libavcodec/avcodec.h | 11 +++
libavcodec/utils.c | 46 +++-
2 files changed, 52 insertions(+), 5 deletions(-)
diff --git a/libavcodec/avcodec.h
Is there any comments?
On 2019/11/27 18:34, "ffmpeg-devel on behalf of Tsuchiya, Yuki (SHES)"
wrote:
Hi,
Could you merge this patch if there is no objection?
> -Original Message-
> From: Tsuchiya, Yuki (SHES)
>
> Implemented according to the
On Fri, Sep 20, 2019 at 07:17:27PM +0800, lance.lmw...@gmail.com wrote:
> From: Limin Wang
>
> Signed-off-by: Limin Wang
> ---
> libavformat/libsrt.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
will apply
thx
[...]
--
Michael GnuPG fingerprint:
On Fri, Sep 20, 2019 at 07:17:28PM +0800, lance.lmw...@gmail.com wrote:
> From: Limin Wang
>
> Signed-off-by: Limin Wang
> ---
> libavformat/libsrt.c | 1 +
> 1 file changed, 1 insertion(+)
will apply
thx
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
No
On Thu, Dec 05, 2019 at 11:10:43AM +0800, Ting Fu wrote:
> Signed-off-by: Ting Fu
> ---
> libswscale/swscale_unscaled.c | 2 --
> 1 file changed, 2 deletions(-)
will apply
thx
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Avoid a single point of failure,
On Thu, Dec 05, 2019 at 11:39:48AM +0100, Paul B Mahol wrote:
> On 12/5/19, Tao Zhang wrote:
> > Hello everyone,
> > Can I assume this patch is ok if no comments or objections?
>
> Yes, give some time for it to be applied.
> I'm busy with other stuff right now. So this patch LGTM (Note to
>
On Thu, Dec 05, 2019 at 09:54:44PM +0800, Limin Wang wrote:
> On Wed, Dec 04, 2019 at 08:31:15PM +0100, Michael Niedermayer wrote:
> > On Fri, Nov 22, 2019 at 09:50:36AM +0800, lance.lmw...@gmail.com wrote:
> > > From: Limin Wang
> > >
> > > Signed-off-by: Limin Wang
> > > ---
> > >
On Thu, 5 Dec 2019, Kusanagi Kouichi wrote:
On 2019-12-04 22:32:52 +0100, Marton Balint wrote:
On Wed, 4 Dec 2019, Kusanagi Kouichi wrote:
> On 2019-12-03 21:25:37 +0100, Marton Balint wrote:
> >
> >
> > On Tue, 3 Dec 2019, Kusanagi Kouichi wrote:
> >
> > > On 2019-11-19 22:59:56
On Fri, Dec 06, 2019 at 12:37:36AM +0100, Paul B Mahol wrote:
> LGTM
will apply
thx
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Never trust a computer, one day, it may think you are the virus. -- Compn
signature.asc
Description: PGP signature
65 matches
Mail list logo
