diff options
author | Andrew <andrewlassalle@chromium.org> | 2020-03-26 13:40:37 -0700 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2020-03-28 17:20:31 +0000 |
commit | 9d5a61d7c71beb655e67eaf5c685e7add8f7768d (patch) | |
tree | 153e89c077acd3d77b638c8fb89bc2a091daa911 /payload_generator/deflate_utils.cc | |
parent | 48ec4c85c00fc7be778dd2d1ce37d8f2908f8f17 (diff) |
update_engine: Fix issues reported by clang-tidy
Fix uninitialized variables and some other minor issues reported by
clang-tidy.
BUG=chromium:982837
TEST=cros_workon_make update_engine --test
Change-Id: I305dedb058c9b0787ba2f68feff42afe6810a276
Reviewed-on: https://chromium-review.googlesource.com/c/aosp/platform/system/update_engine/+/2122683
Reviewed-by: Amin Hassani <ahassani@chromium.org>
Tested-by: Andrew Lassalle <andrewlassalle@chromium.org>
Commit-Queue: Andrew Lassalle <andrewlassalle@chromium.org>
Diffstat (limited to 'payload_generator/deflate_utils.cc')
-rw-r--r-- | payload_generator/deflate_utils.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/payload_generator/deflate_utils.cc b/payload_generator/deflate_utils.cc index 8db67ce0..5d7a766a 100644 --- a/payload_generator/deflate_utils.cc +++ b/payload_generator/deflate_utils.cc @@ -46,7 +46,7 @@ const uint64_t kMinimumSquashfsImageSize = 1 * 1024 * 1024; // bytes // TODO(*): Optimize this so we don't have to read all extents into memory in // case it is large. bool CopyExtentsToFile(const string& in_path, - const vector<Extent> extents, + const vector<Extent>& extents, const string& out_path, size_t block_size) { brillo::Blob data(utils::BlocksInExtents(extents) * block_size); |