From: 52c3e9c0f140a8742034e107fb0f371c0d73bf1d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Marek=20Beh=C3=BAn?= Date: Sun, 12 Sep 2021 04:20:11 +0200 Subject: [PATCH] chromium-93: fix building with system-ffmpeg MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The version of ffmpeg bundled in Chromium changed ABI a little, which affected the code in that the type of the last parameter of av_packet_get_side_data() is now size_t instead of int. This causes Chromium to fail to build with system-ffmpeg if the system ffmpeg's version does not yet have this ABI change. Fix the code to be able to compile with older ffmpeg as well. Signed-off-by: Marek Behún --- media/filters/audio_decoder_unittest.cc | 4 ++++ media/filters/ffmpeg_demuxer.cc | 16 ++++++++++++++++ 2 files changed, 20 insertions(+) Index: chromium-126.0.6478.8/media/filters/ffmpeg_demuxer.cc =================================================================== --- chromium-126.0.6478.8.orig/media/filters/ffmpeg_demuxer.cc +++ chromium-126.0.6478.8/media/filters/ffmpeg_demuxer.cc @@ -343,11 +343,19 @@ } base::span GetSideData(const AVPacket* packet) { +#if LIBAVUTIL_VERSION_MAJOR < 57 + int side_data_size = 0; +#else size_t side_data_size = 0; +#endif uint8_t* side_data = av_packet_get_side_data( packet, AV_PKT_DATA_MATROSKA_BLOCKADDITIONAL, &side_data_size); +#if LIBAVUTIL_VERSION_MAJOR < 57 + return base::span(side_data, base::checked_cast(side_data_size)); +#else return base::span(side_data, side_data_size); +#endif } void FFmpegDemuxerStream::EnqueuePacket(ScopedAVPacket packet) { @@ -474,7 +482,11 @@ base::HeapArray::CopiedFrom(side_data); } +#if LIBAVUTIL_VERSION_MAJOR < 57 + int skip_samples_size = 0; +#else size_t skip_samples_size = 0; +#endif const uint32_t* skip_samples_ptr = reinterpret_cast(av_packet_get_side_data( packet.get(), AV_PKT_DATA_SKIP_SAMPLES, &skip_samples_size));