[FFmpeg-devel] [PATCH v5 3/7] ogg/opus: implement header packet skip in chained ogg bitstreams.

2025-05-09 Thread Romain Beauxis
--- libavformat/oggdec.c | 4 -- libavformat/oggparseopus.c | 87 tests/ref/fate/ogg-opus-chained-meta.txt | 1 - 3 files changed, 60 insertions(+), 32 deletions(-) diff --git a/libavformat/oggdec.c b/libavformat/oggdec.c index 9baf8040

[FFmpeg-devel] [PATCH] avformat/mpegts: update stream info when PMT ES stream_type changes

2025-05-09 Thread Pavel Koshevoy
I have a several .ts captures where video and audio codec changes even though the PMT version does not change and the PIDs stay the same. This happens during transition to/from slate (mpeg2 video and audio) to network broadcast (hevc video and eac3 audio in private PES). I've updated fate ts-demux

[FFmpeg-devel] [PATCH v5 0/7] Remove chained ogg stream header packets from the demuxer

2025-05-09 Thread Romain Beauxis
## Changes since last revision: * Added mechanism to pass new extradata when decoding ogg packets. * Use it to pass new ogg/vorbis setup data to the vorbis decoder. * Add checks for format change when parsing subsequent ogg/vorbis and ogg/opus chained streams Romain Beauxis (7): libavformat/og

[FFmpeg-devel] [PATCH v5 1/7] libavformat/oggdec.h: Document packet function return value.

2025-05-09 Thread Romain Beauxis
--- libavformat/oggdec.h | 6 ++ 1 file changed, 6 insertions(+) diff --git a/libavformat/oggdec.h b/libavformat/oggdec.h index 43df23f4cb..5225b77a07 100644 --- a/libavformat/oggdec.h +++ b/libavformat/oggdec.h @@ -38,6 +38,12 @@ struct ogg_codec { * -1 if an error occurred or

[FFmpeg-devel] [PATCH v5 7/7] libavformat/oggdec.h: Change paket function documentation to return 1 on header packets only.

2025-05-09 Thread Romain Beauxis
--- libavformat/oggdec.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/oggdec.h b/libavformat/oggdec.h index 5083de646c..c15fbe738e 100644 --- a/libavformat/oggdec.h +++ b/libavformat/oggdec.h @@ -42,8 +42,8 @@ struct ogg_codec { * Attempt to process a p

[FFmpeg-devel] [PATCH v5 5/7] libavformat/oggdec.{c, h}: Add new_extradata, use it to pass extradata to the next decoded packet.

2025-05-09 Thread Romain Beauxis
--- libavformat/oggdec.c | 11 +++ libavformat/oggdec.h | 2 ++ 2 files changed, 13 insertions(+) diff --git a/libavformat/oggdec.c b/libavformat/oggdec.c index 5557eb4a14..cb77cdd994 100644 --- a/libavformat/oggdec.c +++ b/libavformat/oggdec.c @@ -77,6 +77,7 @@ static void free_stream(A

[FFmpeg-devel] [PATCH v5 6/7] ogg/vorbis: implement header packet skip in chained ogg bitstreams.

2025-05-09 Thread Romain Beauxis
--- libavcodec/vorbisdec.c | 37 + libavformat/oggparsevorbis.c | 174 + tests/ref/fate/ogg-vorbis-chained-meta.txt | 3 - 3 files changed, 117 insertions(+), 97 deletions(-) diff --git a/libavcodec/vorbisdec.c b/libavcodec/vorbisdec.c

[FFmpeg-devel] [PATCH v5 4/7] ogg/flac: implement header packet skip in chained ogg bitstreams.

2025-05-09 Thread Romain Beauxis
--- libavformat/oggparseflac.c | 28 ++-- tests/ref/fate/ogg-flac-chained-meta.txt | 2 -- 2 files changed, 26 insertions(+), 4 deletions(-) diff --git a/libavformat/oggparseflac.c b/libavformat/oggparseflac.c index f25ed9cc15..d66b85b09e 100644 --- a/libavforma

[FFmpeg-devel] [PATCH v5 2/7] libavformat/oggdec.{c, h}: Implement packet skip on packet return value of 1

2025-05-09 Thread Romain Beauxis
--- libavformat/oggdec.c | 22 ++ libavformat/oggdec.h | 1 + 2 files changed, 15 insertions(+), 8 deletions(-) diff --git a/libavformat/oggdec.c b/libavformat/oggdec.c index 5339fdd32c..9baf8040a9 100644 --- a/libavformat/oggdec.c +++ b/libavformat/oggdec.c @@ -605,20 +605,2

Re: [FFmpeg-devel] [PATCH] avformat/mpegts: update stream info when PMT ES stream_type changes

2025-05-09 Thread Pavel Koshevoy
I will send a new patch with additional changes to handle stream_type == STREAM_TYPE_PRIVATE_DATA. Pavel On Tue, Apr 22, 2025 at 6:51 PM Pavel Koshevoy wrote: > I have a couple of .ts captures where video and audio codec changes > even though the PMT version does not change and the PIDs st

Re: [FFmpeg-devel] [PATCH] fftools/ffmpeg: move elapsed time counter to the end of the status line

2025-05-09 Thread softworkz .
> -Original Message- > From: ffmpeg-devel On Behalf Of Marton > Balint > Sent: Freitag, 9. Mai 2025 20:56 > To: FFmpeg development discussions and patches > Subject: Re: [FFmpeg-devel] [PATCH] fftools/ffmpeg: move elapsed time counter > to the end of the status line > > > > On Fri, 9

Re: [FFmpeg-devel] [PATCH] fftools/ffmpeg: move elapsed time counter to the end of the status line

2025-05-09 Thread Marton Balint
On Fri, 9 May 2025, softworkz . wrote: -Original Message- From: ffmpeg-devel On Behalf Of Marton Balint Sent: Freitag, 9. Mai 2025 14:06 To: ffmpeg-devel@ffmpeg.org Cc: Marton Balint Subject: [FFmpeg-devel] [PATCH] fftools/ffmpeg: move elapsed time counter to the end of the statu

Re: [FFmpeg-devel] [PATCH] fftools/ffmpeg: move elapsed time counter to the end of the status line

2025-05-09 Thread softworkz .
> -Original Message- > From: ffmpeg-devel On Behalf Of James Almer > Sent: Freitag, 9. Mai 2025 18:25 > To: ffmpeg-devel@ffmpeg.org > Subject: Re: [FFmpeg-devel] [PATCH] fftools/ffmpeg: move elapsed time counter > to the end of the status line > > On 5/9/2025 9:06 AM, Marton Balint wro

Re: [FFmpeg-devel] [PATCH] fftools/ffmpeg: move elapsed time counter to the end of the status line

2025-05-09 Thread softworkz .
> -Original Message- > From: ffmpeg-devel On Behalf Of Marton > Balint > Sent: Freitag, 9. Mai 2025 14:06 > To: ffmpeg-devel@ffmpeg.org > Cc: Marton Balint > Subject: [FFmpeg-devel] [PATCH] fftools/ffmpeg: move elapsed time counter to > the end of the status line > > Elapsed time is m

Re: [FFmpeg-devel] [PATCH] fftools/ffmpeg: move elapsed time counter to the end of the status line

2025-05-09 Thread James Almer
On 5/9/2025 9:06 AM, Marton Balint wrote: Elapsed time is much less important than the actual progress, and I am guessing several ffmpeg CLI frontends use the existing status line for showing progress, so putting a new field in the beginning is less optimal anyway. LGTM, but for the record, we

Re: [FFmpeg-devel] [PATCH] avformat/apvdec: remove unused variable

2025-05-09 Thread Leo Izen
On 5/5/25 16:31, Marvin Scholz wrote: --- libavformat/apvdec.c | 1 - 1 file changed, 1 deletion(-) diff --git a/libavformat/apvdec.c b/libavformat/apvdec.c index e53c9ed085..1e5497bcd3 100644 --- a/libavformat/apvdec.c +++ b/libavformat/apvdec.c @@ -53,7 +53,6 @@ static int apv_extract_heade

Re: [FFmpeg-devel] [PATCH v1] avcodec/apv_encoder: Updated APV encoder to set the encoder output to OAPV_CFG_VAL_AU_BS_FMT_NONE format (the only AU without bitstream format)

2025-05-09 Thread James Almer
On 5/9/2025 8:52 AM, Dawid Kozinski wrote: Signed-off-by: Dawid Kozinski --- libavcodec/liboapvenc.c | 12 1 file changed, 12 insertions(+) diff --git a/libavcodec/liboapvenc.c b/libavcodec/liboapvenc.c index 0c95489bc9..d338b33e2d 100644 --- a/libavcodec/liboapvenc.c +++ b/liba

[FFmpeg-devel] Fw: GSoC 2025: We have a few extra slots for orgs needing 1 more contributor (random drawing)

2025-05-09 Thread Michael Niedermayer
Hi all forwarding this, just in case any contributors finished their qualification task after the deadline and their mentors want them considered for this [for MENTORS]: In that case please mail Thilo and me ASAP - Forwarded message from summerofcode-nore...@google.com - Date: Thu, 08

[FFmpeg-devel] [PATCH] fftools/ffmpeg: move elapsed time counter to the end of the status line

2025-05-09 Thread Marton Balint
Elapsed time is much less important than the actual progress, and I am guessing several ffmpeg CLI frontends use the existing status line for showing progress, so putting a new field in the beginning is less optimal anyway. Related to ticket #11582. Signed-off-by: Marton Balint --- fftools/ffmp

Re: [FFmpeg-devel] [PATCH] avformat/wavdec: increase requested probe score for codec probe

2025-05-09 Thread Marton Balint
On Fri, 9 May 2025, Michael Niedermayer wrote: On Fri, May 09, 2025 at 12:28:00AM +0200, Marton Balint wrote: Codec probing was primarily added to the wav demuxer to support DTS-in-wav files, but DTS probing functions return AVPROBE_SCORE_EXTENSION+1, so we can be a bit more strict with the

[FFmpeg-devel] [PATCH v1] avcodec/apv_encoder: Updated APV encoder to set the encoder output to OAPV_CFG_VAL_AU_BS_FMT_NONE format (the only AU without bitstream format)

2025-05-09 Thread Dawid Kozinski
Signed-off-by: Dawid Kozinski --- libavcodec/liboapvenc.c | 12 1 file changed, 12 insertions(+) diff --git a/libavcodec/liboapvenc.c b/libavcodec/liboapvenc.c index 0c95489bc9..d338b33e2d 100644 --- a/libavcodec/liboapvenc.c +++ b/libavcodec/liboapvenc.c @@ -307,6 +307,18 @@ static

Re: [FFmpeg-devel] [PATCH v10 06/15] fftools/textformat: Introduce AVTextFormatOptions for avtext_context_open()

2025-05-09 Thread softworkz .
> -Original Message- > From: Stefano Sabatini > Sent: Donnerstag, 8. Mai 2025 23:39 > To: softworkz . > Cc: FFmpeg development discussions and patches > Subject: Re: [FFmpeg-devel] [PATCH v10 06/15] fftools/textformat: Introduce > AVTextFormatOptions for avtext_context_open() > > On

Re: [FFmpeg-devel] [PATCH 0/5] doc/developer: Add subsection about patch submission via FFstaging

2025-05-09 Thread softworkz .
> -Original Message- > From: ffmpeg-devel On Behalf Of Nicolas > George > Sent: Donnerstag, 8. Mai 2025 14:47 > To: FFmpeg development discussions and patches > Subject: Re: [FFmpeg-devel] [PATCH 0/5] doc/developer: Add subsection about > patch submission via FFstaging > > Lynne (HE12

[FFmpeg-devel] [PATCH v2] avfilter/vf_lut3d_opencl Initial support for OpenCL implementation of vf_lut3d.

2025-05-09 Thread Jan Studený via ffmpeg-devel
Hi, Since this is my first FFmpeg patch I’m not sure if I should include anything extra to make the review easier, like logs, test results, or more explanation. Please let me know if there’s anything I can add or do differently. Best regards, Jan Studený On May 1, 2025 at 20:13 +0300, Jan Stude

Re: [FFmpeg-devel] [PATCH 7/7] avcodec/hevc/ps: Fix dependant layer id check

2025-05-09 Thread Michael Niedermayer
On Thu, May 08, 2025 at 07:10:38PM -0300, James Almer wrote: > On 5/8/2025 6:57 PM, Michael Niedermayer wrote: > > Fixes: shift exponent 49 is too large for 32-bit type 'int' > > Fixes: > > 398060145/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_HEVC_fuzzer-5023082406543360 > > > > Found-by:

Re: [FFmpeg-devel] [PATCH] avformat/wavdec: increase requested probe score for codec probe

2025-05-09 Thread Michael Niedermayer
On Fri, May 09, 2025 at 12:28:00AM +0200, Marton Balint wrote: > Codec probing was primarily added to the wav demuxer to support DTS-in-wav > files, but DTS probing functions return AVPROBE_SCORE_EXTENSION+1, so we can > be > a bit more strict with the required score. > > This fixes MP3 misdetect

Re: [FFmpeg-devel] [PATCH 1/7] avcodec/mmvideo: fix paltte index

2025-05-09 Thread Michael Niedermayer
On Fri, May 09, 2025 at 07:06:42PM +1000, Peter Ross wrote: > On Thu, May 08, 2025 at 11:57:32PM +0200, Michael Niedermayer wrote: > > Fixes: > > 391935573/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_MMVIDEO_fuzzer-4655048979709952 > > Fixes: out of array access > > > > Found-by: continuous

Re: [FFmpeg-devel] [PATCH 6/7] avcodec/rv60dec: inter also fails with qp >= 32

2025-05-09 Thread Michael Niedermayer
On Fri, May 09, 2025 at 07:07:51PM +1000, Peter Ross wrote: > On Thu, May 08, 2025 at 11:57:37PM +0200, Michael Niedermayer wrote: > > Fixes: out of array read in decode_cu_16x16() > > Fixes: > > 398049430/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_RV60_fuzzer-5525836849807360 > > > > Foun

Re: [FFmpeg-devel] [PATCH 4/7] avformat/iff: Check nb_channels == 0 in MHDR

2025-05-09 Thread Michael Niedermayer
On Fri, May 09, 2025 at 07:07:19PM +1000, Peter Ross wrote: > On Thu, May 08, 2025 at 11:57:35PM +0200, Michael Niedermayer wrote: > > Fixes: division by 0 > > Fixes: > > 395163171/clusterfuzz-testcase-minimized-ffmpeg_dem_IFF_fuzzer-542604339373670 > > > > Found-by: continuous fuzzing process >

Re: [FFmpeg-devel] [PATCH 6/7] avcodec/rv60dec: inter also fails with qp >= 32

2025-05-09 Thread Peter Ross
On Thu, May 08, 2025 at 11:57:37PM +0200, Michael Niedermayer wrote: > Fixes: out of array read in decode_cu_16x16() > Fixes: > 398049430/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_RV60_fuzzer-5525836849807360 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/t

Re: [FFmpeg-devel] [PATCH 4/7] avformat/iff: Check nb_channels == 0 in MHDR

2025-05-09 Thread Peter Ross
On Thu, May 08, 2025 at 11:57:35PM +0200, Michael Niedermayer wrote: > Fixes: division by 0 > Fixes: > 395163171/clusterfuzz-testcase-minimized-ffmpeg_dem_IFF_fuzzer-542604339373670 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg > Signed

Re: [FFmpeg-devel] [PATCH 1/7] avcodec/mmvideo: fix paltte index

2025-05-09 Thread Peter Ross
On Thu, May 08, 2025 at 11:57:32PM +0200, Michael Niedermayer wrote: > Fixes: > 391935573/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_MMVIDEO_fuzzer-4655048979709952 > Fixes: out of array access > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/master/proje