Fix UB - signed integer overflow in regex. Thanks to Tim Shen for the patch. Reviewed as https://reviews.llvm.org/D39066

llvm-svn: 316172
This commit is contained in:
Marshall Clow 2017-10-19 17:39:16 +00:00
parent ea35e46b71
commit 52f0885d43
2 changed files with 42 additions and 0 deletions

View File

@ -4064,6 +4064,8 @@ basic_regex<_CharT, _Traits>::__parse_DUP_COUNT(_ForwardIterator __first,
__first != __last && ( __val = __traits_.value(*__first, 10)) != -1;
++__first)
{
if (__c >= std::numeric_limits<int>::max() / 10)
__throw_regex_error<regex_constants::error_badbrace>();
__c *= 10;
__c += __val;
}

View File

@ -0,0 +1,40 @@
//===----------------------------------------------------------------------===//
//
// The LLVM Compiler Infrastructure
//
// This file is dual licensed under the MIT and the University of Illinois Open
// Source Licenses. See LICENSE.TXT for details.
//
//===----------------------------------------------------------------------===//
// <regex>
// UNSUPPORTED: libcpp-no-exceptions
// UNSUPPORTED: c++03
// the "n" in `a{n}` should be within the numeric limits.
#include <regex>
#include <cassert>
int main() {
for (std::regex_constants::syntax_option_type op :
{std::regex::basic, std::regex::grep}) {
try {
(void)std::regex("a\\{100000000000000000\\}", op);
assert(false);
} catch (const std::regex_error &e) {
assert(e.code() == std::regex_constants::error_badbrace);
}
}
for (std::regex_constants::syntax_option_type op :
{std::regex::ECMAScript, std::regex::extended, std::regex::egrep,
std::regex::awk}) {
try {
(void)std::regex("a{100000000000000000}", op);
assert(false);
} catch (const std::regex_error &e) {
assert(e.code() == std::regex_constants::error_badbrace);
}
}
return 0;
}