[FFmpeg-devel] [PATCH] avcodec/decode: use avcodec_get_hw_config()

Fixes libavcodec/decode.c:1035:61: runtime error: member access within null pointer of type 'const struct AVCodecHWConfigInternal'. This can happen when hwaccel fails to initialize and hw_configs[i] is NULL. Signed-off-by: Kacper Michajłow --- libavcodec/decode.c | 2 +- 1 file

[FFmpeg-devel] [PATCH] avcodec/vp9mvs: fix misaligned access when clearing VP9mv

Fixes runtime error: member access within misaligned address for type 'av_alias64', which requires 8 byte alignment. VP9mv is aligned to 4 bytes, so instead doing 8 bytes clear, let's do 2 times 4 bytes. Signed-off-by: Kacper Michajłow --- libavcodec/vp9mvs.c | 3 ++- 1

[FFmpeg-devel] [PATCH] avformat/nuv: return early if header id fails to read

Fixes use of uninitialized value in memcmp below, reported by MSAN. Found by OSS-Fuzz. Signed-off-by: Kacper Michajłow --- libavformat/nuv.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/libavformat/nuv.c b/libavformat/nuv.c index 507a73b0fe..49915ecf16 100644 --- a

[FFmpeg-devel] [PATCH] avcodec/jpegxl_parser: check remaining data buffer size

Fixes use of uninitialized value, reported by MSAN. Found by OSS-Fuzz. Signed-off-by: Kacper Michajłow --- libavcodec/jpegxl_parser.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/libavcodec/jpegxl_parser.c b/libavcodec/jpegxl_parser.c index 8c45e1a1b7..8371d78a45

[FFmpeg-devel] [PATCH 1/4] avcodec/jpegxl_parser: ensure input padding is zeroed

Fixes use of uninitialized value, reported by MSAN. Found by OSS-Fuzz. Signed-off-by: Kacper Michajłow --- libavcodec/jpegxl_parser.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/jpegxl_parser.c b/libavcodec/jpegxl_parser.c index 8c45e1a1b7..f833f844c4 100644 --- a/libavcodec

[FFmpeg-devel] [PATCH 2/4] avcodec/parser: ensure input padding is zeroed

Fixes use of uninitialized value, reported by MSAN. Found by OSS-Fuzz. Signed-off-by: Kacper Michajłow --- libavcodec/parser.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/parser.c b/libavcodec/parser.c index af17ee9c15..426cc314fb 100644 --- a/libavcodec/parser.c +++ b

[FFmpeg-devel] [PATCH 3/4] avformat/img2dec: ensure input padding is zeroed

Fixes use of uninitialized value, reported by MSAN. Specifically in jpegxl parser. Found by OSS-Fuzz. Signed-off-by: Kacper Michajłow --- libavformat/img2dec.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavformat/img2dec.c b/libavformat/img2dec.c index ba52353074..c667d8574c

[FFmpeg-devel] [PATCH 4/4] avformat/jpegxl_anim_dec: ensure input padding is zeroed

Fixes use of uninitialized value, reported by MSAN. Found by OSS-Fuzz. Signed-off-by: Kacper Michajłow --- libavformat/jpegxl_anim_dec.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavformat/jpegxl_anim_dec.c b/libavformat/jpegxl_anim_dec.c index ac95d3b961..2338a2e8c0 100644 --- a

[FFmpeg-devel] [PATCH] avfilter/[vf_nnedi, vf_estdif]: Check interlaced flag on correct frame

Fixes regression in vf_nnedi after 24dc6d386c6f7edb8f6945319f53a7f0b1642bb8 and vf_estdif while at it. Signed-off-by: Kacper Michajłow --- libavfilter/vf_estdif.c | 2 +- libavfilter/vf_nnedi.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/libavfilter/vf_estdif.c b

[FFmpeg-devel] [PATCH] libavutil/opt: Small changes in example.

Fix const corectness and zero init the struct. This example code would actually crash when initializing string. --- libavutil/opt.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavutil/opt.h b/libavutil/opt.h index 9a76a47..9430b98 100644 --- a/libavutil/opt.h +++ b/l

[FFmpeg-devel] [PATCH] libavformat/rtpdec_asf: zero initialize the AVIOContext struct

This fixes crash in avformat_open_input() when accessing protocol_whitelist field. --- libavformat/rtpdec_asf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/rtpdec_asf.c b/libavformat/rtpdec_asf.c index 8459a51..e344376 100644 --- a/libavformat/rtpdec_asf.c +++ b

[FFmpeg-devel] [PATCH] libavformat/rtpdec_asf: zero initialize the AVIOContext struct

This fixes crash in avformat_open_input() when accessing protocol_whitelist field. --- libavformat/rtpdec_asf.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/rtpdec_asf.c b/libavformat/rtpdec_asf.c index 8459a51..2c09fda 100644 --- a/libavformat/rtpdec_asf.c +++ b

[FFmpeg-devel] [PATCH] Don't disable SSA Optimizer on MSVC v19.00.24218+.

--- configure | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/configure b/configure index 398e843..3bfd514 100755 --- a/configure +++ b/configure @@ -6317,9 +6317,9 @@ EOF check_func strtoll || add_cflags -Dstrtoll=_strtoi64 check_func strtoull || add_cflags -Ds

[FFmpeg-devel] [PATCH] configure: Fix DEF file post-processing with LTO enabled.

With LTO enabled exported symbol entry looks like: av_audio_convert @3 DATA In order to maintain valid format we need to strip everything after @. This patch fixes linking libraries compiled with MinGW toolchain with LTO enabled. Signed-off-by: Kacper Michajłow --- configure | 4 ++-- 1 file

[FFmpeg-devel] [PATCH] rtmpproto: Don't mistake app for playpath.

App is always first in the url path. This commit fixes the case when URL is provides as "rtmp://server[:port]/app" and playpath is declared in AVOption. Signed-off-by: Kacper Michajłow --- libavformat/rtmpproto.c | 33 +++-- 1 file changed, 19 inserti

Re: [FFmpeg-devel] [PATCH] rtmpproto: Don't mistake app for playpath.

2014-10-19 16:40 GMT+02:00 Michael Niedermayer : > > On Sun, Oct 19, 2014 at 02:12:14PM +0200, Kacper Michajłow wrote: > > App is always first in the url path. This commit fixes the case when URL is > > provides as "rtmp://server[:port]/app" and playpath is declared in

[FFmpeg-devel] [PATCH v2] rtmpproto: Don't mistake app for playpath.

For URLs "rtmp://server[:port]/foo" determine what `foo` refers to. If application name has been defined by the user assume that `foo` is a playpath, otherwise assume application name. Signed-off-by: Kacper Michajłow --- libavformat/rtmpproto.c | 39 +---

[FFmpeg-devel] [PATCH 1/3] avcodec/libaribb24: change new lines to \n in ASS header

Fixes remaining \r\n is ASS header after 57c545090d. --- libavcodec/libaribb24.c | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/libavcodec/libaribb24.c b/libavcodec/libaribb24.c index 009c995ded..0e4fa9935c 100644 --- a/libavcodec/libaribb24.c +++ b/

[FFmpeg-devel] [PATCH 2/3] avcodec/libaribcaption: change new lines to \n in ASS header

Fixes remaining \r\n is ASS header after 57c545090d. --- libavcodec/libaribcaption.c | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/libavcodec/libaribcaption.c b/libavcodec/libaribcaption.c index 0b67d41772..46a2a591c5 100644 --- a/libavcodec/libar

[FFmpeg-devel] [PATCH 3/3] avcodec/libzvbi-teletextdec: change new lines to \n in ASS header

Fixes remaining \r\n is ASS header after 57c545090d. Fixes AVERROR_BUG error during init as this decoder expected `\r\n` in default ASS header. strstr(..., "\r\n[Events]\r\n") failed after changes in 57c545090d. Fixes: 57c545090d --- libavcodec/libzvbi-teletextdec.c | 6 +++--- 1 file changed, 3

[FFmpeg-devel] [PATCH v5] avcodec/h2645_sei: validate Mastering Display Colour Volume SEI values

unknown or unspecified or specified by other means not specified in this Specification. Signed-off-by: Kacper Michajłow --- libavcodec/h2645_sei.c | 55 ++ 1 file changed, 39 insertions(+), 16 deletions(-) diff --git a/libavcodec/h2645_sei.c b/libavcodec

[FFmpeg-devel] [PATCH] avformat/data_uri: Fix base64 decode buffer size calculation

Also reject input if it is too short. Found by OSS-Fuzz. Signed-off-by: Kacper Michajłow --- libavformat/data_uri.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/data_uri.c b/libavformat/data_uri.c index 3868a19630..f97ecbab37 100644 --- a/libavformat

[FFmpeg-devel] [PATCH] avformat/srtpproto: pass options to nested protocol

This fixes passing options dict. Fixes some timeouts found by OSS-Fuzz. Signed-off-by: Kacper Michajłow --- libavformat/srtpproto.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavformat/srtpproto.c b/libavformat/srtpproto.c index 61669e05ad..02cf156327 100644

[FFmpeg-devel] [PATCH] avformat/rtpproto: free ip filters on open error

Found by OSS-Fuzz. Signed-off-by: Kacper Michajłow --- libavformat/rtpproto.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavformat/rtpproto.c b/libavformat/rtpproto.c index b1cdf061f0..15d0050936 100644 --- a/libavformat/rtpproto.c +++ b/libavformat/rtpproto.c @@ -379,6 +379,7

[FFmpeg-devel] [PATCH v2 1/3] avcodec/libaribb24: change new lines to \n in ASS header

Fixes remaining \r\n is ASS header after 57c545090d. Signed-off-by: Kacper Michajłow --- libavcodec/libaribb24.c | 24 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/libavcodec/libaribb24.c b/libavcodec/libaribb24.c index 009c995ded..0e4fa9935c 100644

[FFmpeg-devel] [PATCH v2 2/3] avcodec/libaribcaption: change new lines to \n in ASS header

Fixes remaining \r\n is ASS header after 57c545090d. Signed-off-by: Kacper Michajłow --- libavcodec/libaribcaption.c | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/libavcodec/libaribcaption.c b/libavcodec/libaribcaption.c index 0b67d41772

[FFmpeg-devel] [PATCH v2 3/3] avcodec/libzvbi-teletextdec: change new lines to \n in ASS header

Fixes remaining \r\n is ASS header after 57c545090d. Fixes AVERROR_BUG error during init as this decoder expected `\r\n` in default ASS header. strstr(..., "\r\n[Events]\r\n") failed after changes in 57c545090d. Fixes: 57c545090d Signed-off-by: Kacper Michajłow --- libavcod

[FFmpeg-devel] [PATCH v2] avformat/hls: update current segment reference before use

It may be invalidated by the time it is used. Fixes use after free when accessing current segment. Fixes: #10825 Signed-off-by: Kacper Michajłow --- libavformat/hls.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavformat/hls.c b/libavformat/hls.c index 214a99c7ba..62473a15dd

[FFmpeg-devel] [PATCH] avformat/rtsp: Set timeouts also when reading header

Signed-off-by: Kacper Michajłow --- libavformat/rtsp.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavformat/rtsp.c b/libavformat/rtsp.c index b0c61ee00a..4b067ee76b 100644 --- a/libavformat/rtsp.c +++ b/libavformat/rtsp.c @@ -135,6 +135,8 @@ static AVDictionary *map_to_opts

[FFmpeg-devel] [PATCH] avformat/rpl: reject invalid sample rate

Fixes overflow check for bit_rate multiplication few lines below. Found by OSS-Fuzz. Signed-off-by: Kacper Michajłow --- libavformat/rpl.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavformat/rpl.c b/libavformat/rpl.c index 09d0b68f74..32a762b60a 100644 --- a/libavformat/rpl.c

[FFmpeg-devel] [PATCH v2] avformat/data_uri: Fix base64 decode buffer size calculation

Signed-off-by: Kacper Michajłow --- libavformat/data_uri.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/data_uri.c b/libavformat/data_uri.c index 3868a19630..5991ad4ed4 100644 --- a/libavformat/data_uri.c +++ b/libavformat/data_uri.c @@ -73,7 +73,7 @@ static

[FFmpeg-devel] [PATCH] avcodec/h2645_sei: validate Mastering Display Colour Volume SEI values

unknown or unspecified or specified by other means not specified in this Specification. Signed-off-by: Kacper Michajłow --- libavcodec/h2645_sei.c | 53 +- 1 file changed, 37 insertions(+), 16 deletions(-) diff --git a/libavcodec/h2645_sei.c b/libavcodec

[FFmpeg-devel] [PATCH v2] avcodec/h2645_sei: validate Mastering Display Colour Volume SEI values

unknown or unspecified or specified by other means not specified in this Specification. Signed-off-by: Kacper Michajłow --- libavcodec/h2645_sei.c | 53 +- 1 file changed, 37 insertions(+), 16 deletions(-) diff --git a/libavcodec/h2645_sei.c b/libavcodec

[FFmpeg-devel] [PATCH v3] avcodec/h2645_sei: validate Mastering Display Colour Volume SEI values

unknown or unspecified or specified by other means not specified in this Specification. Signed-off-by: Kacper Michajłow --- libavcodec/h2645_sei.c | 55 ++ 1 file changed, 39 insertions(+), 16 deletions(-) diff --git a/libavcodec/h2645_sei.c b/libavcodec

[FFmpeg-devel] [PATCH v4] avcodec/h2645_sei: validate Mastering Display Colour Volume SEI values

unknown or unspecified or specified by other means not specified in this Specification. Signed-off-by: Kacper Michajłow --- libavcodec/h2645_sei.c | 55 ++ 1 file changed, 39 insertions(+), 16 deletions(-) diff --git a/libavcodec/h2645_sei.c b/libavcodec

[FFmpeg-devel] [PATCH] avformat/hls: update current segment reference before use

It may be invalidated by the time it is used. Fixes use after free when accessing current segment. Fixes: #10825 --- libavformat/hls.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavformat/hls.c b/libavformat/hls.c index f6b44c2e35..94bc6bc064 100644 --- a/libavformat/hls.c +++ b/li

[FFmpeg-devel] [PATCH 1/3] avcodec/vulkan_decode: fix struct type for h265_profile

Signed-off-by: Kacper Michajłow --- libavcodec/vulkan_decode.c | 2 +- libavcodec/vulkan_decode.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/libavcodec/vulkan_decode.c b/libavcodec/vulkan_decode.c index f20733fb39..04350578f9 100644 --- a/libavcodec/vulkan_decode.c

[FFmpeg-devel] [PATCH 2/3] avcodec/vulkan_decode: check specVersion to avoid using incompatible implementations

Fixes: VUID-VkVideoSessionCreateInfoKHR-pStdHeaderVersion-07190 VUID-VkVideoSessionCreateInfoKHR-pStdHeaderVersion-07191 As a bonus avoids crashing AMD video driver on Windows, which currently doesn't advertise support. Signed-off-by: Kacper Michajłow --- libavcodec/vulkan_decode.

[FFmpeg-devel] [PATCH 3/3] avcodec/vulkan_decode: print also codec header name

Signed-off-by: Kacper Michajłow --- libavcodec/vulkan_decode.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavcodec/vulkan_decode.c b/libavcodec/vulkan_decode.c index 7607edf52e..0d561b7450 100644 --- a/libavcodec/vulkan_decode.c +++ b/libavcodec/vulkan_decode.c @@ -863,6 +863,9

[FFmpeg-devel] [PATCH 1/2] configure: don't force specific C++ standard library linking

Other C++ standard libraries exist. Also, this is not a proper way to link the standard library anyway. Instead when a C++ dependency is detected, switch to the C++ compiler driver to properly link everything. Signed-off-by: Kacper Michajłow --- configure | 27 ++- 1

[FFmpeg-devel] [PATCH 2/2] configure: add libplacebo to the list of C++ dependencies

If libplacebo is statically linked with glslang it requires C++ standard library. Also recently more C++ code has been added to libplacebo. Signed-off-by: Kacper Michajłow --- configure | 1 + 1 file changed, 1 insertion(+) diff --git a/configure b/configure index 90ee6e4f7d..ed714639d1 100755

[FFmpeg-devel] [PATCH v2] configure: add libplacebo to the list of C++ dependencies

If libplacebo is statically linked with glslang it requires C++ standard library. Also recently more C++ code has been added to libplacebo. Signed-off-by: Kacper Michajłow --- configure | 1 + 1 file changed, 1 insertion(+) diff --git a/configure b/configure index f3ff48586a..0dbdffb5c1 100755

[FFmpeg-devel] [PATCH v2 1/2] configure: don't force specific C++ standard library linking

Other C++ standard libraries exist. Also, this is not a proper way to link the standard library anyway. Instead when a C++ dependency is detected, switch to the C++ compiler driver to properly link everything. Signed-off-by: Kacper Michajłow --- configure | 26 ++ 1 file

[FFmpeg-devel] [PATCH] configure: use -flto=auto if available

Signed-off-by: Kacper Michajłow --- configure | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/configure b/configure index 9e51abd0d3..bf5a687239 100755 --- a/configure +++ b/configure @@ -7173,8 +7173,9 @@ check_optflags -fno-signed-zeros if enabled lto; then

[FFmpeg-devel] [PATCH] swscale: add missing opaque parameter after f2de911818

Fixes function prototype mismatch, warning Wlto-type-mismatch. Signed-off-by: Kacper Michajłow --- libswscale/x86/rgb2rgb_template.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libswscale/x86/rgb2rgb_template.c b/libswscale/x86/rgb2rgb_template.c index 4aba25dd51

[FFmpeg-devel] [PATCH v2] configure: try to use -flto=thin or -flto=auto if available

Signed-off-by: Kacper Michajłow --- configure | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/configure b/configure index 9e51abd0d3..a651fdec5a 100755 --- a/configure +++ b/configure @@ -7173,8 +7173,9 @@ check_optflags -fno-signed-zeros if enabled lto; then

[FFmpeg-devel] [PATCH] avfilter/vf_scale: validate values before converting to integer

Fixes the conversion of double values to integer, which may be out of the representable range. Also, bail out on overflow check instead of printing an error only. Found by OSS-Fuzz. Signed-off-by: Kacper Michajłow --- libavfilter/vf_scale.c | 33 +++-- 1 file

[FFmpeg-devel] [PATCH] avcodec/wmavoice: use av_clipd for double values

Fixes Clang warning. Signed-off-by: Kacper Michajłow --- libavcodec/wmavoice.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/wmavoice.c b/libavcodec/wmavoice.c index 258c71330c..3db73773b7 100644 --- a/libavcodec/wmavoice.c +++ b/libavcodec/wmavoice.c @@ -655,7

[FFmpeg-devel] [PATCH] avformat/vpk: fix divide by zero

Can happen after calling avformat_find_stream_info() when the codec fails to open, but return value is 0 and subsequent uses of this context have zero value in channel number. Found by OSS-Fuzz. Signed-off-by: Kacper Michajłow --- libavformat/vpk.c | 2 ++ 1 file changed, 2 insertions(+) diff

[FFmpeg-devel] [PATCH] avformat/mov: ensure required number of bytes is read

Fixes: use-of-uninitialized-value Found by OSS-Fuzz. --- libavformat/mov.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 1052691936..f2d8aee766 100644 --- a/libavformat/mov.c +++ b/libavformat/mov.c @@ -7096,7 +7096,7 @@ static int

Re: [FFmpeg-devel] [PATCH] configure: support MSYS2

2015-01-25 21:36 GMT+01:00 Hendrik Leppkes : > On Sun, Jan 25, 2015 at 7:44 PM, James Almer wrote: > > > On 25/01/15 2:47 PM, Hendrik Leppkes wrote: > > > On Sun, Jan 25, 2015 at 6:43 PM, Michael Niedermayer > > > > wrote: > > > > > >> On Sun, Jan 25, 2015 at 05:02:58PM +0100, Reimar Döffinger w

Re: [FFmpeg-devel] hevc : support deinterlacing inside the decoder

2015-02-08 10:48 GMT+01:00 Carl Eugen Hoyos : > Mickaël Raulet insa-rennes.fr> writes: > > > As we can consider, we won't have 4k interlaced > > content, copying a field into a frame should be ok. > > This is what has been done in this implementation. > > Do you have a sample? > I am only interes

[FFmpeg-devel] [PATCH] avcodec: Fix libdcadec include dir

--- configure | 2 +- libavcodec/libdcadec.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/configure b/configure index a449bc7..1ef83e3 100755 --- a/configure +++ b/configure @@ -4925,7 +4925,7 @@ enabled libcelt && require libcelt celt/celt.h celt_

[FFmpeg-devel] [PATCH] avcodec/vulkan_decode: Make Vulkan decode less spammy in verbose logs

Drop per frame decode messages to AV_LOG_TRACE level. Signed-off-by: Kacper Michajłow --- libavcodec/vulkan_av1.c | 2 +- libavcodec/vulkan_h264.c | 2 +- libavcodec/vulkan_hevc.c | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/libavcodec/vulkan_av1.c b/libavcodec

[FFmpeg-devel] [PATCH] configure: Fix Microsoft tools detection

LLVM tools print installation path upon execution. If one uses LLVM tools bundled with Microsoft Visual Studio installation, they would be incorrectly detected as Microsoft's ones. Signed-off-by: Kacper Michajłow --- configure | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)

[FFmpeg-devel] [PATCH] avformat/http: copy only mime type from Content-Type

Content-Type can include charset and boundary which is not a part of mime type and shouldn't be copied as such. Fixes HLS playback when the Content-Type includes additional fields. Signed-off-by: Kacper Michajłow --- libavformat/http.c | 2 +- 1 file changed, 1 insertion(+), 1 del

[FFmpeg-devel] [PATCH v2] avformat/http: copy only mime type from Content-Type

Content-Type can include charset and boundary which is not a part of mime type and shouldn't be copied as such. Fixes HLS playback when the Content-Type includes additional fields. Signed-off-by: Kacper Michajłow --- libavformat/http.c | 2 +- 1 file changed, 1 insertion(+), 1 del

[FFmpeg-devel] [PATCH] lavu/vulkan: fix handle type for 32-bit targets

Fixes compilation with clang which errors out on Wint-conversion. Signed-off-by: Kacper Michajłow --- libavutil/hwcontext_vulkan.c | 2 +- libavutil/vulkan.h | 4 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/libavutil/hwcontext_vulkan.c b/libavutil

[FFmpeg-devel] [PATCH] lavfi/vf_libplacebo: allow linking to shared library with dllimport

Address of dll imported variables can't be used for constant initialization in C language modes. --- libavfilter/vf_libplacebo.c | 39 - 1 file changed, 21 insertions(+), 18 deletions(-) diff --git a/libavfilter/vf_libplacebo.c b/libavfilter/vf_libplacebo.c ind

[FFmpeg-devel] [PATCH] avcodec/jpegxl_parser: check entropy_decoder_read_symbol return value

Found by OSS-Fuzz. Signed-off-by: Kacper Michajłow --- libavcodec/jpegxl_parser.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/libavcodec/jpegxl_parser.c b/libavcodec/jpegxl_parser.c index 8c45e1a1b7..746c429b9c 100644 --- a/libavcodec/jpegxl_parser.c +++ b

[FFmpeg-devel] [PATCH] avfilter/af_ashowinfo: fix scaling factor of replaygain peak values

Fixes: 8542f9c4f17125d483c40c0c5723842f1c982f81 Signed-off-by: Kacper Michajłow --- libavfilter/af_ashowinfo.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavfilter/af_ashowinfo.c b/libavfilter/af_ashowinfo.c index b83847b866..de94d4c7c2 100644 --- a/libavfilter

[FFmpeg-devel] [PATCH] avutil/file: fix av_file_map file mapping on Windows

when trying to write to a read-only memory page. Signed-off-by: Kacper Michajłow --- libavutil/file.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavutil/file.c b/libavutil/file.c index db8507286b..4ef940a6c3 100644 --- a/libavutil/file.c +++ b/libavutil/file.c @@ -112,7

[FFmpeg-devel] [PATCH] avformat/mov: fix eof check after avio_skip()

EOF check after reading the size and type after seek, ensuring the EOF flag is set when necessary. Signed-off-by: Kacper Michajłow --- libavformat/mov.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 2c8be51063..9a388bf723 100644

[FFmpeg-devel] [PATCH v2] avformat/mov: fix eof check in mov_read_iinf()

EOF check after reading the size and type, ensuring the EOF flag is set when necessary. Signed-off-by: Kacper Michajłow --- libavformat/mov.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libavformat/mov.c b/libavformat/mov.c index 2c8be51063..9a388bf723 100644 --- a

[FFmpeg-devel] [PATCH] avutil/hwcontext_amf: fix crash on uninit after init failed

amf_device_create() calls amf_device_uninit() on errors, but if things were not initialized it will null deref amf_ctx->factory. Fixes: https://github.com/mpv-player/mpv/issues/15814 Signed-off-by: Kacper Michajłow --- libavutil/hwcontext_amf.c | 6 -- 1 file changed, 4 insertions(+)

[FFmpeg-devel] [PATCH v2] avfilter/af_ashowinfo: fix scaling factor of replaygain peak values

Commit 8542f9c4f17125d483c40c0c5723842f1c982f81 changed replaygain peak values to use 100k instead of UINT32_MAX as peak, with possibility of overflow. af_ashowinfo was never updated to reflect this, so we update it now. Fixes: 8542f9c4f17125d483c40c0c5723842f1c982f81 Signed-off-by: Kacper

[FFmpeg-devel] [PATCH] avformat/vqf: fix memory leak in add_metadata()

Signed-off-by: Kacper Michajłow --- libavformat/vqf.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/libavformat/vqf.c b/libavformat/vqf.c index 58b1546f53..fbe54739cd 100644 --- a/libavformat/vqf.c +++ b/libavformat/vqf.c @@ -66,10 +66,10 @@ static int add_metadata

[FFmpeg-devel] [PATCH] tools/target_swr_fuzzer: fix memory leak on av_samples_fill_arrays() error

--- tools/target_swr_fuzzer.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/tools/target_swr_fuzzer.c b/tools/target_swr_fuzzer.c index f154a11632..59fa24af64 100644 --- a/tools/target_swr_fuzzer.c +++ b/tools/target_swr_fuzzer.c @@ -79,7 +79,7 @@ int LLVMFuzzerTestOneIn

[FFmpeg-devel] [PATCH 1/2] avutil/intmath: use AV_HAS_BUILTIN to detect builtin availability

Fixes use of bultins on clang x86_64-pc-windows-msvc which does not define any __GNUC__. Also on other targets __GNUC__ is defined to 4 by default, so any feature testing based on version is not really valid. Signed-off-by: Kacper Michajłow --- libavutil/intmath.h | 12

[FFmpeg-devel] [PATCH 0/6] Minor fixes and for fuzzing targets

Minor fixes and for fuzzing targets. Mostly motivated to reduce spam in the build log. While at it, tiff/tdsc check was fixed, which was real bug. Kacper Michajłow (6): tools/Makefile: add identifier macros for specific fuzzing targets tools/target_dec_fuzzer: fix tiff/tdsc check tools

[FFmpeg-devel] [PATCH 1/6] tools/Makefile: add identifier macros for specific fuzzing targets

This will allow to add conditional code per target codec. Signed-off-by: Kacper Michajłow --- tools/Makefile | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tools/Makefile b/tools/Makefile index 7ae6e3cb75..3f99b337e5 100644 --- a/tools/Makefile +++ b/tools/Makefile

[FFmpeg-devel] [PATCH 3/6] tools/target_dec_fuzzer: suppress Wdeclaration-after-statement

To avoid spam in log, each fuzzer is built separately so it's amplified a lot. Signed-off-by: Kacper Michajłow --- tools/target_dec_fuzzer.c | 41 +++ 1 file changed, 24 insertions(+), 17 deletions(-) diff --git a/tools/target_dec_fuzzer.c b/

[FFmpeg-devel] [PATCH 2/6] tools/target_dec_fuzzer: fix tiff/tdsc check

Comparing `FFMPEG_DECODER == tiff` never worked because tiff is not defined. Original commit expected string compare, but C preprocesor doesn't work this way. Fixes: 3371d0611fcd31e0bc72553d88774512a58bd2ef Signed-off-by: Kacper Michajłow --- tools/target_dec_fuzzer.c | 2 +- 1 file chang

[FFmpeg-devel] [PATCH 4/6] tools/target_dec_fuzzer: suppress Wunused-function

Signed-off-by: Kacper Michajłow --- tools/target_dec_fuzzer.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/tools/target_dec_fuzzer.c b/tools/target_dec_fuzzer.c index a15f1a3f9c..7bbf1375d0 100644 --- a/tools/target_dec_fuzzer.c +++ b/tools/target_dec_fuzzer.c @@ -72,6 +72,8 @@ static

[FFmpeg-devel] [PATCH 6/6] tools/target_dem_fuzzer: remove unused fuzz_tag

Signed-off-by: Kacper Michajłow --- tools/target_dem_fuzzer.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/tools/target_dem_fuzzer.c b/tools/target_dem_fuzzer.c index 19bc1f09c1..e169438ceb 100644 --- a/tools/target_dem_fuzzer.c +++ b/tools/target_dem_fuzzer.c @@ -98,10 +98,7 @@ static

[FFmpeg-devel] [PATCH 5/6] tools/target_dem_fuzzer: make fuzz data pointer constant

Mostly to avoid warnings. Signed-off-by: Kacper Michajłow --- tools/target_dem_fuzzer.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/target_dem_fuzzer.c b/tools/target_dem_fuzzer.c index 8e96fad7f8..19bc1f09c1 100644 --- a/tools/target_dem_fuzzer.c +++ b/tools

[FFmpeg-devel] [PATCH] tools/target_swr_fuzzer: fix memory leak on av_samples_fill_arrays() error

Signed-off-by: Kacper Michajłow --- tools/target_swr_fuzzer.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/tools/target_swr_fuzzer.c b/tools/target_swr_fuzzer.c index f154a11632..59fa24af64 100644 --- a/tools/target_swr_fuzzer.c +++ b/tools/target_swr_fuzzer.c

[FFmpeg-devel] [PATCH] w32pthreads: add support for setting thread name

Signed-off-by: Kacper Michajłow --- compat/w32pthreads.h | 30 ++ libavutil/thread.h | 2 ++ 2 files changed, 32 insertions(+) diff --git a/compat/w32pthreads.h b/compat/w32pthreads.h index fd6428e29f..83b4819205 100644 --- a/compat/w32pthreads.h +++ b/compat

[FFmpeg-devel] [PATCH v2] w32pthreads: add support for setting thread name

Signed-off-by: Kacper Michajłow --- compat/w32pthreads.h | 30 ++ libavutil/thread.h | 2 ++ 2 files changed, 32 insertions(+) diff --git a/compat/w32pthreads.h b/compat/w32pthreads.h index fd6428e29f..8d5b4729fa 100644 --- a/compat/w32pthreads.h +++ b/compat

[FFmpeg-devel] [PATCH 2/2] avformat/internal: add missing __clang__ check

Clang x86_64-pc-windows-msvc doesn't define __GNUC__. Signed-off-by: Kacper Michajłow --- libavformat/internal.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/internal.h b/libavformat/internal.h index fe428d85eb..bf83571430 100644 --- a/libavformat/inter

[FFmpeg-devel] [PATCH 1/2] avformat/demux: use io_close2 when closing avfromat

ff-by: Kacper Michajłow --- libavformat/demux.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/libavformat/demux.c b/libavformat/demux.c index 2795863567..ecd4f40da9 100644 --- a/libavformat/demux.c +++ b/libavformat/demux.c @@ -383,11 +383,10 @@ void avformat_close_

[FFmpeg-devel] [PATCH 2/2] avformat/imfdec: inherit opaque from parent AVFormatContext

io_open and io_close2 callbacks may use opaque pointer stored in the context. They are already inherited, so opaque should also be passed through. Fixes IMF playback in mpv. Signed-off-by: Kacper Michajłow --- libavformat/imfdec.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavformat

[FFmpeg-devel] [PATCH 1/2] configure: use proper Windows-style static library naming

ct mingw build. MSVC builds work the same, because there libraries are passed as files directly to linker command. This also removes LD_LIB from Win32/64 target as there is one type of .lib in practice. We cannot build both shared and static as noted by the next line. Signed-off-by: Kacper Micha

[FFmpeg-devel] [PATCH 2/2] configure: add -Wl when linker is called indirectly

It's possible to call linker indirectly through driver like Clang. In which cases linker args has to be prefixed with -Wl. Signed-off-by: Kacper Michajłow --- configure | 5 + 1 file changed, 5 insertions(+) diff --git a/configure b/configure index c57da316cc..a5c3bd3917 100755

[FFmpeg-devel] [PATCH v2] configure: add -Wl when linker is called indirectly

It's possible to call linker indirectly through driver like Clang. In which cases linker args has to be prefixed with -Wl. Signed-off-by: Kacper Michajłow --- configure | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/configure b/configure index c57da316cc..0dede

[FFmpeg-devel] [PATCH] configure: fix Microsoft tools detection

test if "Microsoft" is at the beginning of the line, as it is always the case. Signed-off-by: Kacper Michajłow --- configure | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/configure b/configure index 534b443f7d..5b2a04ae6a 100755 --- a/configure +++ b/confi

[FFmpeg-devel] [PATCH v2 0/4] Minor fixes and for fuzzing targets

Minor fixes and for fuzzing targets. Mostly motivated to reduce spam in the build log. v2 only for rebase only. Kacper Michajłow (4): tools/target_dec_fuzzer: move things to suppress Wdeclaration-after-statement tools/target_dec_fuzzer: suppress Wunused-function tools/target_dem_fuzzer

[FFmpeg-devel] [PATCH v2 1/4] tools/target_dec_fuzzer: move things to suppress Wdeclaration-after-statement

To avoid spam in log, each fuzzer is built separately so it's amplified a lot. Signed-off-by: Kacper Michajłow --- tools/target_dec_fuzzer.c | 31 ++- 1 file changed, 18 insertions(+), 13 deletions(-) diff --git a/tools/target_dec_fuzzer.c b/

[FFmpeg-devel] [PATCH v2 2/4] tools/target_dec_fuzzer: suppress Wunused-function

Signed-off-by: Kacper Michajłow --- tools/target_dec_fuzzer.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/tools/target_dec_fuzzer.c b/tools/target_dec_fuzzer.c index 7afb23619e..6950d134d9 100644 --- a/tools/target_dec_fuzzer.c +++ b/tools/target_dec_fuzzer.c @@ -72,6 +72,8 @@ static

[FFmpeg-devel] [PATCH v2 3/4] tools/target_dem_fuzzer: make fuzz data pointer constant

Mostly to avoid warnings. Signed-off-by: Kacper Michajłow --- tools/target_dem_fuzzer.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/target_dem_fuzzer.c b/tools/target_dem_fuzzer.c index 8e96fad7f8..19bc1f09c1 100644 --- a/tools/target_dem_fuzzer.c +++ b/tools

[FFmpeg-devel] [PATCH v2 4/4] tools/target_dem_fuzzer: remove unused fuzz_tag

Signed-off-by: Kacper Michajłow --- tools/target_dem_fuzzer.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/tools/target_dem_fuzzer.c b/tools/target_dem_fuzzer.c index 19bc1f09c1..e169438ceb 100644 --- a/tools/target_dem_fuzzer.c +++ b/tools/target_dem_fuzzer.c @@ -98,10 +98,7 @@ static

[FFmpeg-devel] [PATCH] configure: treat unrecognized flags as errors on MSVC

upport for this flag. Clang-CL sets _MSC_FULL_VER as one example. Signed-off-by: Kacper Michajłow --- configure | 6 ++ 1 file changed, 6 insertions(+) diff --git a/configure b/configure index ed6430ea32..20276a3c4f 100755 --- a/configure +++ b/configure @@ -5276,6 +5276,12 @@ else ar_

[FFmpeg-devel] [PATCH] avformat/scd: ensure SCD_MIN_HEADER_SIZE bytes are read

Instead of accessing unintialized data when input is shorter than expected size. Fixes use of uninitialized value in MSAN build. Found by OSS-Fuzz. Signed-off-by: Kacper Michajłow --- libavformat/scd.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/libavformat/scd.c b

[FFmpeg-devel] [PATCH] MAINTAINERS: add myself

Signed-off-by: Kacper Michajłow --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index 07cea8afeb..650300dadc 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -547,6 +547,7 @@ James Darnley Jan Ekström Joakim Plate Jun Zhao +Kacper Michajłow Kieran

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

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 changed, 3 insertions(+), 3 deletions(-) diff --git a/libavformat/s

[FFmpeg-devel] [PATCH] checkasm/swscale: fix function prototypes

This aligns declared function types in checkasm with real definition. Fixes FATE: checkasm-{sw_rgb,sw_scale,sw_yuv2rgb,sw_yuv2yuv} Fixes: runtime error: call to function through pointer to incorrect function type Fixes: c1a0e657638f7007dcc807a2d985c22631fcd6d3 Signed-off-by: Kacper Michajłow

[FFmpeg-devel] [PATCH 1/2] avcodec/opus: don't materialize buf pointer from null

Fixes: avcodec/opus/dec.c: runtime error: applying non-zero offset 10 to null pointer Signed-off-by: Kacper Michajłow --- libavcodec/opus/dec.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/libavcodec/opus/dec.c b/libavcodec/opus/dec.c index 6c59dc1f46..dbefcddb0c 100644 --- a

[FFmpeg-devel] [PATCH] avutil/avstring: shrink allocation from av_get_token to fit token

obtaining the return value, simply realloc the buffer to the actual size needed for output string. Fixes OOM when parsing filter graph string. Fixes OSS-Fuzz: 394983446 Signed-off-by: Kacper Michajłow --- libavutil/avstring.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a

[FFmpeg-devel] [PATCH v2] avutil/avstring: shrink allocation from av_get_token to fit token

obtaining the return value, simply realloc the buffer to the actual size needed for output string. Fixes OOM when parsing filter graph string. Fixes OSS-Fuzz: 394983446 Signed-off-by: Kacper Michajłow --- libavutil/avstring.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git

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

Signed-off-by: Kacper Michajłow --- fftools/cmdutils.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/fftools/cmdutils.c b/fftools/cmdutils.c index be21ed2c6c..e442935cae 100644 --- a/fftools/cmdutils.c +++ b/fftools/cmdutils.c @@ -255,9 +255,10 @@ static int write_option(void

  1   2   >