It shouldn't allocate buffer before the resolution
change event appeared in decoder setup.
And it should not apply to new resolution before
the buffer from the previous sequence is dequeued.
Change-Id: Id04550b0f17e1501b670a3bcbdd860d5836259bf
Signed-off-by: Hsia-Jun(Randy) Li
---
> -Original Message-
> From: ffmpeg-devel On Behalf Of
> Ronald S. Bultje
> Sent: Friday, June 30, 2023 1:27 AM
> To: FFmpeg development discussions and patches de...@ffmpeg.org>
> Subject: Re: [FFmpeg-devel] [PATCH v3] avformat/ivfenc: Set the "number of
> frames" in IVF header
>
>
Fixes: memleak
Fixes:
60058/clusterfuzz-testcase-minimized-ffmpeg_dem_IFF_fuzzer-5665259244093440
Found-by: continuous fuzzing process
https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Signed-off-by: Michael Niedermayer
---
libavformat/id3v2.c | 4 +++-
1 file changed, 3
Fixes: signed integer overflow: 2147443649 + 65535 cannot be represented in
type 'int'
Fixes:
60054/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_RKA_fuzzer-5095674572832768
Found-by: continuous fuzzing process
https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg
Signed-off-by:
Signed-off-by: Andreas Rheinhardt
---
libavcodec/hevc_ps.c | 17 +
1 file changed, 9 insertions(+), 8 deletions(-)
diff --git a/libavcodec/hevc_ps.c b/libavcodec/hevc_ps.c
index d7930d3ac3..4c4c1e2c17 100644
--- a/libavcodec/hevc_ps.c
+++ b/libavcodec/hevc_ps.c
@@ -725,7 +725,8
On 6/29/2023 7:28 PM, Andreas Rheinhardt wrote:
Signed-off-by: Andreas Rheinhardt
---
libavcodec/evc_ps.c | 8 ++--
1 file changed, 2 insertions(+), 6 deletions(-)
diff --git a/libavcodec/evc_ps.c b/libavcodec/evc_ps.c
index ec405345ae..fc2105b352 100644
--- a/libavcodec/evc_ps.c
+++
On 6/29/2023 7:28 PM, Andreas Rheinhardt wrote:
Signed-off-by: Andreas Rheinhardt
---
libavcodec/vvc_parser.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavcodec/vvc_parser.c b/libavcodec/vvc_parser.c
index 69696eef57..0a8ef9f177 100644
---
Signed-off-by: James Almer
---
tests/fate/cbs.mak | 40 ++
tests/ref/fate/cbs-vvc-AUD_A_3 | 1 +
tests/ref/fate/cbs-vvc-BOUNDARY_A_3| 1 +
tests/ref/fate/cbs-vvc-BUMP_A_2| 1 +
tests/ref/fate/cbs-vvc-CROP_B_4
Signed-off-by: James Almer
---
libavcodec/cbs_h2645.c| 35 ---
libavcodec/cbs_h266.h | 17 +++--
libavcodec/cbs_h266_syntax_template.c | 17 ++---
libavcodec/h266_metadata_bsf.c| 13 +-
Signed-off-by: Andreas Rheinhardt
---
libavcodec/evc_ps.c | 8 ++--
1 file changed, 2 insertions(+), 6 deletions(-)
diff --git a/libavcodec/evc_ps.c b/libavcodec/evc_ps.c
index ec405345ae..fc2105b352 100644
--- a/libavcodec/evc_ps.c
+++ b/libavcodec/evc_ps.c
@@ -143,12 +143,10 @@ int
Signed-off-by: Andreas Rheinhardt
---
libavcodec/vvc_parser.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavcodec/vvc_parser.c b/libavcodec/vvc_parser.c
index 69696eef57..0a8ef9f177 100644
--- a/libavcodec/vvc_parser.c
+++ b/libavcodec/vvc_parser.c
@@ -496,7 +496,7 @@
On 6/29/2023 6:53 PM, Andreas Rheinhardt wrote:
VVCParserContext.au_info is only used once (and in a read-only manner);
but this happens immediately after au_info has been completely
overwritten. Therefore one can just the src structure used to overwrite
au_info directly and remove au_info.
VVCParserContext.au_info is only used once (and in a read-only manner);
but this happens immediately after au_info has been completely
overwritten. Therefore one can just the src structure used to overwrite
au_info directly and remove au_info.
This also means that the whole referencing and
Signed-off-by: James Almer
---
fftools/ffplay.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fftools/ffplay.c b/fftools/ffplay.c
index 15fd644974..663f61d8b2 100644
--- a/fftools/ffplay.c
+++ b/fftools/ffplay.c
@@ -2639,7 +2639,7 @@ static int
On 6/29/2023 5:54 PM, Andreas Rheinhardt wrote:
Signed-off-by: Andreas Rheinhardt
---
libavcodec/vvc_parser.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/libavcodec/vvc_parser.c b/libavcodec/vvc_parser.c
index f350fe70a9..efea833565 100644
--- a/libavcodec/vvc_parser.c
+++
On 6/28/23 20:12, Leo Izen wrote:
On 6/16/23 13:46, Kacper Michajlow wrote:
On Thu, 1 Jun 2023 at 21:44, Kacper Michajłow wrote:
Content-Type can include charset and boundary which is not a part of
mime type and shouldn't be copied as such.
Fixes HLS playback when the Content-Type includes
Signed-off-by: James Almer
---
tests/fate/cbs.mak | 40 ++
tests/ref/fate/cbs-vvc-AUD_A_3 | 1 +
tests/ref/fate/cbs-vvc-BOUNDARY_A_3| 1 +
tests/ref/fate/cbs-vvc-BUMP_A_2| 1 +
tests/ref/fate/cbs-vvc-CROP_B_4
Signed-off-by: James Almer
---
tests/fate/cbs.mak | 40 ++
tests/ref/fate/cbs-vvc-AUD_A_3 | 1 +
tests/ref/fate/cbs-vvc-BOUNDARY_A_3| 1 +
tests/ref/fate/cbs-vvc-BUMP_A_2| 1 +
tests/ref/fate/cbs-vvc-CROP_B_4
Signed-off-by: Andreas Rheinhardt
---
libavcodec/vvc_parser.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/libavcodec/vvc_parser.c b/libavcodec/vvc_parser.c
index f350fe70a9..efea833565 100644
--- a/libavcodec/vvc_parser.c
+++ b/libavcodec/vvc_parser.c
@@ -22,9 +22,7 @@
#include
Thomas Siedel:
> From: Nuo Mi
>
> Add nal parser ff_vvc_parser to parse vvc elementary bitstreams.
>
> Co-authored-by: Thomas Siedel
> ---
> configure| 1 +
> libavcodec/Makefile | 2 +
> libavcodec/h2645_parse.c | 71 -
> libavcodec/h266_parser.c | 601
Signed-off-by: James Almer
---
libavcodec/cbs_h2645.c| 31 +++
libavcodec/cbs_h266.h | 7 ++
libavcodec/cbs_h266_syntax_template.c | 7 +++---
3 files changed, 22 insertions(+), 23 deletions(-)
diff --git a/libavcodec/cbs_h2645.c
Paul B Mahol:
> On Thu, Jun 29, 2023 at 9:35 PM Andreas Rheinhardt <
> andreas.rheinha...@outlook.com> wrote:
>
>> Paul B Mahol:
>>> On Thu, Jun 29, 2023 at 8:18 PM Andreas Rheinhardt <
>>> andreas.rheinha...@outlook.com> wrote:
>>>
The discrepancy between the definition and the declaration
On 6/29/2023 4:42 PM, Paul B Mahol wrote:
On Thu, Jun 29, 2023 at 9:35 PM Andreas Rheinhardt <
andreas.rheinha...@outlook.com> wrote:
Paul B Mahol:
On Thu, Jun 29, 2023 at 8:18 PM Andreas Rheinhardt <
andreas.rheinha...@outlook.com> wrote:
The discrepancy between the definition and the
On Thu, Jun 29, 2023 at 9:35 PM Andreas Rheinhardt <
andreas.rheinha...@outlook.com> wrote:
> Paul B Mahol:
> > On Thu, Jun 29, 2023 at 8:18 PM Andreas Rheinhardt <
> > andreas.rheinha...@outlook.com> wrote:
> >
> >> The discrepancy between the definition and the declaration
> >> in allfilters.c
Paul B Mahol:
> On Thu, Jun 29, 2023 at 8:18 PM Andreas Rheinhardt <
> andreas.rheinha...@outlook.com> wrote:
>
>> The discrepancy between the definition and the declaration
>> in allfilters.c is actually UB.
>>
>
> I get no such message with ubsan.
>
UBSan is a runtime UB-detector, not a
Paul B Mahol:
> On Wed, Jun 28, 2023 at 7:34 PM Andreas Rheinhardt <
> andreas.rheinha...@outlook.com> wrote:
>
>> Paul B Mahol:
>>> On Wed, Jun 28, 2023 at 11:57 AM Paul B Mahol wrote:
>>>
On Wed, Jun 28, 2023 at 11:11 AM Andreas Rheinhardt <
andreas.rheinha...@outlook.com>
On Thu, Jun 29, 2023 at 8:18 PM Andreas Rheinhardt <
andreas.rheinha...@outlook.com> wrote:
> The discrepancy between the definition and the declaration
> in allfilters.c is actually UB.
>
I get no such message with ubsan.
>
> Signed-off-by: Andreas Rheinhardt
> ---
>
On Wed, Jun 28, 2023 at 7:34 PM Andreas Rheinhardt <
andreas.rheinha...@outlook.com> wrote:
> Paul B Mahol:
> > On Wed, Jun 28, 2023 at 11:57 AM Paul B Mahol wrote:
> >
> >>
> >>
> >> On Wed, Jun 28, 2023 at 11:11 AM Andreas Rheinhardt <
> >> andreas.rheinha...@outlook.com> wrote:
> >>
> >>>
James Almer:
> On 6/29/2023 3:17 PM, Andreas Rheinhardt wrote:
>> Fixes a warning from GCC when in pedantic mode:
>> "extra semicolon in struct or union specified"
>>
>> Signed-off-by: Andreas Rheinhardt
>> ---
>> libavcodec/cbs_h266.h | 4 ++--
>> 1 file changed, 2 insertions(+), 2
On 6/29/2023 3:19 PM, Andreas Rheinhardt wrote:
The discrepancy between the definition and the declaration
in allfilters.c is actually UB.
Signed-off-by: Andreas Rheinhardt
---
libavfilter/vf_ccrepack.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
On 6/29/2023 3:19 PM, Andreas Rheinhardt wrote:
The discrepancy between the definition and the declaration
in parsers.c is actually UB.
Signed-off-by: Andreas Rheinhardt
---
libavcodec/vvc_parser.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavcodec/vvc_parser.c
The discrepancy between the definition and the declaration
in allformats.c is actually UB.
Signed-off-by: Andreas Rheinhardt
---
Will apply this patchset soon.
libavformat/ac4dec.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavformat/ac4dec.c b/libavformat/ac4dec.c
On 6/29/2023 3:17 PM, Andreas Rheinhardt wrote:
Fixes a warning from GCC when in pedantic mode:
"extra semicolon in struct or union specified"
Signed-off-by: Andreas Rheinhardt
---
libavcodec/cbs_h266.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git
The discrepancy between the definition and the declaration
in allfilters.c is actually UB.
Signed-off-by: Andreas Rheinhardt
---
libavfilter/vf_ccrepack.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavfilter/vf_ccrepack.c b/libavfilter/vf_ccrepack.c
index
The discrepancy between the definition and the declaration
in parsers.c is actually UB.
Signed-off-by: Andreas Rheinhardt
---
libavcodec/vvc_parser.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavcodec/vvc_parser.c b/libavcodec/vvc_parser.c
index
Fixes a warning from GCC when in pedantic mode:
"extra semicolon in struct or union specified"
Signed-off-by: Andreas Rheinhardt
---
libavcodec/cbs_h266.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/libavcodec/cbs_h266.h b/libavcodec/cbs_h266.h
index
Add an optional filter_line3 to the available optimisations.
filter_line3 is equivalent to filter_line, memcpy, filter_line
filter_line shares quite a number of loads and some calculations in
common with its next iteration and testing shows that using aarch64
neon filter_line3s performance is
Signed-off-by: John Cox
---
libavfilter/aarch64/vf_bwdif_init_aarch64.c | 21 ++
libavfilter/aarch64/vf_bwdif_neon.S | 215
2 files changed, 236 insertions(+)
diff --git a/libavfilter/aarch64/vf_bwdif_init_aarch64.c
b/libavfilter/aarch64/vf_bwdif_init_aarch64.c
Needed for tail fixup of neon code
Signed-off-by: John Cox
---
libavfilter/bwdif.h| 5 +
libavfilter/vf_bwdif.c | 10 +-
2 files changed, 10 insertions(+), 5 deletions(-)
diff --git a/libavfilter/bwdif.h b/libavfilter/bwdif.h
index ae1616d366..cce99953f3 100644
---
Signed-off-by: John Cox
---
tests/checkasm/vf_bwdif.c | 54 +++
1 file changed, 54 insertions(+)
diff --git a/tests/checkasm/vf_bwdif.c b/tests/checkasm/vf_bwdif.c
index 034bbabb4c..5fdba09fdc 100644
--- a/tests/checkasm/vf_bwdif.c
+++
Round job start lines down to a multiple of 4. This means that if
filter_line3 exists then filter_line will not sometimes be called
once at the end of a slice depending on thread count. The final slice
may do up to 3 extra lines but filter_edge is faster than filter_line
so it is unlikely to
Signed-off-by: John Cox
---
libavfilter/aarch64/vf_bwdif_init_aarch64.c | 20
libavfilter/aarch64/vf_bwdif_neon.S | 104
2 files changed, 124 insertions(+)
diff --git a/libavfilter/aarch64/vf_bwdif_init_aarch64.c
b/libavfilter/aarch64/vf_bwdif_init_aarch64.c
Signed-off-by: John Cox
---
tests/checkasm/vf_bwdif.c | 81 +++
1 file changed, 81 insertions(+)
diff --git a/tests/checkasm/vf_bwdif.c b/tests/checkasm/vf_bwdif.c
index 5fdba09fdc..3399cacdf7 100644
--- a/tests/checkasm/vf_bwdif.c
+++
Needed for tail fixup of neon code
Signed-off-by: John Cox
---
libavfilter/bwdif.h| 4
libavfilter/vf_bwdif.c | 8
2 files changed, 8 insertions(+), 4 deletions(-)
diff --git a/libavfilter/bwdif.h b/libavfilter/bwdif.h
index ae6f6ce223..ae1616d366 100644
---
Signed-off-by: John Cox
---
libavfilter/aarch64/vf_bwdif_init_aarch64.c | 28 ++
libavfilter/aarch64/vf_bwdif_neon.S | 278
2 files changed, 306 insertions(+)
diff --git a/libavfilter/aarch64/vf_bwdif_init_aarch64.c
b/libavfilter/aarch64/vf_bwdif_init_aarch64.c
Signed-off-by: John Cox
---
libavfilter/aarch64/vf_bwdif_neon.S | 59 +
1 file changed, 59 insertions(+)
diff --git a/libavfilter/aarch64/vf_bwdif_neon.S
b/libavfilter/aarch64/vf_bwdif_neon.S
index b863b3447d..6c5d1598f4 100644
---
Signed-off-by: John Cox
---
tests/checkasm/vf_bwdif.c | 37 +
1 file changed, 37 insertions(+)
diff --git a/tests/checkasm/vf_bwdif.c b/tests/checkasm/vf_bwdif.c
index 46224bb575..034bbabb4c 100644
--- a/tests/checkasm/vf_bwdif.c
+++
Signed-off-by: John Cox
---
libavfilter/aarch64/vf_bwdif_init_aarch64.c | 17 +++
libavfilter/aarch64/vf_bwdif_neon.S | 53 +
2 files changed, 70 insertions(+)
diff --git a/libavfilter/aarch64/vf_bwdif_init_aarch64.c
b/libavfilter/aarch64/vf_bwdif_init_aarch64.c
Needed for tail fixup of neon code
Signed-off-by: John Cox
---
libavfilter/bwdif.h| 3 +++
libavfilter/vf_bwdif.c | 6 +++---
2 files changed, 6 insertions(+), 3 deletions(-)
diff --git a/libavfilter/bwdif.h b/libavfilter/bwdif.h
index 6a0f70487a..ae6f6ce223 100644
---
Add macros for dual scalar half->single multiply and accumulate
Add macro for shift, saturate and shorten single to byte
Add filter constants
Signed-off-by: John Cox
---
libavfilter/aarch64/vf_bwdif_neon.S | 46 +
1 file changed, 46 insertions(+)
diff --git
Outline but no actual functions.
Signed-off-by: John Cox
---
libavfilter/aarch64/Makefile| 2 ++
libavfilter/aarch64/vf_bwdif_init_aarch64.c | 39 +
libavfilter/aarch64/vf_bwdif_neon.S | 25 +
libavfilter/bwdif.h |
Also adds a filter_line3 method which on aarch64 neon yields approx 30%
speedup over 2xfilter_line and a memcpy
John Cox (15):
avfilter/vf_bwdif: Add outline for aarch neon functions
avfilter/vf_bwdif: Add common macros and consts for aarch64 neon
avfilter/vf_bwdif: Export C filter_intra
On 3/21/2023 12:01 PM, Thomas Siedel wrote:
This patch set adds H266/VVC support.
This includes parsing, muxing, demuxing, decoding and encoding.
Decoding is done using the external library VVdeC
(https://github.com/fraunhoferhhi/vvdec.git) and can be enabled with
--enable-libvvdec.
Encoding is
Hi,
On Thu, Jun 29, 2023 at 8:54 AM Dai, Jianhui J <
jianhui.j.dai-at-intel@ffmpeg.org> wrote:
> Should set "number of frames" to bytes 24-27 of IVF header, not
> duration.
> It is described by [1], and confirmed by parsing all IVF files in [2].
>
> This commit also updates the md5sum of
If you apply this I will apply my pending libswresample commits and also
remove sonic decoder from libavcodec.
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or
On 5/17/2023 9:59 AM, Dmitrii Ovchinnikov wrote:
---
configure | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/configure b/configure
index bb7be67676..937be943a4 100755
--- a/configure
+++ b/configure
@@ -7082,7 +7082,7 @@ fi
enabled amf &&
check_cpp_condition
Should set "number of frames" to bytes 24-27 of IVF header, not
duration.
It is described by [1], and confirmed by parsing all IVF files in [2].
This commit also updates the md5sum of refs to pass fate-cbs.
[1] Duck IVF - MultimediaWiki
https://wiki.multimedia.cx/index.php/Duck_IVF
[2]
Hello,
On Thu, 29 Jun 2023, at 09:14, Nicolas George wrote:
>>The reason why XML parsing shouldn't be
>> implemented in FFmpeg isn't just because it is difficult but because:
>>
>> * it is out of scope
>
> It is necessary.
Says you.
This is not an opinion that seems to
Quoting Dai, Jianhui J (2023-06-29 08:03:18)
>
>
> > -Original Message-
> > From: ffmpeg-devel On Behalf Of
> > Anton Khirnov
> > Sent: Wednesday, June 28, 2023 11:25 PM
> > To: ffmpeg-devel@ffmpeg.org
> > Subject: Re: [FFmpeg-devel] [PATCH v2] avformat/ivfenc: Set the "number of
> >
Tomas Härdin (12023-06-29):
> This is very revealing.
Indeed.
> The reason why XML parsing shouldn't be
> implemented in FFmpeg isn't just because it is difficult but because:
>
> * it is out of scope
It is necessary.
> * it is technical debt
> * shitty parsing is the
> -Original Message-
> From: ffmpeg-devel On Behalf Of
> Anton Khirnov
> Sent: Wednesday, June 28, 2023 11:25 PM
> To: ffmpeg-devel@ffmpeg.org
> Subject: Re: [FFmpeg-devel] [PATCH v2] avformat/ivfenc: Set the "number of
> frames" in IVF header
>
> Quoting Dai, Jianhui J (2023-06-05
61 matches
Mail list logo
