From d8e3555e015da54dcb09b640540a8529e59d8fe5 Mon Sep 17 00:00:00 2001 From: "trop[bot]" <37223003+trop[bot]@users.noreply.github.com> Date: Wed, 25 Sep 2024 11:21:44 +0200 Subject: [PATCH] fix: -Wunsafe-buffer-usage warning in asar_util's ReadFileToString() (#43929) Co-authored-by: trop[bot] <37223003+trop[bot]@users.noreply.github.com> Co-authored-by: Charles Kerr --- shell/common/asar/asar_util.cc | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/shell/common/asar/asar_util.cc b/shell/common/asar/asar_util.cc index 0d3ebb33eb2f..c5436cf07b77 100644 --- a/shell/common/asar/asar_util.cc +++ b/shell/common/asar/asar_util.cc @@ -127,11 +127,8 @@ bool ReadFileToString(const base::FilePath& path, std::string* contents) { return false; contents->resize(info.size); - if (static_cast(info.size) != - src.Read(info.offset, const_cast(contents->data()), - contents->size())) { + if (!src.ReadAndCheck(info.offset, base::as_writable_byte_span(*contents))) return false; - } if (info.integrity) ValidateIntegrityOrDie(base::as_byte_span(*contents), *info.integrity);