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

Add bounds check in FilePickerMultiGet::PrepareNextLevel() (#5189)

Browse Source 
Summary:
Add bounds check when looping through empty levels in FilePickerMultiGet
Pull Request resolved: https://github.com/facebook/rocksdb/pull/5189

Differential Revision: D14925334

Pulled By: anand1976

fbshipit-source-id: 65d53247cf443153e28ce2b8b753fa51c6ae4566
This commit is contained in:
anand76 2019-04-12 18:01:00 -07:00 committed by Facebook Github Bot
parent cca141ecf8
commit 29111e92b4
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
db/version_set.cc
View File

@ -672,8 +672,11 @@ class FilePickerMultiGet {
fp_ctx.search_right_bound = FileIndexer::kLevelMaxIndex; fp_ctx.search_right_bound = FileIndexer::kLevelMaxIndex;
} }
// Skip all subsequent empty levels // Skip all subsequent empty levels
while ((*level_files_brief_)[++curr_level_].num_files == 0) { do {
} ++curr_level_;
} while ((curr_level_ < num_levels_) &&
(*level_files_brief_)[curr_level_].num_files == 0);
continue;
} }
// Some files may overlap each other. We find // Some files may overlap each other. We find