database/doris
2
0
Fork 0
Code Issues Pull Requests Actions 3 Packages Projects Releases Wiki Activity

branch-2.1: [fix](be) core dump because of invalid bitmap data #53088 (#53935)

Browse Source 
Cherry-picked from #53088

Co-authored-by: JinYang <130516674+gohalo@users.noreply.github.com>
This commit is contained in:
github-actions[bot]
2025-07-27 07:54:23 +08:00
committed by GitHub
parent 159975f237
commit 8869783458
2 changed files with 12 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
be/src/util/bitmap_value.h
View File

@ -1908,7 +1908,12 @@ public:
case BitmapTypeCode::BITMAP64_V2:
_type = BITMAP;
_is_shared = false;
_bitmap = std::make_shared<detail::Roaring64Map>(detail::Roaring64Map::read(src));
try {
_bitmap = std::make_shared<detail::Roaring64Map>(detail::Roaring64Map::read(src));
} catch (const std::runtime_error& e) {
LOG(ERROR) << "Decode roaring bitmap failed, " << e.what();
return false;
}
break;
case BitmapTypeCode::SET: {
_type = SET;

6
be/test/util/bitmap_value_test.cpp
View File

@ -1188,4 +1188,10 @@ TEST(BitmapValueTest, bitmap_value_iterator_test) {
}
}
}
TEST(BitmapValueTest, invalid_data) {
BitmapValue bitmap;
char data[] = {0x02, static_cast<char>(0xff), 0x03};
EXPECT_FALSE(bitmap.deserialize(data));
}
} // namespace doris