Commit 1d355440 authored by Maksim Ivanov's avatar Maksim Ivanov Committed by Commit Bot

Fix use-after-move in //components/web_package/

Fix use-after-move (potential) bugs found by the
"bugprone-use-after-move" clang-tidy check.

Bug: 1122844
Change-Id: Ic432627efc292b958b9d30dc3e8886aee02c2b4f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2382832Reviewed-by: default avatarKunihiko Sakamoto <ksakamoto@chromium.org>
Commit-Queue: Maksim Ivanov <emaxx@chromium.org>
Cr-Commit-Position: refs/heads/master@{#802996}
parent 1ceb74be
...@@ -45,8 +45,10 @@ class TestDataSource : public mojom::BundleDataSource { ...@@ -45,8 +45,10 @@ class TestDataSource : public mojom::BundleDataSource {
: data_(reinterpret_cast<const char*>(data.data()), data.size()) {} : data_(reinterpret_cast<const char*>(data.data()), data.size()) {}
void Read(uint64_t offset, uint64_t length, ReadCallback callback) override { void Read(uint64_t offset, uint64_t length, ReadCallback callback) override {
if (offset >= data_.size()) if (offset >= data_.size()) {
std::move(callback).Run(base::nullopt); std::move(callback).Run(base::nullopt);
return;
}
const uint8_t* start = const uint8_t* start =
reinterpret_cast<const uint8_t*>(data_.data()) + offset; reinterpret_cast<const uint8_t*>(data_.data()) + offset;
uint64_t available_length = std::min(length, data_.size() - offset); uint64_t available_length = std::min(length, data_.size() - offset);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment