[FFmpeg-devel] [PATCH 1/1] libavformat/img2enc.c: rm warning ‘.tmp’ directive output may be truncated

2025-07-28 Thread Cai Fan
Signed-off-by: Cai Fan --- libavformat/img2enc.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavformat/img2enc.c b/libavformat/img2enc.c index 672cfe18fd..f7d983ec54 100644 --- a/libavformat/img2enc.c +++ b/libavformat/img2enc.c @@ -182,7 +182,8 @@ static int write_pac

Re: [FFmpeg-devel] [PATCH 0/2] Support the new APMP boxes and spherical metadata

2025-07-28 Thread James Almer
On 7/28/2025 10:50 PM, Vittorio Giovara wrote: From a244d96e045439f9e3a4db99e2f479e8686221da Mon Sep 17 00:00:00 2001 From: Vittorio Giovara Date: Wed, 23 Jul 2025 23:42:14 +0200 Subject: [PATCH 2/2] lavu/spherical: Add support for Spherical Immersive type --- libavformat/mov.c | 3 +++ li

Re: [FFmpeg-devel] [PATCH 0/2] Support the new APMP boxes and spherical metadata

2025-07-28 Thread Vittorio Giovara
On Thu, Jul 24, 2025 at 12:13 AM Vittorio Giovara < vittorio.giov...@gmail.com> wrote: > This patchset adds new boxes and projection values from from > https://developer.apple.com/av-foundation/Stereo-Video-ISOBMFF-Extensions.pdf > > Vittorio Giovara (2): > mov: Export frame packing information

Re: [FFmpeg-devel] forgejo PR vs git send email questions

2025-07-28 Thread James Almer
On 7/28/2025 8:13 PM, Michael Niedermayer wrote: Hi with git send email i can send 5 independant patches with git send-email --to='FFmpeg development discussions and patches ' -5 and have some CC: tags to make sure people (like bug reporter or security researcher) get notified Does our for

[FFmpeg-devel] forgejo PR vs git send email questions

2025-07-28 Thread Michael Niedermayer
Hi with git send email i can send 5 independant patches with git send-email --to='FFmpeg development discussions and patches ' -5 and have some CC: tags to make sure people (like bug reporter or security researcher) get notified Does our forgejo act on CC: ? Also with 5 independant patches,

Re: [FFmpeg-devel] [PATCH] avformat/subfile: clip seek offset

2025-07-28 Thread Michael Niedermayer
On Fri, Jul 11, 2025 at 09:49:29PM +0200, Kacper Michajłow wrote: > Fixes: signed integer overflow: 9223372036854737920 + 1649410 cannot be > represented in type 'int64_t' > > Fixes OSS-Fuzz: 410100610 > > Signed-off-by: Kacper Michajłow > --- > libavformat/subfile.c | 6 +++--- > 1 file change

Re: [FFmpeg-devel] [RFC] Advanced Error Codes

2025-07-28 Thread Nicolas George
Sorry, forgot a bit: > The first group: > > - do not get a context; > > - perform a rather simple task; - have a perimeter small enough that an error code is enough for the caller to know what went wrong; > > - could be called in tight loops, even when they fail repeatedly, and > theref

Re: [FFmpeg-devel] [RFC] Advanced Error Codes

2025-07-28 Thread Nicolas George
Michael Niedermayer (HE12025-07-06): > Hi Nicolas Hi. Sorry I procrastinated to reply to that too. > theres a fixed length array of AdvancedError structs. More global state is not a good idea. > AdvancedError has a fixed length char field. > Thats where a single custom text string can be stored

Re: [FFmpeg-devel] [PATCH v2] ogg/vorbis: implement header packet skip in chained ogg bitstreams.

2025-07-28 Thread Romain Beauxis
Le dim. 27 juil. 2025 à 19:22, Michael Niedermayer a écrit : > > Hi Romain > > On Wed, Jul 23, 2025 at 02:06:07PM -0500, Romain Beauxis wrote: > > Le sam. 21 juin 2025 à 16:59, Michael Niedermayer < mich...@niedermayer.cc> > > a écrit : > > > > > > On Sat, Jun 21, 2025 at 10:45:32AM +0200, Romain

Re: [FFmpeg-devel] [PATCH] libavfilter/vf_scale_npp: fix yuva420p -> yuv420p conversion

2025-07-28 Thread Jorge Estrada
ping On Sat, Jul 19, 2025 at 3:32 PM Jorge Estrada wrote: > When converting from yuva420p to yuv420p the resize loop would try to > access an output plane that did not exist. > Added a check for the output planes existence. > > Example cmd: ffmpeg -hwaccel cuda -hwaccel_output_format cuda -i > e

Re: [FFmpeg-devel] [PATCH] ffprobe: add -codec: option

2025-07-28 Thread Stefano Sabatini
Hi, sorry for the slow reply. On date Tuesday 2025-06-10 17:02:05 +0200, ffmpeg-devel Mailing List wrote: > From: Maryla Ustarroz-Calonge > Subject: [PATCH] ffprobe: add -codec: option > Date: Tue, 10 Jun 2025 17:02:05 +0200 > To: ffmpeg-devel@ffmpeg.org > X-Mailer: git-send-email 2.50.0.rc1.591.

Re: [FFmpeg-devel] [PATCH v3 2/2] lavc/vvc: Add max parameter to kth_order_egk_decode

2025-07-28 Thread Frank Plowman
On 25/07/2025 06:48, Nuo Mi wrote: > On Sun, Jul 20, 2025 at 1:17 AM Frank Plowman wrote: > >> Prior to this patch, kth_order_egk_decode could read arbitrarily >> large values which then overflowed and caused various issues. >> Patch fixes this by making kth_order_egk_decode falliable, >> requiri

Re: [FFmpeg-devel] forgejo labels

2025-07-28 Thread Timo Rothenpieler
On 7/28/2025 7:24 PM, Michael Niedermayer wrote: On Mon, Jul 28, 2025 at 06:15:53PM +0200, Timo Rothenpieler wrote: On 7/28/2025 4:40 PM, Diederick C. Niehorster wrote: On Mon, Jul 28, 2025 at 4:21 PM Michael Niedermayer wrote: Hi regression: issue aaddresses a regression bug:someth

Re: [FFmpeg-devel] [PATCH 3/4] avcodec/sanm: Check w, h for subversion < 2

2025-07-28 Thread Michael Niedermayer
On Thu, Jul 03, 2025 at 10:32:35PM +0200, Manuel Lauss wrote: > Servus Michael, > > On Thu, Jun 19, 2025 at 5:05 AM Michael Niedermayer > wrote: > > > > Fixes: > > 410609432/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_SANM_fuzzer-4935159201988608 > > Fixes: out of array access > > > > Foun

Re: [FFmpeg-devel] forgejo labels

2025-07-28 Thread Michael Niedermayer
On Mon, Jul 28, 2025 at 06:15:53PM +0200, Timo Rothenpieler wrote: > On 7/28/2025 4:40 PM, Diederick C. Niehorster wrote: > > On Mon, Jul 28, 2025 at 4:21 PM Michael Niedermayer > > wrote: > > > > > > Hi > > > > > > regression: issue aaddresses a regression > > > bug:something is not wor

Re: [FFmpeg-devel] forgejo labels

2025-07-28 Thread Timo Rothenpieler
On 7/28/2025 4:40 PM, Diederick C. Niehorster wrote: On Mon, Jul 28, 2025 at 4:21 PM Michael Niedermayer wrote: Hi regression: issue aaddresses a regression bug:something is not working This is inconsistent Label X cannot sometimes mean "X is removed" and "X is added" bug seems mean

Re: [FFmpeg-devel] [PATCH 1/3] avcodec/shorten: Clear the additionally allocated space on realloc

2025-07-28 Thread Michael Niedermayer
On Thu, Jul 24, 2025 at 06:40:17PM +0200, Michael Niedermayer wrote: > Fixes: use of uninitialized memory > Fixes: > 421954767/clusterfuzz-testcase-minimized-ffmpeg_AV_CODEC_ID_SHORTEN_fuzzer-515682786246656 > > Found-by: continuous fuzzing process > https://github.com/google/oss-fuzz/tree/maste

Re: [FFmpeg-devel] [PATCH 3/4] avformat/mxfdec: Ensure klv->key is initialized

2025-07-28 Thread Michael Niedermayer
On Thu, Jul 24, 2025 at 08:25:35PM +0200, Tomas Härdin wrote: > tor 2025-07-24 klockan 00:51 +0200 skrev Marton Balint: > > > > > > On Wed, 23 Jul 2025, Tomas Härdin wrote: > > > > > ons 2025-07-23 klockan 13:45 +0200 skrev Michael Niedermayer: > > > > Fixes: read of uninitialized memory > > > >

Re: [FFmpeg-devel] [PATCH] avformat/asfdec_f: Check amount of value read

2025-07-28 Thread Michael Niedermayer
On Mon, Jul 28, 2025 at 01:10:57AM +0200, Kacper Michajlow wrote: > On Sun, 27 Jul 2025 at 22:57, Michael Niedermayer > wrote: > > > > On Wed, Jul 23, 2025 at 05:51:35PM +0200, Kacper Michajlow wrote: > > > On Wed, 23 Jul 2025 at 10:43, Michael Niedermayer > > > wrote: > > > > > > > > Fixes: use

Re: [FFmpeg-devel] [PATCH 1/2] avcodec/alsdec: Allow the user to set a maximum prediction order

2025-07-28 Thread Michael Niedermayer
On Tue, Jul 22, 2025 at 03:48:38AM +0200, Michael Niedermayer wrote: > The only offical profile i could find (ALS simple profile) has a max of 15 > while the bitstream allows 1023 > which is very slow. > > We do have a fate sample with 1023 > > Fixes: Timeout > Fixes: > 429645375/clusterfuzz-te

Re: [FFmpeg-devel] [PATCH] avformat/hlsenc: limit segments count to search duplicated segments filenames

2025-07-28 Thread Michael Niedermayer
On Sat, Jul 26, 2025 at 07:09:54PM +0300, Artem Smorodin wrote: > This linear search has a complexity of O(n). When ffmpeg attempts to parse a > playlist containing approximately 100,000 segments, it effectively causes a > hang for several minutes. > > This patch limits the allowed size for duplic

Re: [FFmpeg-devel] [PATCH 1/1] avformat/movenc: fix HEVC fmp4 HLS init segment for Apple playback

2025-07-28 Thread Michael Niedermayer
On Mon, Jul 21, 2025 at 05:01:47PM -0400, David McElroy wrote: > HEVC fmp4 HLS video produced by ffmpeg is currently unplayable on Apple > software (Safari, QuickTime, AVFoundation). > > This is caused by an empty sdtp atom being erroneously written to the > fmp4 init segment. The `has_disposable`

Re: [FFmpeg-devel] [PATCH] avcodec/apv: align APV color format support with latest liboapv version

2025-07-28 Thread Michael Niedermayer
On Wed, Jul 23, 2025 at 01:13:49PM +0200, Dariusz Frankiewicz wrote: > Signed-off-by: Dariusz Frankiewicz > --- > configure | 2 +- > libavcodec/liboapvenc.c | 24 > 2 files changed, 25 insertions(+), 1 deletion(-) > > diff --git a/configure b/configure >

Re: [FFmpeg-devel] [PATCH] vulkan_encode_h264/5: Fix uninitialized return value in write_extra_headers

2025-07-28 Thread David Rosca
On Thu, Nov 14, 2024 at 10:39 PM David Rosca wrote: > > --- > libavcodec/vulkan_encode_h264.c | 1 + > libavcodec/vulkan_encode_h265.c | 1 + > 2 files changed, 2 insertions(+) > > diff --git a/libavcodec/vulkan_encode_h264.c b/libavcodec/vulkan_encode_h264.c > index cdc87fb4ca..fdd3cc8a55 100644

Re: [FFmpeg-devel] forgejo labels

2025-07-28 Thread Diederick C. Niehorster
On Mon, Jul 28, 2025 at 4:21 PM Michael Niedermayer wrote: > > Hi > > regression: issue aaddresses a regression > bug:something is not working > > This is inconsistent > Label X cannot sometimes mean "X is removed" and "X is added" > > bug seems meaning that the PR adds a bug > regression

[FFmpeg-devel] forgejo labels

2025-07-28 Thread Michael Niedermayer
Hi regression: issue aaddresses a regression bug:something is not working This is inconsistent Label X cannot sometimes mean "X is removed" and "X is added" bug seems meaning that the PR adds a bug regression seems meaning that the PR removes a regression IMO this should be done consist

Re: [FFmpeg-devel] Again pre-multiplied alpha

2025-07-28 Thread Nicolas George
Niklas Haas (HE12025-07-24): > On what component are you missing an error here? Recently I wrote: “stacking images with different kind of alpha or sending this kind of frames to a muxer with uncoded frames” So: at least filters and muxers with uncoded frames. The protection must of course go in t

Re: [FFmpeg-devel] Again pre-multiplied alpha

2025-07-28 Thread Nicolas George
Michael Niedermayer (HE12025-07-28): > About STF, it depends on who does the work. > If a FFmpeg developer does the work, then i think STF is a good idea. > OTOH It would be an inappropriate task for a new contributor (like in GSoC). Indeed, way too hard for somebody who does not already know liba

Re: [FFmpeg-devel] [PATCH] checkasm/h264dsp: Fix stack-buffer-overflow, effective-type violations

2025-07-28 Thread Michael Niedermayer
Hi Marvin On Fri, Jul 11, 2025 at 05:34:48PM +0200, Marvin Scholz wrote: > > > On 25 Jun 2025, at 23:39, Tristan Matthews via ffmpeg-devel wrote: > > > On Monday, June 16th, 2025 at 2:22 PM, Andreas Rheinhardt > > wrote: > > > > Tristan Matthews via ffmpeg-devel: > > > > On Monday, June 16th,

Re: [FFmpeg-devel] [PATCH 1/1] libavformat/img2enc.c: rm warning ‘.tmp’ directive output may be truncated

2025-07-28 Thread Steven Liu
Cai Fan 于2025年7月28日周一 11:45写道: > > Signed-off-by: Cai Fan > --- > libavformat/img2enc.c | 4 > 1 file changed, 4 insertions(+) > > diff --git a/libavformat/img2enc.c b/libavformat/img2enc.c > index 41638d92b8..672cfe18fd 100644 > --- a/libavformat/img2enc.c > +++ b/libavformat/img2enc.c > @