gdatetime: Avoid integer overflow creating dates too far in the past

oss-fuzz#22758

Signed-off-by: Philip Withnall <pwithnall@endlessos.org>
This commit is contained in:
Philip Withnall 2020-10-01 11:45:22 +01:00
parent 5464c4d292
commit b5656d2524
2 changed files with 11 additions and 3 deletions

View File

@ -1032,7 +1032,8 @@ g_date_time_new_from_unix_local (gint64 t)
GDateTime *datetime;
GTimeZone *local;
if (t > G_MAXINT64 / USEC_PER_SECOND)
if (t > G_MAXINT64 / USEC_PER_SECOND ||
t < G_MININT64 / USEC_PER_SECOND)
return NULL;
local = g_time_zone_new_local ();
@ -1067,7 +1068,8 @@ g_date_time_new_from_unix_utc (gint64 t)
GDateTime *datetime;
GTimeZone *utc;
if (t > G_MAXINT64 / USEC_PER_SECOND)
if (t > G_MAXINT64 / USEC_PER_SECOND ||
t < G_MININT64 / USEC_PER_SECOND)
return NULL;
utc = g_time_zone_new_utc ();

View File

@ -152,7 +152,7 @@ test_GDateTime_new_from_unix (void)
g_date_time_unref (dt);
}
/* Check that trying to create a #GDateTime too far in the future reliably
/* Check that trying to create a #GDateTime too far in the future (or past) reliably
* fails. Previously, the checks for this overflowed and it silently returned
* an incorrect #GDateTime. */
static void
@ -167,6 +167,12 @@ test_GDateTime_new_from_unix_overflow (void)
dt = g_date_time_new_from_unix_local (G_MAXINT64);
g_assert_null (dt);
dt = g_date_time_new_from_unix_utc (G_MININT64);
g_assert_null (dt);
dt = g_date_time_new_from_unix_local (G_MININT64);
g_assert_null (dt);
}
static void