[FFmpeg-devel] Re: [REFUND-REQUEST] VDD'25 travel expense

2025-11-07 Thread Michael Niedermayer via ffmpeg-devel
On Mon, Nov 03, 2025 at 12:07:24PM +0100, Niklas Haas via ffmpeg-devel wrote: > Hi all, > > I travelled to VDD'25 where I held two talks, one of which was directly > related to FFmpeg. I am hereby requesting reimbursement for my travel > expenses, which were: > > -

[FFmpeg-devel] [PATCH] avutil/timecode: Fix -Wformat-truncation warning (PR #20860)

2025-11-07 Thread mkver via ffmpeg-devel
PR #20860 opened by mkver URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20860 Patch URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20860.patch Using unsigned for fps more natural since the corresponding AVTimecode field is unsigned and fixes a -Wformat-truncation warning from GCC 16: in case

[FFmpeg-devel] Re: [PATCH] avcodec/rv60dec: add upper bound check for qp

2025-11-07 Thread Michael Niedermayer via ffmpeg-devel
Hi Reaxx On Fri, Nov 07, 2025 at 10:49:05PM +0100, Reaxx via ffmpeg-devel wrote: > Hi Michael, > > Yes, the commit message LGTM , Thank you for the detailed explanation and > for taking the time to review this thoroughly. I appreciate your patience. patch applied thx [...] -- Michael GnuP

[FFmpeg-devel] Re: [PATCH] avcodec/rv60dec: add upper bound check for qp

2025-11-07 Thread Reaxx via ffmpeg-devel
Hi Michael, Yes, the commit message LGTM , Thank you for the detailed explanation and for taking the time to review this thoroughly. I appreciate your patience. On Fri, 7 Nov 2025 at 22:39, Michael Niedermayer via ffmpeg-devel < [email protected]> wrote: > Hi > > On Fri, Nov 07, 2025 at 10

[FFmpeg-devel] Re: [PATCH] avcodec/rv60dec: add upper bound check for qp

2025-11-07 Thread Michael Niedermayer via ffmpeg-devel
Hi On Fri, Nov 07, 2025 at 10:19:27PM +0100, Reaxx via ffmpeg-devel wrote: > Thanks for reviewing and applying the patch. Apologies for the incorrect > commit message , I appreciate you correcting it. Thanks again actually, can you check if my commit message is correct before i apply it ? commit

[FFmpeg-devel] Re: [PATCH] avcodec/rv60dec: add upper bound check for qp

2025-11-07 Thread Reaxx via ffmpeg-devel
Thanks for reviewing and applying the patch. Apologies for the incorrect commit message , I appreciate you correcting it. Thanks again On Fri, 7 Nov 2025 at 22:14, Michael Niedermayer via ffmpeg-devel < [email protected]> wrote: > Hi Reaxx > > On Fri, Nov 07, 2025 at 06:20:38PM +0100, Reaxx

[FFmpeg-devel] Re: [PATCH] avcodec/rv60dec: add upper bound check for qp

2025-11-07 Thread Michael Niedermayer via ffmpeg-devel
Hi Reaxx On Fri, Nov 07, 2025 at 06:20:38PM +0100, Reaxx via ffmpeg-devel wrote: > This patch fixes an out-of-bounds read in the RV60 decoder where qp can > reach 65, exceeding the rv60_qp_to_idx[64] array bounds. The previous fix > (61cbcaf93f) only covered intra frames. This adds validation at t

[FFmpeg-devel] [PATCH] avcodec/rv60dec: add upper bound check for qp

2025-11-07 Thread Reaxx via ffmpeg-devel
This patch fixes an out-of-bounds read in the RV60 decoder where qp can reach 65, exceeding the rv60_qp_to_idx[64] array bounds. The previous fix (61cbcaf93f) only covered intra frames. This adds validation at the source for all frame types. 0001-avcodec-rv60dec-add-upper-bound-check-for-qp.patch

[FFmpeg-devel] Re: [PATCH] Better printing of errors on format negotiation failure (PR #20851)

2025-11-07 Thread Nicolas George via ffmpeg-devel
Niklas Haas via ffmpeg-devel (HE12025-11-06): > PR #20851 opened by Niklas Haas (haasn) > URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20851 > Patch URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20851.patch > > The current code only ever printed the pixel format; even when the > negotiatio

[FFmpeg-devel] Re: [RFC] mailing list From mangling and bounces and DMARC

2025-11-07 Thread Nicolas George via ffmpeg-devel
Pavel Roslyy via ffmpeg-devel (HE12025-11-04): > The headers that are signed for DKIM can be configured. It is not > recommended, but if needed you can remove Subject from the list of > signed headers. “Eh, Google, can I edit your DKIM settings to exclude the Subject header, so that we can add ‘[

[FFmpeg-devel] Re: [RFC] Funded Task Ideas

2025-11-07 Thread Nicolas George via ffmpeg-devel
Timo Rothenpieler via ffmpeg-devel (HE12025-11-06): > Couldn't we simply have any design-discussions and decisions before the > bounty is started, so the design becomes part of what the bounty is posted > for? Not really, I would say. To paragraph the famous saying “no battle plan survives the fir

[FFmpeg-devel] [PATCH] libavutil/timecodec: fix potential snprintf truncation in av_timecode_make_string (PR #20859)

2025-11-07 Thread caifan via ffmpeg-devel
PR #20859 opened by caifan URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20859 Patch URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20859.patch ensure formatted timecode string fits in AV_TIMECODE_STR_SIZE, eliminates -Wformat-truncation compiler warning. Signed-off-by: caifan3 >From 7301

[FFmpeg-devel] [PATCH] libavformat/vorbiscomment: fix potential buffer truncation (PR #20858)

2025-11-07 Thread caifan via ffmpeg-devel
PR #20858 opened by caifan URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20858 Patch URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20858.patch add av_assert0() for chapter_time and chapter_number, ensure buffer size is sufficient to hold formatted strings, eliminate -Wformat-truncation warni

[FFmpeg-devel] Re: [GASPP PATCH 1/2] Pass "-machine" options through to armasm

2025-11-07 Thread Rémi Denis-Courmont via ffmpeg-devel
Le 7 novembre 2025 05:51:33 GMT+02:00, Harish Raja Selvan via ffmpeg-devel a écrit : >Hi Martin, >Just wanted to check if you have an estimate for when the ARM64EC related >patches for gas-preprocessor and the makedef script changes might be >upstreamed. Not before you stop ignoring feedback

[FFmpeg-devel] Re: [GASPP PATCH 1/2] Pass "-machine" options through to armasm

2025-11-07 Thread Martin Storsjö via ffmpeg-devel
Hi, On Fri, 7 Nov 2025, Harish Raja Selvan wrote: Just wanted to check if you have an estimate for when the ARM64EC related patches for gas-preprocessor The patch for gas-preprocessor was merged already many weeks ago. (We agreed to go with my version of that patch.) and the makedef scrip

[FFmpeg-devel] [PATCH] libavformat/rtsp: fix potential buffer truncation in ff_rtsp_make_setup_request (PR #20857)

2025-11-07 Thread caifan via ffmpeg-devel
PR #20857 opened by caifan URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20857 Patch URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20857.patch GCC with -Wformat-truncation warns that writing the transport string with snprintf may result in truncation, as up to 4095 bytes from '%s' are writt