Skip to content
体验新版
项目
组织
正在加载...
登录
切换导航
打开侧边栏
2dot5
ClickHouse
提交
8212e32d
C
ClickHouse
项目概览
2dot5
/
ClickHouse
通知
3
Star
0
Fork
0
代码
文件
提交
分支
Tags
贡献者
分支图
Diff
Issue
0
列表
看板
标记
里程碑
合并请求
0
DevOps
流水线
流水线任务
计划
Wiki
0
Wiki
分析
仓库
DevOps
项目成员
Pages
C
ClickHouse
项目概览
项目概览
详情
发布
仓库
仓库
文件
提交
分支
标签
贡献者
分支图
比较
Issue
0
Issue
0
列表
看板
标记
里程碑
合并请求
0
合并请求
0
Pages
DevOps
DevOps
流水线
流水线任务
计划
分析
分析
仓库分析
DevOps
Wiki
0
Wiki
成员
成员
收起侧边栏
关闭侧边栏
动态
分支图
创建新Issue
流水线任务
提交
Issue看板
体验新版 GitCode,发现更多精彩内容 >>
未验证
提交
8212e32d
编写于
1月 23, 2021
作者:
A
alexey-milovidov
提交者:
GitHub
1月 23, 2021
浏览文件
操作
浏览文件
下载
差异文件
Merge pull request #19508 from ClickHouse/backport/20.12/19472
Backport #19472 to 20.12: Fix buffer overflow in DateLUT
上级
35b50a1b
b4ef8573
变更
3
隐藏空白更改
内联
并排
Showing
3 changed file
with
6 addition
and
1 deletion
+6
-1
base/common/DateLUTImpl.h
base/common/DateLUTImpl.h
+5
-1
tests/queries/0_stateless/01666_date_lut_buffer_overflow.reference
...ries/0_stateless/01666_date_lut_buffer_overflow.reference
+0
-0
tests/queries/0_stateless/01666_date_lut_buffer_overflow.sql
tests/queries/0_stateless/01666_date_lut_buffer_overflow.sql
+1
-0
未找到文件。
base/common/DateLUTImpl.h
浏览文件 @
8212e32d
...
...
@@ -229,8 +229,12 @@ public:
inline
UInt8
daysInMonth
(
UInt16
year
,
UInt8
month
)
const
{
UInt16
idx
=
year
-
DATE_LUT_MIN_YEAR
;
if
(
unlikely
(
idx
>=
DATE_LUT_YEARS
))
return
31
;
/// Implementation specific behaviour on overflow.
/// 32 makes arithmetic more simple.
DayNum
any_day_of_month
=
DayNum
(
years_lut
[
year
-
DATE_LUT_MIN_YEAR
]
+
32
*
(
month
-
1
));
DayNum
any_day_of_month
=
DayNum
(
years_lut
[
idx
]
+
32
*
(
month
-
1
));
return
lut
[
any_day_of_month
].
days_in_month
;
}
...
...
tests/queries/0_stateless/01666_date_lut_buffer_overflow.reference
0 → 100644
浏览文件 @
8212e32d
tests/queries/0_stateless/01666_date_lut_buffer_overflow.sql
0 → 100644
浏览文件 @
8212e32d
SELECT
toDate
(
'2105-12-31'
)
+
INTERVAL
number
MONTH
FROM
system
.
numbers
LIMIT
25000
FORMAT
Null
;
编辑
预览
Markdown
is supported
0%
请重试
或
添加新附件
.
添加附件
取消
You are about to add
0
people
to the discussion. Proceed with caution.
先完成此消息的编辑!
取消
想要评论请
注册
或
登录