andreacavalli/rocksdb
0
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Initialize scratch to nullptr explicitly to make clang analyzer happy (#6577)

Browse Source 
Summary:
`scratch` is not initialized in `Align` because it will be set outside of it. But clang analyzer is strict on initializing it before return.
Pull Request resolved: https://github.com/facebook/rocksdb/pull/6577

Test Plan: make analyze

Reviewed By: siying

Differential Revision: D20607303

Pulled By: cheng-chang

fbshipit-source-id: 2843d759345a057a8e122178d30b90deff0f9b2a
This commit is contained in:
Cheng Chang 2020-03-23 20:12:38 -07:00 committed by Facebook GitHub Bot
parent d300d10962
commit 43aee93d2b
1 changed files with 1 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
file/random_access_file_reader.cc
View File

@ -163,6 +163,7 @@ FSReadRequest Align(const FSReadRequest& r, size_t alignment) {
req.offset = static_cast<uint64_t>(
TruncateToPageBoundary(alignment, static_cast<size_t>(r.offset)));
req.len = Roundup(End(r), alignment) - req.offset;
req.scratch = nullptr;
return req;
}