[FFmpeg-devel] [PATCH] avfilter: add inverse tone mapping filter

2025-06-30 Thread Sarthak Indurkhya via ffmpeg-devel
--- Begin Message --- Hello FFmpeg developers, This patch introduces a new video filter called inversetonemap for FFmpeg. The filter performs SDR to HDR conversion by mapping SDR BT.709 video to HDR BT.2020 PQ, using local adaptation and inverse tone mapping. The goal is to provide a simple, flex

Re: [FFmpeg-devel] [PATCH] avformat/mpegts: Add AVFMT_FLAG_ALLOW_CODEC_CHANGES (v2)

2025-06-30 Thread Pavel Koshevoy
On Mon, Jun 30, 2025, 21:46 Michael Niedermayer wrote: > On Mon, Jun 30, 2025 at 07:32:36PM -0600, Pavel Koshevoy wrote: > > On Mon, Jun 30, 2025 at 3:46 PM Scott Theisen < > > scott.the.elm-at-gmail@ffmpeg.org> wrote: > > > > > On 2025/06/29 23:55, Pavel Koshevoy wrote: > > > > Make midstrea

Re: [FFmpeg-devel] [PATCH] avformat/mpegts: Add AVFMT_FLAG_ALLOW_CODEC_CHANGES (v2)

2025-06-30 Thread Michael Niedermayer
On Mon, Jun 30, 2025 at 07:32:36PM -0600, Pavel Koshevoy wrote: > On Mon, Jun 30, 2025 at 3:46 PM Scott Theisen < > scott.the.elm-at-gmail@ffmpeg.org> wrote: > > > On 2025/06/29 23:55, Pavel Koshevoy wrote: > > > Make midstream AVStream.codecpar updates optional and disabled > > > by default,

Re: [FFmpeg-devel] [PATCH 5/5] avformat/rtpdec_asf: fix leak in ff_wms_parse_sdp_a_line()

2025-06-30 Thread Michael Niedermayer
On Sun, Jun 29, 2025 at 02:45:26PM +0800, Lidong Yan wrote: > In ff_wms_parse_sdp_a_line(), it allocates memory in buf, but doesn't > free buf when avformat_alloc_context() failed. Add av_free(buf) before > return to prevent from leak. > > Signed-off-by: Lidong Yan <502024330...@smail.nju.edu.cn>

Re: [FFmpeg-devel] [PATCH] avformat/mpegts: Add AVFMT_FLAG_ALLOW_CODEC_CHANGES (v2)

2025-06-30 Thread Pavel Koshevoy
On Mon, Jun 30, 2025 at 3:46 PM Scott Theisen < scott.the.elm-at-gmail@ffmpeg.org> wrote: > On 2025/06/29 23:55, Pavel Koshevoy wrote: > > Make midstream AVStream.codecpar updates optional and disabled > > by default, so that avformat API clients can enable this feature > > explicitly when the

Re: [FFmpeg-devel] [PATCH] avformat/mpegts: Add AVFMT_FLAG_ALLOW_CODEC_CHANGES (v2)

2025-06-30 Thread Scott Theisen
On 2025/06/29 23:55, Pavel Koshevoy wrote: Make midstream AVStream.codecpar updates optional and disabled by default, so that avformat API clients can enable this feature explicitly when they add support for midstream codec changes. Is this signaled in any way or is the client expected to check

Re: [FFmpeg-devel] [PATCH] aacenc_tns: clamp filter direction energy measurement

2025-06-30 Thread Thierry Foucu
On Fri, Feb 7, 2025 at 7:36 PM Lynne wrote: > The issue is that: > > float en[2]; > ... > tns->n_filt[w] = is8 ? 1 : order != TNS_MAX_ORDER ? 2 : 3; > for (g = 0; g < tns->n_filt[w]; g++) { > tns->direction[w][g] = slant != 2 ? slant : en[g] < en[!g]; > > When using the AAC Main profile, n_fi

Re: [FFmpeg-devel] [EXTERNE] Re: [PATCH] avcodec/aac_parser: fix aac profile probing

2025-06-30 Thread Baptiste Coudurier
> On Jun 30, 2025, at 11:08 AM, Nicolas Gaullier > wrote: > > On 6/28/25 03:54, Baptiste Coudurier wrote: >>> On Jun 27, 2025, at 5:01 AM, Nicolas Gaullier >>> wrote: >>> >>> On 6/26/25 19:30, Baptiste Coudurier wrote: Hi Nicolas > On Jun 26, 2025, at 4:06 AM, Nicolas Gaullier

[FFmpeg-devel] [PATCH] fftools/cmdutils: don't try to load arguments from file if not needed

2025-06-30 Thread Kacper Michajłow
CLI option parser checks if argument exists when needed, but in this case only OPT_TYPE_BOOL where checked, so OPT_TYPE_FUNC without argument where trying to load a file from `arg` which is NULL in this case. Fixes crash on `ffmpeg -/version` Fixes: 6d17991b7e1bf1a5d104c8a6261709f7e6640d97 Signed

Re: [FFmpeg-devel] [EXTERNE] Re: [PATCH] avcodec/aac_parser: fix aac profile probing

2025-06-30 Thread Nicolas Gaullier
On 6/28/25 03:54, Baptiste Coudurier wrote: On Jun 27, 2025, at 5:01 AM, Nicolas Gaullier wrote: On 6/26/25 19:30, Baptiste Coudurier wrote: Hi Nicolas On Jun 26, 2025, at 4:06 AM, Nicolas Gaullier wrote: On 6/17/25 13:41, Nicolas Gaullier wrote: Reverts 64bb91fd3b5a00a8849531c7e8dd207f2

[FFmpeg-devel] [PATCH v3 21/21] fate/seek: fix missing mpeg4video parser dependency

2025-06-30 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/seek.mak | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tests/fate/seek.mak b/tests/fate/seek.mak index aa1f856f7b..5504e331e6 100644 --- a/tests/fate/seek.mak +++ b/tests/fate/seek.mak @@ -62,7 +62,6 @@ FATE_SEEK_VSYNTH_LENA +

[FFmpeg-devel] [PATCH v3 20/21] fate/all: fix missing some mov demuxer dependencies

2025-06-30 Thread Nicolas Gaullier
In aac/aac-fixed, also remove unnecessary aac demuxer dependency. Signed-off-by: Nicolas Gaullier --- tests/fate/aac.mak| 2 +- tests/fate/h264.mak | 2 +- tests/fate/lavf-container.mak | 8 tests/fate/mov.mak| 14 +++--- 4 files changed, 13

[FFmpeg-devel] [PATCH v3 19/21] fate/all: add missing crc/framecrc/md5/framemd5/pipe dependencies

2025-06-30 Thread Nicolas Gaullier
When CMD is crc/framecrc, always use the macros CRC/FRAMECRC, even if it includes unnecessary requirements for rawvideo/pcm_s16le encoders (as actually noticed in a comment of the Makefile). Signed-off-by: Nicolas Gaullier --- tests/Makefile| 3 ++ tests/fate/aac.mak

[FFmpeg-devel] [PATCH v3 13/21] fate/hlsenc: fix multiple dependencies

2025-06-30 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/hlsenc.mak | 18 +- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/tests/fate/hlsenc.mak b/tests/fate/hlsenc.mak index 98341b2b81..72c75bec3a 100644 --- a/tests/fate/hlsenc.mak +++ b/tests/fate/hlsenc.mak @@ -5,7 +5,7 @

[FFmpeg-devel] [PATCH v3 18/21] fate/all: add missing file protocol dependencies

2025-06-30 Thread Nicolas Gaullier
First, always require file protocol when FATE suite is used. Then, add missing dependencies while removing duplicates. Signed-off-by: Nicolas Gaullier --- tests/Makefile | 1 + tests/fate/cbs.mak | 4 ++-- tests/fate/cover-art.mak| 8 tests/fate/demux.mak

[FFmpeg-devel] [PATCH v3 08/21] fate/vorbis: fix multiple dependencies

2025-06-30 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/vorbis.mak | 24 +--- 1 file changed, 13 insertions(+), 11 deletions(-) diff --git a/tests/fate/vorbis.mak b/tests/fate/vorbis.mak index 781430bcd3..73e837f5a6 100644 --- a/tests/fate/vorbis.mak +++ b/tests/fate/vorbis.mak @@ -1,

[FFmpeg-devel] [PATCH v3 17/21] fate/all: fix multiple dependencies

2025-06-30 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/api.mak | 2 +- tests/fate/cbs.mak | 2 +- tests/fate/ffmpeg.mak | 6 +++--- tests/fate/filter-audio.mak | 8 tests/fate/filter-video.mak | 2 +- tests/fate/fits.mak | 4 ++-- tests/fate/gif.mak | 2 +

[FFmpeg-devel] [PATCH v3 16/21] fate/hevc: fix multiple dependencies

2025-06-30 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/hevc.mak | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tests/fate/hevc.mak b/tests/fate/hevc.mak index 3ad6e255a9..7b22e36a3d 100644 --- a/tests/fate/hevc.mak +++ b/tests/fate/hevc.mak @@ -219,7 +219,7 @@ FATE_HEVC-$(cal

[FFmpeg-devel] [PATCH v3 15/21] fate/pcm: fix multiple dependencies

2025-06-30 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/pcm.mak | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/tests/fate/pcm.mak b/tests/fate/pcm.mak index b10233a9eb..75636118b2 100644 --- a/tests/fate/pcm.mak +++ b/tests/fate/pcm.mak @@ -1,4 +1,4 @@ -FATE_SAMPLES_PCM-$

[FFmpeg-devel] [PATCH v3 14/21] fate/mpeg4: fix multiple dependencies

2025-06-30 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/mpeg4.mak | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/tests/fate/mpeg4.mak b/tests/fate/mpeg4.mak index 9a3cd4e2fb..d6cce97a70 100644 --- a/tests/fate/mpeg4.mak +++ b/tests/fate/mpeg4.mak @@ -3,23 +3,22 @@ MPEG4_RE

[FFmpeg-devel] [PATCH v3 11/21] fate/ac3: fix multiple dependencies

2025-06-30 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/ac3.mak | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/tests/fate/ac3.mak b/tests/fate/ac3.mak index e52678a2fd..30f8b98ff1 100644 --- a/tests/fate/ac3.mak +++ b/tests/fate/ac3.mak @@ -65,17 +65,17 @@ fate-eac3-5: REF

[FFmpeg-devel] [PATCH v3 12/21] fate/cover-art: fix multiple dependencies

2025-06-30 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/cover-art.mak | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/tests/fate/cover-art.mak b/tests/fate/cover-art.mak index faa2182745..7ee9443864 100644 --- a/tests/fate/cover-art.mak +++ b/tests/fate/cover-art.mak @

[FFmpeg-devel] [PATCH v3 09/21] fate/aac: fix multiple dependencies

2025-06-30 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/aac.mak | 14 +++--- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/tests/fate/aac.mak b/tests/fate/aac.mak index f5a41458e6..48a4410d6c 100644 --- a/tests/fate/aac.mak +++ b/tests/fate/aac.mak @@ -241,21 +241,21 @@ fate-aac-la

[FFmpeg-devel] [PATCH v3 10/21] fate/audio: fix multiple dependencies

2025-06-30 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/audio.mak | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/tests/fate/audio.mak b/tests/fate/audio.mak index 6e091760ea..6d3cae8f52 100644 --- a/tests/fate/audio.mak +++ b/tests/fate/audio.mak @@ -5,12 +5,12 @@ FATE_SA

[FFmpeg-devel] [PATCH v3 06/21] fate/gapless: fix multiple dependencies

2025-06-30 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/gapless.mak | 76 +- 1 file changed, 38 insertions(+), 38 deletions(-) diff --git a/tests/fate/gapless.mak b/tests/fate/gapless.mak index dfae964662..e6fba1436b 100644 --- a/tests/fate/gapless.mak +++ b/tests/

[FFmpeg-devel] [PATCH v3 07/21] fate/lavf-container: fix multiple dependencies

2025-06-30 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/lavf-container.mak | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/fate/lavf-container.mak b/tests/fate/lavf-container.mak index 6a46f98b5d..395b97b492 100644 --- a/tests/fate/lavf-container.mak +++ b/tests/fate/lavf-c

[FFmpeg-devel] [PATCH v3 05/21] fate/mov: fix multiple dependencies

2025-06-30 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/mov.mak | 38 ++ 1 file changed, 22 insertions(+), 16 deletions(-) diff --git a/tests/fate/mov.mak b/tests/fate/mov.mak index 73deb210cd..8ad8d26144 100644 --- a/tests/fate/mov.mak +++ b/tests/fate/mov.mak @@ -1,4

[FFmpeg-devel] [PATCH v3 04/21] fate/all: switch-fix mov muxer dependency to mp4 muxer dependency

2025-06-30 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/enc_external.mak | 6 +++--- tests/fate/lavf-container.mak | 2 +- tests/fate/mov.mak| 14 +++--- tests/fate/subtitles.mak | 2 +- 4 files changed, 12 insertions(+), 12 deletions(-) diff --git a/tests/fate/enc_external.

[FFmpeg-devel] [PATCH v3 03/21] fate/demux: fix multiple dependencies

2025-06-30 Thread Nicolas Gaullier
Signed-off-by: Nicolas Gaullier --- tests/fate/demux.mak | 46 ++-- 1 file changed, 23 insertions(+), 23 deletions(-) diff --git a/tests/fate/demux.mak b/tests/fate/demux.mak index a654cdcbbf..0f87abadad 100644 --- a/tests/fate/demux.mak +++ b/tests/fate/d

[FFmpeg-devel] [PATCH v3 02/21] fate/all: add missing dependencies for extradata bsf

2025-06-30 Thread Nicolas Gaullier
--- tests/fate/concatdec.mak | 2 +- tests/fate/demux.mak | 16 tests/fate/flvenc.mak | 2 +- tests/fate/h264.mak | 8 +--- tests/fate/hevc.mak | 4 ++-- tests/fate/lavf-container.mak | 8 tests/fate/segment.mak|

[FFmpeg-devel] [PATCH v3 01/21] tests/Makefile: make easier to check for multiple dependencies

2025-06-30 Thread Nicolas Gaullier
--- tests/Makefile | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/tests/Makefile b/tests/Makefile index 505d7f9c6d..7187b744cb 100644 --- a/tests/Makefile +++ b/tests/Makefile @@ -97,21 +97,21 @@ TRANSCODE = $(call ALLYES, $(firstword $(1))_ENCODER $(lastword

[FFmpeg-devel] [PATCH v3 00/21] fate: fix several dependencies

2025-06-30 Thread Nicolas Gaullier
Since v2: - rebased to current master - removed "libavcodec/tests: fix gitignore hashtable" which was applied recently - added two new patches at the end of the serie Nicolas Gaullier (21): tests/Makefile: make easier to check for multiple dependencies fate/all: add missing dependencies for ex

Re: [FFmpeg-devel] [EXTERNE] Re: [PATCH v2 00/20] fate: fix several dependencies

2025-06-30 Thread Nicolas Gaullier
On 6/27/25 23:25, Michael Niedermayer wrote: Hi On Mon, Jun 23, 2025 at 07:41:32PM +0200, Nicolas Gaullier wrote: For remembering https://ffmpeg.org/pipermail/ffmpeg-devel/2024-October/335422.html This v2 is a "full rebase", which means some problems have been fixed in the while (not many), so

[FFmpeg-devel] [PATCH v4 3/3] avformat/sapenc: reword fail to cleanup in sap_write_header()

2025-06-30 Thread Lidong Yan
In sap_write_header(), normal execution would fall through to fail labeled code, thus cleanup would be a better name compared to fail. Replace the use of fail label with cleanup label. Signed-off-by: Lidong Yan <502024330...@smail.nju.edu.cn> --- libavformat/sapenc.c | 26 +---

[FFmpeg-devel] [PATCH v4 2/3] avformat/sapenc: fix leak in sap_write_header()

2025-06-30 Thread Lidong Yan
In sap_write_header(), ff_format_set_url() assign new allocated new_url to contexts[i]->url but forgot to free it later. Add for loop to free contexts[i]->url before av_free(context). To prevent from writing free-for-loop in every return point, replace `return 0` with `ret = 0` so normal execution

[FFmpeg-devel] [PATCH v4 1/3] avfilter/asrc_sinc: fix leak in config_input()

2025-06-30 Thread Lidong Yan
In config_input(), fir_to_phase() allocates memory in h[longer], which would leak if av_calloc() to s->coeffs failed. lpf() allocates memory in h[0] and h[1], which would leak if fir_to_phase() failed. To fix this leak, add av_free(h[longer]) in as cleanup code, and replace return AVERROR* with got

[FFmpeg-devel] [PATCH v4 0/3] fix leak in avfilter/asrc_sinc and avformat/sapenc

2025-06-30 Thread Lidong Yan
This patch series fixes memory leaks in `avfilter/asrc_sinc` and `avformat/sapenc`. -BEGIN PGP PUBLIC KEY BLOCK- mDMEaEpkmRYJKwYBBAHaRw8BAQdAGwGqH/Dwod+i6kR0/Rhn5GanJ7wK8mM9tWP/ W2qu8Ti0HTUwMjAyNDMzMDA1NkBzbWFpbC5uanUuZWR1LmNuiJkEExYKAEEWIQQC zskBcOehk1y8GoKZR31bPD+6owUCaEpkmQIbAwUJBaOagA

[FFmpeg-devel] X Window Capture using FFmpeg

2025-06-30 Thread novaTopFlex via ffmpeg-devel
--- Begin Message --- I have just reported an incident where I had attempted to utilize ffmpeg​ with the specific purpose of X window capture. As of now, X window capture is not currently supported using FFmpeg. So far, the latest command that I had attempted for an X window capture was the foll

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

2025-06-30 Thread Jan Studený via ffmpeg-devel
--- Begin Message --- Hello FFmpeg maintainers, I submitted a patch nearly two months ago that adds OpenCL‐based acceleration to the lut3d filter (see below for details). Version 2 haven’t received any feedback, so I’m sending this reminder as I’d really appreciate someone taking a look at it.

Re: [FFmpeg-devel] [PATCH] avfilter: add inverse tone mapping filter

2025-06-30 Thread Kacper Michajlow
On Mon, 30 Jun 2025 at 11:45, Sarthak Indurkhya via ffmpeg-devel wrote: > > > > > -- Forwarded message -- > From: Sarthak Indurkhya > To: "ffmpeg-devel@ffmpeg.org" > Cc: > Bcc: > Date: Mon, 30 Jun 2025 09:44:54 + > Subject: [PATCH] avfilter: add inverse tone mapping filter >

[FFmpeg-devel] [PATCH] avfilter: add inverse tone mapping filter

2025-06-30 Thread Sarthak Indurkhya via ffmpeg-devel
--- Begin Message --- Hello FFmpeg developers, This patch introduces a new video filter called inversetonemap for FFmpeg. The filter performs SDR to HDR conversion by mapping SDR BT.709 video to HDR BT.2020 PQ, using local adaptation and inverse tone mapping. The goal is to provide a simple, flex

Re: [FFmpeg-devel] [PATCH] Revert "Add FUNDING.json"

2025-06-30 Thread Nicolas George
Michael Niedermayer (HE12025-06-24): > Over the years one learns that with some people the value for the community > is higher to just ignore them You are absolutely right on this, but you need to push this to its logical conclusion: it is not allowed to ignore somebody in a democracy, you can dec