Re: [FFmpeg-devel] SDR lib comments

2023-09-29 Thread Nicolas George
Rémi Denis-Courmont (12023-09-29): > And as a downstream OSS application developer, I'd really rather > postproc be officially and properly dropped, so I can justify dropping > it downstream as well. Let me get this straight? You COULD just add --disable-postproc in your downstream project and be

[FFmpeg-devel] [PATCH] avdevice/v4l2: Switch to wrapped AVFrames and implement strides

2023-09-29 Thread Asahi Lina via ffmpeg-devel
ID_H264) { +if (codec_id == AV_CODEC_ID_H264) { avpriv_stream_set_need_parsing(st, AVSTREAM_PARSE_FULL_ONCE); } -if (desired_format == V4L2_PIX_FMT_YVU420) -st->codecpar->codec_tag = MKTAG('Y', 'V', '1', '2'); -else if (d

Re: [FFmpeg-devel] [PATCH] avdevice/v4l2: Switch to wrapped AVFrames and implement strides

2023-09-29 Thread Ridley Combs via ffmpeg-devel
; > sh; > +} > + > +/* Convert since the types are mismatched... */ > +for (int i = 0; i < 4; i++) > +linesize[i] = s->linesize[i]; > + > +res = av_image_fill_plane_sizes(sizes, s->pix_fmt, s->height, > linesize); > +

Re: [FFmpeg-devel] SDR lib comments

2023-09-29 Thread Rémi Denis-Courmont
Le 29 septembre 2023 10:03:13 GMT+03:00, Nicolas George a écrit : >Rémi Denis-Courmont (12023-09-29): >> And as a downstream OSS application developer, I'd really rather >> postproc be officially and properly dropped, so I can justify dropping >> it downstream as well. > >Let me get this straig

[FFmpeg-devel] [PATCH v1] avcodec/hevcpred_template: Removed unreachable code

2023-09-29 Thread Dawid Kozinski
Signed-off-by: Dawid Kozinski --- libavcodec/hevcpred_template.c | 11 --- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/libavcodec/hevcpred_template.c b/libavcodec/hevcpred_template.c index 16d1c7f35f..46bd806523 100644 --- a/libavcodec/hevcpred_template.c +++ b/libavcode

Re: [FFmpeg-devel] SDR lib comments

2023-09-29 Thread Rémi Denis-Courmont
Le 29 septembre 2023 10:03:13 GMT+03:00, Nicolas George a écrit : >Well, I think this makes it obvious why I think people representing >downstream projects should have very little influence on the future of >FFmpeg. First, I don't *represent* any downstream. I just happen to be one of several

Re: [FFmpeg-devel] SDR lib comments

2023-09-29 Thread Anton Khirnov
Quoting Michael Niedermayer (2023-09-28 21:28:35) > On Thu, Sep 28, 2023 at 12:28:22PM +0200, Anton Khirnov wrote: > > Quoting Michael Niedermayer (2023-09-23 18:43:44) > > > Is a seperate library really the right choice for SDR in FFmpeg at this > > > stage ? > > > > > > Also code can always be

Re: [FFmpeg-devel] [PATCH v2] lavc/libaribcaption.c: add MSZ characters related options

2023-09-29 Thread TADANO Tokumei
On 2023/09/23 11:15, TADANO Tokumei wrote: This is updated patch to "[PATCH] lavc/libaribcaption.c: add -replace_fullwidth_japanese option" (Message-Id: <20230908130050.85688-1-aiming...@pc.nifty.jp>). If specified fonts contain half-width glyphs, it make better rendering with `-replace_msz_as

Re: [FFmpeg-devel] [PATCH] avdevice/v4l2: Switch to wrapped AVFrames and implement strides

2023-09-29 Thread Andreas Rheinhardt
Asahi Lina via ffmpeg-devel: > V4L2 provides a line stride to the client for hardware that has > alignment requirements. rawvideo cannot represent this, so switch to > wrapped_avframe for raw video formats and calculate the plane strides > manually. > > This is slightly messy because the existing

[FFmpeg-devel] [PATCH] x86inc: Add REPX macro to repeat instructions/operations

2023-09-29 Thread Frank Plowman
From: Henrik Gramner When operating on large blocks of data it's common to repeatedly use an instruction on multiple registers. Using the REPX macro makes it easy to quickly write dense code to achieve this without having to explicitly duplicate the same instruction over and over. For example,

Re: [FFmpeg-devel] SDR lib comments

2023-09-29 Thread Nicolas George
Anton Khirnov (12023-09-29): > What does anybody gain from it being in our source tree, as opposed to a > separate library? Not having to maintain yet another packaging and build system. Not having an external dependency for something that could be considered a core feature. -- Nicolas George

[FFmpeg-devel] [PATCH] libavutil/hwcontext_cuda: fix bug in cuda_device_uninit

2023-09-29 Thread Roman Arzumanyan
Hello, In my previous patch which introduced the AV_CUDA_USE_CURRENT_CONTEXT feature I unintentionally made a mistake of not checking this flag when uninitializing the CUDA. It was causing cuda_device_uninit() function to destroy the CUDA context it didn't create. So far it only reproduces under

Re: [FFmpeg-devel] SDR lib comments

2023-09-29 Thread Vittorio Giovara
On Fri, Sep 29, 2023 at 3:03 AM Nicolas George wrote: > Well, I think this makes it obvious why I think people representing > downstream projects should have very little influence on the future of > FFmpeg. > In a previous thread you said SDR shouldn't be a separate library because it wastes dow

Re: [FFmpeg-devel] SDR lib comments

2023-09-29 Thread Rémi Denis-Courmont
Le 29 septembre 2023 14:43:11 GMT+03:00, Nicolas George a écrit : >Anton Khirnov (12023-09-29): >> What does anybody gain from it being in our source tree, as opposed to a >> separate library? > >Not having to maintain yet another packaging and build system. Talk about flimsy "scraps of the bo

Re: [FFmpeg-devel] SDR lib comments

2023-09-29 Thread Nicolas George
Rémi Denis-Courmont (12023-09-29): > A core feature that depends on niche hardware libpostproc does not depend on niche hardware. If the discussion bores you to the point of not remembering what is being discussed, maybe just dropping it and letting those who care spend their time. -- Nicolas

[FFmpeg-devel] [PATCH] configure: disable vulkan if min version insufficient

2023-09-29 Thread Tristan Matthews
Fixes: https://trac.ffmpeg.org/ticket/10596 --- configure | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/configure b/configure index 20db1801ed..50ba6f772f 100755 --- a/configure +++ b/configure @@ -7154,7 +7154,8 @@ enabled crystalhd && check_lib crystalhd "stdint.h libcry

[FFmpeg-devel] [PATCH] avformat/isom_tags: add missing CFHD tag

2023-09-29 Thread Paul B Mahol
Attached. From 1e9495ecc84dd3744e9e223e5001ea847baa7fd5 Mon Sep 17 00:00:00 2001 From: Paul B Mahol Date: Fri, 29 Sep 2023 16:38:09 +0200 Subject: [PATCH] avformat/isom_tags: add missing CFHD tag Signed-off-by: Paul B Mahol --- libavformat/isom_tags.c | 2 ++ 1 file changed, 2 insertions(+) di

Re: [FFmpeg-devel] [PATCH] avformat/isom_tags: add missing CFHD tag

2023-09-29 Thread Kieran Kunhya
On Fri, 29 Sept 2023 at 15:46, Paul B Mahol wrote: > Attached. ok ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with s

[FFmpeg-devel] [PATCH] avformat/hls: Re-use crypto URLContext

2023-09-29 Thread Luis Scheurenbrand via ffmpeg-devel
Reset crypto state and keep nested http context alive. Previously, an unencrypted file, followed by an encrypted file would result in a corrupted stream, as the stream would not be closed and an encrypted file would be forwarded to the demuxer. fixes: https://trac.ffmpeg.org/ticket/10599 fixes: h

[FFmpeg-devel] [PATCH 1/2] avformat/matroskaenc: Don't create wrong packet durations

2023-09-29 Thread Andreas Rheinhardt
We have to write an explicit BlockDuration element (and use a BlockGroup instead of a SimpleBlock) in case the Track has a DefaultDuration that is inconsistent with the duration of the packet. The matroska-h264-remux test uses a file with coded fields where the duration of a Block is the duration

[FFmpeg-devel] [PATCH 2/2] avformat/matroskaenc: Write default duration for audio

2023-09-29 Thread Andreas Rheinhardt
This is easily possible for those codecs with a fixed frame-size (in samples). Signed-off-by: Andreas Rheinhardt --- btw: Presumably AVCodecParameters.framerate should be preferred over AVStream.avg_frame_rate if set? libavformat/matroskaenc.c | 7 tests/ref/fate/matroska-

Re: [FFmpeg-devel] [PATCH 1/2] avformat/matroskaenc: Don't create wrong packet durations

2023-09-29 Thread James Almer
On 9/29/2023 1:01 PM, Andreas Rheinhardt wrote: We have to write an explicit BlockDuration element (and use a BlockGroup instead of a SimpleBlock) in case the Track has a DefaultDuration that is inconsistent with the duration of the packet. The matroska-h264-remux test uses a file with coded fie

Re: [FFmpeg-devel] [PATCH] avcodec/ffv1*: add more pixel formats support

2023-09-29 Thread Michael Niedermayer
On Thu, Sep 28, 2023 at 09:54:04PM +0200, Paul B Mahol wrote: > Attached. > ffv1dec.c | 11 +++ > ffv1enc.c |9 +++-- > 2 files changed, 18 insertions(+), 2 deletions(-) > 4e3e427b7ea8b73dd00628f317f0f5a96e9e5592 > 0001-avcodec-ffv1-add-GBRAP14-GRAY14-YUVA422P12-YUVA444P1.patc

Re: [FFmpeg-devel] [PATCH] avformat/hls: Re-use crypto URLContext

2023-09-29 Thread Michael Niedermayer
On Fri, Sep 29, 2023 at 05:10:18PM +0200, Luis Scheurenbrand via ffmpeg-devel wrote: > Reset crypto state and keep nested http context alive. > Previously, an unencrypted file, followed by an encrypted file would > result in a corrupted stream, as the stream would not be closed and > an encrypted

[FFmpeg-devel] [PATCH 1/5] lavc/aacpsdsp: simplify R-V V stereo interpolate

2023-09-29 Thread Rémi Denis-Courmont
Remove some useless vector splat. --- libavcodec/riscv/aacpsdsp_rvv.S | 12 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/libavcodec/riscv/aacpsdsp_rvv.S b/libavcodec/riscv/aacpsdsp_rvv.S index b581383f77..b85a5cc92c 100644 --- a/libavcodec/riscv/aacpsdsp_rvv.S +++ b/l

[FFmpeg-devel] [PATCH 2/5] lavc/aacpsdsp: unroll R-V V stereo interpolate

2023-09-29 Thread Rémi Denis-Courmont
--- libavcodec/riscv/aacpsdsp_rvv.S | 46 - 1 file changed, 23 insertions(+), 23 deletions(-) diff --git a/libavcodec/riscv/aacpsdsp_rvv.S b/libavcodec/riscv/aacpsdsp_rvv.S index b85a5cc92c..1a92fed515 100644 --- a/libavcodec/riscv/aacpsdsp_rvv.S +++ b/libavcodec/r

[FFmpeg-devel] [PATCH 4/5] lavc/aacpsdsp: unroll RISC-V V hybrid_analysis_ileave

2023-09-29 Thread Rémi Denis-Courmont
--- libavcodec/riscv/aacpsdsp_rvv.S | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/riscv/aacpsdsp_rvv.S b/libavcodec/riscv/aacpsdsp_rvv.S index f552d50e23..007002de70 100644 --- a/libavcodec/riscv/aacpsdsp_rvv.S +++ b/libavcodec/riscv/aacpsdsp_rvv.S @@ -168,10 +

[FFmpeg-devel] [PATCH 3/5] lavc/aacpsdsp: unroll RISC-V V mul_pair_single

2023-09-29 Thread Rémi Denis-Courmont
--- libavcodec/riscv/aacpsdsp_rvv.S | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/riscv/aacpsdsp_rvv.S b/libavcodec/riscv/aacpsdsp_rvv.S index 1a92fed515..f552d50e23 100644 --- a/libavcodec/riscv/aacpsdsp_rvv.S +++ b/libavcodec/riscv/aacpsdsp_rvv.S @@ -38,14 +3

[FFmpeg-devel] [PATCH 5/5] lavc/aacpsdsp: unroll RISC-V V hybrid_synthesis_deint

2023-09-29 Thread Rémi Denis-Courmont
--- libavcodec/riscv/aacpsdsp_rvv.S | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/riscv/aacpsdsp_rvv.S b/libavcodec/riscv/aacpsdsp_rvv.S index 007002de70..fe250cd83b 100644 --- a/libavcodec/riscv/aacpsdsp_rvv.S +++ b/libavcodec/riscv/aacpsdsp_rvv.S @@ -203,12 +

Re: [FFmpeg-devel] [PATCH 1/2] avformat/matroskaenc: Don't create wrong packet durations

2023-09-29 Thread Andreas Rheinhardt
James Almer: > On 9/29/2023 1:01 PM, Andreas Rheinhardt wrote: >> We have to write an explicit BlockDuration element (and use >> a BlockGroup instead of a SimpleBlock) in case the Track >> has a DefaultDuration that is inconsistent with the duration >> of the packet. >> >> The matroska-h264-remux t

[FFmpeg-devel] [PATCH] avformat/hls: Re-use crypto URLContext

2023-09-29 Thread Luis Scheurenbrand via ffmpeg-devel
Reset crypto state and keep nested http context alive. Previously, an unencrypted file, followed by an encrypted file would result in a corrupted stream, as the stream would not be closed and an encrypted file would be forwarded to the demuxer. fixes: https://trac.ffmpeg.org/ticket/10599 fixes: ht

[FFmpeg-devel] [PATCH] lavc/fmtconvert: unroll R-V V int32_to_float_fmul_scalar

2023-09-29 Thread Rémi Denis-Courmont
--- libavcodec/riscv/fmtconvert_rvv.S | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/riscv/fmtconvert_rvv.S b/libavcodec/riscv/fmtconvert_rvv.S index 51365753d9..d0e2f106d5 100644 --- a/libavcodec/riscv/fmtconvert_rvv.S +++ b/libavcodec/riscv/fmtconvert_rvv.S @@ -2

Re: [FFmpeg-devel] [PATCH v3] avformat/mxfdec: Remove this_partition

2023-09-29 Thread Michael Niedermayer
On Thu, Sep 28, 2023 at 01:12:02PM +0200, Tomas Härdin wrote: > ons 2023-09-27 klockan 23:12 +0200 skrev Michael Niedermayer: > > Hi > > > > On Wed, Sep 27, 2023 at 01:37:40PM +0200, Tomas Härdin wrote: > > > fre 2023-09-22 klockan 21:13 +0200 skrev Michael Niedermayer: > > > > Suggested-by: Tomas

[FFmpeg-devel] [PATCH 1/3] riscv: factor out the bswap32 assembler

2023-09-29 Thread Rémi Denis-Courmont
--- libavcodec/riscv/bswapdsp_rvb.S | 43 +- libavutil/riscv/bswap_rvb.S | 65 + 2 files changed, 67 insertions(+), 41 deletions(-) create mode 100644 libavutil/riscv/bswap_rvb.S diff --git a/libavcodec/riscv/bswapdsp_rvb.S b/libavcodec/ris

[FFmpeg-devel] [PATCH 2/3] swscale/rgb2rgb: add R-V Zbb shuffle_bytes_3210

2023-09-29 Thread Rémi Denis-Courmont
--- libswscale/riscv/Makefile | 1 + libswscale/riscv/rgb2rgb.c | 7 ++- libswscale/riscv/rgb2rgb_rvb.S | 30 ++ 3 files changed, 37 insertions(+), 1 deletion(-) create mode 100644 libswscale/riscv/rgb2rgb_rvb.S diff --git a/libswscale/riscv/Makefile b/

[FFmpeg-devel] [PATCH 3/3] swscale/rgb2rgb: remove R-V V shuffle_bytes_3012

2023-09-29 Thread Rémi Denis-Courmont
This is slower than the Zbb version on real hardware due to register strides. Proper support for vector byte-swap requires the Zvbb extension, but it's much too early for me to worry about it. --- libswscale/riscv/rgb2rgb.c | 2 -- libswscale/riscv/rgb2rgb_rvv.S | 25 -

[FFmpeg-devel] [PATCH] avdevice/lavfi: Fix double-free on error

2023-09-29 Thread Andreas Rheinhardt
After the AVFrame has been wrapped into a buffer, it is owned by the buffer and must not be freed manually any more. Yet this happens on subsequent errors. This bug was introduced in 6ca43a9675d651d7ea47c7ba2fafb1bf831c4d0b. Signed-off-by: Andreas Rheinhardt --- libavdevice/lavfi.c | 9 +---

Re: [FFmpeg-devel] [REFUND-REQUEST]] VDD'23 Travel

2023-09-29 Thread Michael Niedermayer
On Mon, Sep 25, 2023 at 12:44:27PM +0200, Niklas Haas wrote: > Hi, > > was a great VDD! Here are my travel costs: > > 316,00 EUR Flight STR <-> DUB > 101,00 EUR Parking at STR > 8,00 EUR Airport Shuttlebus in DUB > > I had planned to use the cheaper (~60 EUR) parking, but it was already > clos

Re: [FFmpeg-devel] [REFUND-REQUEST] VDD23 Travel

2023-09-29 Thread Michael Niedermayer
On Fri, Sep 29, 2023 at 08:18:19AM +0200, Stefano Sabatini wrote: > On date Monday 2023-09-25 14:17:06 +0100, Derek Buitenhuis wrote: > > Hello, > > > > Nice to see you all again after so long. > > > > My request is for my flights to/from Dublin, which cost 90.58 GBP. > > Approved on my side, pe

Re: [FFmpeg-devel] [PATCH] configure: disable vulkan if min version insufficient

2023-09-29 Thread Timo Rothenpieler
On 29.09.2023 15:52, Tristan Matthews wrote: Fixes: https://trac.ffmpeg.org/ticket/10596 --- configure | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/configure b/configure index 20db1801ed..50ba6f772f 100755 --- a/configure +++ b/configure @@ -7154,7 +7154,8 @@ enabled c

Re: [FFmpeg-devel] [PATCH] configure: disable vulkan if min version insufficient

2023-09-29 Thread Tristan Matthews
On Fri, Sep 29, 2023 at 1:37 PM Timo Rothenpieler wrote: > > On 29.09.2023 15:52, Tristan Matthews wrote: > > Fixes: https://trac.ffmpeg.org/ticket/10596 > > --- > > configure | 3 ++- > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > diff --git a/configure b/configure > > index 20db18

Re: [FFmpeg-devel] [PATCH] configure: disable vulkan if min version insufficient

2023-09-29 Thread Hendrik Leppkes
On Fri, Sep 29, 2023 at 3:55 PM Tristan Matthews wrote: > > Fixes: https://trac.ffmpeg.org/ticket/10596 > --- > configure | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/configure b/configure > index 20db1801ed..50ba6f772f 100755 > --- a/configure > +++ b/configure > @@

Re: [FFmpeg-devel] [PATCH] configure: disable vulkan if min version insufficient

2023-09-29 Thread Tristan Matthews
On Fri, Sep 29, 2023 at 2:32 PM Hendrik Leppkes wrote: > > On Fri, Sep 29, 2023 at 3:55 PM Tristan Matthews wrote: > > > > Fixes: https://trac.ffmpeg.org/ticket/10596 > > --- > > configure | 3 ++- > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > diff --git a/configure b/configure > >

Re: [FFmpeg-devel] [PATCH 3/5] avformat/mov: Better check for duplicate iloc

2023-09-29 Thread Michael Niedermayer
On Tue, Apr 25, 2023 at 03:22:50PM -0700, Vignesh Venkatasubramanian wrote: > On Mon, Apr 17, 2023 at 4:18 PM Michael Niedermayer > wrote: > > > > On Mon, Apr 17, 2023 at 12:36:26PM +0200, Anton Khirnov wrote: > > > Quoting Michael Niedermayer (2023-04-17 00:25:16) > > > > Fixes: memleak > > > > F

Re: [FFmpeg-devel] [PATCH] configure: disable vulkan if min version insufficient

2023-09-29 Thread Timo Rothenpieler
On 29.09.2023 20:10, Tristan Matthews wrote: On Fri, Sep 29, 2023 at 1:37 PM Timo Rothenpieler wrote: On 29.09.2023 15:52, Tristan Matthews wrote: Fixes: https://trac.ffmpeg.org/ticket/10596 --- configure | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/configure b/c

Re: [FFmpeg-devel] [PATCH 6/6] avformat/mov: Check avif_info

2023-09-29 Thread Michael Niedermayer
On Thu, Sep 28, 2023 at 12:37:57PM +0200, Anton Khirnov wrote: > Quoting Michael Niedermayer (2023-09-21 20:09:12) > > Fixes: leak > > Fixes: > > 62164/clusterfuzz-testcase-minimized-ffmpeg_DEMUXER_fuzzer-6674082962997248 > > > > Found-by: continuous fuzzing process > > https://github.com/google

Re: [FFmpeg-devel] [PATCH] configure: disable vulkan if min version insufficient

2023-09-29 Thread Tristan Matthews
On Fri, Sep 29, 2023 at 3:26 PM Timo Rothenpieler wrote: > > On 29.09.2023 20:10, Tristan Matthews wrote: > > On Fri, Sep 29, 2023 at 1:37 PM Timo Rothenpieler > > wrote: > >> > >> On 29.09.2023 15:52, Tristan Matthews wrote: > >>> Fixes: https://trac.ffmpeg.org/ticket/10596 > >>> --- > >>>c

[FFmpeg-devel] [PATCH] avcodec/libkvazaar: Simplified codec context color settings.

2023-09-29 Thread John Mather via ffmpeg-devel
Both FFmpeg and libkvazaar follow H.273, so we don't have to utilize a large LUT. Signed-off-by: John Mather --- libavcodec/libkvazaar.c | 62 ++--- 1 file changed, 8 insertions(+), 54 deletions(-) diff --git a/libavcodec/libkvazaar.c b/libavcodec/libkvazaar.

[FFmpeg-devel] [PATCH v2] avcodec/libkvazaar: Respect codec context color settings.

2023-09-29 Thread John Mather via ffmpeg-devel
This patch makes the libkvazaar encoder respect color settings that are present on the codec context, including color range, primaries, transfer function and colorspace. --- libavcodec/libkvazaar.c | 9 + 1 file changed, 9 insertions(+) diff --git a/libavcodec/libkvazaar.c b/libavcodec/li

Re: [FFmpeg-devel] [PATCH] avcodec/libkvazaar: Simplified codec context color settings.

2023-09-29 Thread John Mather via ffmpeg-devel
Please ignore this patch - it is incomplete. On Fri, Sep 29, 2023 at 4:36 PM John Mather wrote: > Both FFmpeg and libkvazaar follow H.273, so we don't have to utilize a > large LUT. > > Signed-off-by: John Mather > --- > libavcodec/libkvazaar.c | 62 ++--- >

Re: [FFmpeg-devel] [PATCH v2] avcodec/libkvazaar: Respect codec context color settings.

2023-09-29 Thread Vittorio Giovara
On Fri, Sep 29, 2023 at 5:12 PM John Mather via ffmpeg-devel < ffmpeg-devel@ffmpeg.org> wrote: > This patch makes the libkvazaar encoder respect color settings that are > present on the codec context, including color range, primaries, transfer > function and colorspace. > --- > libavcodec/libkvaz

[FFmpeg-devel] [PATCH 1/6] avformat/avidec: support huge durations

2023-09-29 Thread Michael Niedermayer
Fixes: signed integer overflow: 109817402400 * 301990077 cannot be represented in type 'long long' Fixes: 51896/clusterfuzz-testcase-minimized-ffmpeg_dem_AVI_fuzzer-6706191715139584 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by

[FFmpeg-devel] [PATCH 2/6] avformat/avidec: Correct unsigend type

2023-09-29 Thread Michael Niedermayer
Fixes: signed integer overflow: 9223372036854775807 + 1 cannot be represented in type 'long long' Fixes: 51896/clusterfuzz-testcase-minimized-ffmpeg_dem_AVI_fuzzer-6706191715139584 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by:

[FFmpeg-devel] [PATCH 3/6] vformat/cafdec: dont seek beyond 64bit

2023-09-29 Thread Michael Niedermayer
Fixes: signed integer overflow: 64 + 9223372036854775807 cannot be represented in type 'long long' Fixes: 51896/clusterfuzz-testcase-minimized-ffmpeg_dem_CAF_fuzzer-6418242730328064 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by

[FFmpeg-devel] [PATCH 4/6] avformat/cafdec: saturate start + size into 64bit

2023-09-29 Thread Michael Niedermayer
Fixes: signed integer overflow: 64 + 9223372036854775803 cannot be represented in type 'long long' Fixes: 51896/clusterfuzz-testcase-minimized-ffmpeg_dem_CAF_fuzzer-6536881135550464 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by

[FFmpeg-devel] [PATCH 5/6] avformat/dxa: Adjust order of operations around block align

2023-09-29 Thread Michael Niedermayer
Fixes: 51896/clusterfuzz-testcase-minimized-ffmpeg_dem_DXA_fuzzer-5730576523198464 Fixes: signed integer overflow: 2147483566 + 82 cannot be represented in type 'int' Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by: Michael Niede

[FFmpeg-devel] [PATCH 6/6] avformat/iff: Saturate avio_tell() + 12

2023-09-29 Thread Michael Niedermayer
Fixes: signed integer overflow: 9223372036854775796 + 12 cannot be represented in type 'long long' Fixes: 51896/clusterfuzz-testcase-minimized-ffmpeg_dem_IFF_fuzzer-4898373660704768 Found-by: continuous fuzzing process https://github.com/google/oss-fuzz/tree/master/projects/ffmpeg Signed-off-by

[FFmpeg-devel] [PATCH] ffbuild: Add gzip -n flag to fix reproducible builds

2023-09-29 Thread Rob Hall
Without this flag, timestamps were embedded into the final binary if CUDA was enabled. Signed-off-by: Rob Hall --- ffbuild/common.mak | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ffbuild/common.mak b/ffbuild/common.mak index f52473453e..ac54ac0681 100644 --- a/ffbuild/comm

Re: [FFmpeg-devel] [PATCH v3] avformat/mxfdec: Remove this_partition

2023-09-29 Thread Michael Niedermayer
On Fri, Sep 29, 2023 at 06:55:22PM +0200, Michael Niedermayer wrote: [...] > ok will apply with that fixing this issue caused ossfuzz to reveal 24 unrelated unfixed issues that where hidden in the same issue number as this thx [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611

Re: [FFmpeg-devel] [PATCH 4/6] avformat/cafdec: saturate start + size into 64bit

2023-09-29 Thread James Almer
On 9/29/2023 8:19 PM, Michael Niedermayer wrote: Fixes: signed integer overflow: 64 + 9223372036854775803 cannot be represented in type 'long long' Fixes: 51896/clusterfuzz-testcase-minimized-ffmpeg_dem_CAF_fuzzer-6536881135550464 Found-by: continuous fuzzing process https://github.com/google