diff options
author | Jay Berkenbilt <ejb@ql.org> | 2020-11-21 19:12:31 +0100 |
---|---|---|
committer | Jay Berkenbilt <ejb@ql.org> | 2020-11-21 19:43:04 +0100 |
commit | 9d6448157175d8e03a42d6942d4c058b93daf42b (patch) | |
tree | 5fd49e7404b598735128fd02391fc9b0dee2aa17 /libtests | |
parent | 4b4b31bf23daa1479f16ed368316df603fb2407f (diff) | |
download | qpdf-9d6448157175d8e03a42d6942d4c058b93daf42b.tar.zst |
Handle negative numbers in QIntC::range_check (fuzz issue 26994)
Diffstat (limited to 'libtests')
-rw-r--r-- | libtests/qintc.cc | 40 | ||||
-rw-r--r-- | libtests/qtest/qintc/qintc.out | 13 |
2 files changed, 53 insertions, 0 deletions
diff --git a/libtests/qintc.cc b/libtests/qintc.cc index 6b35e837..32c3713f 100644 --- a/libtests/qintc.cc +++ b/libtests/qintc.cc @@ -25,6 +25,29 @@ static void try_convert_real( std::cout << ((passed == exp_pass) ? " PASSED" : " FAILED") << std::endl; } +#define try_range_check(exp_pass, a, b) \ + try_range_check_real(#a " + " #b, exp_pass, a, b) + +template <typename T> +static void try_range_check_real( + char const* description, bool exp_pass, + T const& a, T const& b) +{ + bool passed = false; + try + { + QIntC::range_check(a, b); + std::cout << description << ": okay"; + passed = true; + } + catch (std::range_error& e) + { + std::cout << description << ": " << e.what(); + passed = false; + } + std::cout << ((passed == exp_pass) ? " PASSED" : " FAILED") << std::endl; +} + int main() { uint32_t u1 = 3141592653U; // Too big for signed type @@ -56,5 +79,22 @@ int main() try_convert(true, QIntC::to_uchar<char>, c2); try_convert(true, QIntC::to_char<char>, c2); + auto constexpr max_ll = std::numeric_limits<long long>::max(); + auto constexpr max_ull = std::numeric_limits<unsigned long long>::max(); + auto constexpr min_ll = std::numeric_limits<long long>::min(); + auto constexpr max_sc = std::numeric_limits<signed char>::max(); + try_range_check(true, 1, 2); + try_range_check(true, -1, 2); + try_range_check(true, -100, -200); + try_range_check(true, max_ll, 0LL); + try_range_check(false, max_ll, 1LL); + try_range_check(true, max_ll, 0LL); + try_range_check(false, max_ll, 1LL); + try_range_check(true, max_ull, 0ULL); + try_range_check(false, max_ull, 1ULL); + try_range_check(true, min_ll, 0LL); + try_range_check(false, min_ll, -1LL); + try_range_check(false, max_sc, max_sc); + try_range_check(true, '!', '#'); return 0; } diff --git a/libtests/qtest/qintc/qintc.out b/libtests/qtest/qintc/qintc.out index 2a2ff9f5..5520c635 100644 --- a/libtests/qtest/qintc/qintc.out +++ b/libtests/qtest/qintc/qintc.out @@ -13,3 +13,16 @@ QIntC::to_uchar<int32_t>(i2): 81 Q PASSED QIntC::to_uchar<signed char>(c1): integer out of range converting ÷ from a 1-byte signed type to a 1-byte unsigned type PASSED QIntC::to_uchar<char>(c2): W W PASSED QIntC::to_char<char>(c2): W W PASSED +1 + 2: okay PASSED +-1 + 2: okay PASSED +-100 + -200: okay PASSED +max_ll + 0LL: okay PASSED +max_ll + 1LL: adding 1 to 9223372036854775807 would cause an integer overflow PASSED +max_ll + 0LL: okay PASSED +max_ll + 1LL: adding 1 to 9223372036854775807 would cause an integer overflow PASSED +max_ull + 0ULL: okay PASSED +max_ull + 1ULL: adding 1 to 18446744073709551615 would cause an integer overflow PASSED +min_ll + 0LL: okay PASSED +min_ll + -1LL: adding -1 to -9223372036854775808 would cause an integer underflow PASSED +max_sc + max_sc: adding to would cause an integer overflow PASSED +'!' + '#': okay PASSED |