thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-11-21 15:12:02 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #22099 from ClickHouse/map-populate-series-ubsan

Browse Source 
Fix UBSan report in mapPopulateSeries.
This commit is contained in:
alexey-milovidov 2021-03-25 16:00:21 +03:00 committed by GitHub
commit cb6926629e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/Functions/array/mapPopulateSeries.cpp
View File

@ -190,7 +190,7 @@ private:
}
static constexpr size_t MAX_ARRAY_SIZE = 1ULL << 30;
if (static_cast<size_t>(max_key - min_key) > MAX_ARRAY_SIZE)
if (static_cast<size_t>(max_key) - static_cast<size_t>(min_key) > MAX_ARRAY_SIZE)
throw Exception(ErrorCodes::TOO_LARGE_ARRAY_SIZE, "Too large array size in the result of function {}", getName());
/* fill the result arrays */

0
tests/queries/0_stateless/01777_map_populate_series_ubsan.reference Normal file
View File

2
tests/queries/0_stateless/01777_map_populate_series_ubsan.sql Normal file
View File

@ -0,0 +1,2 @@
-- Should correctly throw exception about overflow:
SELECT mapPopulateSeries([-9223372036854775808, toUInt32(2)], [toUInt32(1023), -1]); -- { serverError 128 }