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

Fix: remove the potential dead store variable in block_based_table_reader.cc (#6204)

Browse Source 
Summary:
buf_offset does not need to get the value from req.len for othe final block. It can cause test fail for clan_analyze. Remove it.
Pull Request resolved: https://github.com/facebook/rocksdb/pull/6204

Test Plan: pass make asan_check

Differential Revision: D19145335

Pulled By: zhichao-cao

fbshipit-source-id: 8f6e74565746381b5c5ef598b97d746517b36e5b
This commit is contained in:
Zhichao Cao 2019-12-18 00:48:48 -08:00 committed by Facebook Github Bot
parent 2afea29762
commit c399704c7a
1 changed files with 0 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
table/block_based/block_based_table_reader.cc
View File

@ -2374,7 +2374,6 @@ void BlockBasedTable::RetrieveMultipleBlocks(
req.scratch = new char[req.len]; req.scratch = new char[req.len];
} else { } else {
req.scratch = scratch + buf_offset; req.scratch = scratch + buf_offset;
buf_offset += req.len;
} }
req.status = IOStatus::OK(); req.status = IOStatus::OK();
read_reqs.emplace_back(req); read_reqs.emplace_back(req);