coffee/fmt
1
0
Fork 0
mirror of https://github.com/fmtlib/fmt.git synced 2026-06-15 08:26:13 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix out-of-range float to int conversion in to_nonnegative_int

Browse Source
This commit is contained in:
aizu-m 2026-06-10 18:03:35 +05:30
parent e8deaf2ec3
commit 7bd6c1b94c
2 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
include/fmt/chrono.h
View File

@ -934,10 +934,9 @@ inline auto to_nonnegative_int(T value, Int upper) -> Int {
}
template <typename T, typename Int, FMT_ENABLE_IF(!std::is_integral<T>::value)>
inline auto to_nonnegative_int(T value, Int upper) -> Int {
auto int_value = static_cast<Int>(value);
if (int_value < 0 || value > static_cast<T>(upper))
if (value < 0 || value >= static_cast<T>(upper) + 1)
FMT_THROW(format_error("invalid value"));
return int_value;
return static_cast<Int>(value);
}
constexpr auto pow10(std::uint32_t n) -> long long {

3
test/chrono-test.cc
View File

@ -989,6 +989,9 @@ TEST(chrono_test, glibc_extensions) {
TEST(chrono_test, out_of_range) {
auto d = std::chrono::duration<unsigned long, std::giga>(538976288);
EXPECT_THROW((void)fmt::format("{:%j}", d), fmt::format_error);
// A floating-point day count that doesn't fit in int.
auto fd = std::chrono::duration<double>(1e300);
EXPECT_THROW((void)fmt::format("{:%j}", fd), fmt::format_error);
}
TEST(chrono_test, year_month_day) {