diff options
author | Tim Blechmann <[email protected]> | 2025-07-04 15:42:26 +0800 |
---|---|---|
committer | Tim Blechmann <[email protected]> | 2025-07-04 11:39:13 +0000 |
commit | cb0c0b68669e08c8372cdd8e4e8a2a4d2d410db5 (patch) | |
tree | 694a42d8089d3a171530452a884c4367a91c7007 | |
parent | f0286fe28418c5db2b0c07f525ae20dd2344dd5c (diff) |
The buffer returned by av_dict_get_string needs to be freed with
av_free, not std::free.
Caught by address sanitizer on windows.
Pick-to: 6.8 6.9 6.10
Change-Id: I2224fa3e2405a26aed1eedfb93d02f3b6ec9f531
Reviewed-by: Mikko Hallamaa <[email protected]>
Reviewed-by: Tim Blechmann <[email protected]>
Reviewed-by: Nils Petter Skålerud <[email protected]>
-rw-r--r-- | src/plugins/multimedia/ffmpeg/qffmpeg.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/plugins/multimedia/ffmpeg/qffmpeg.cpp b/src/plugins/multimedia/ffmpeg/qffmpeg.cpp index 11e8850a6..596af78cb 100644 --- a/src/plugins/multimedia/ffmpeg/qffmpeg.cpp +++ b/src/plugins/multimedia/ffmpeg/qffmpeg.cpp @@ -346,7 +346,7 @@ QDebug operator<<(QDebug dbg, const AVDictionary &dict) { char *buffer = 0; auto freeBuffer = QScopeGuard([&] { - std::free(buffer); + av_free(buffer); }); int status = av_dict_get_string(&dict, &buffer, '=', ','); |