From b4d60ba1367f15843577d4363b32fb16847b9582 Mon Sep 17 00:00:00 2001 From: Alexander Kanavin Date: Tue, 22 Aug 2023 19:57:48 +0200 Subject: [PATCH] glib/gfileutils.c: use 64 bits for value in get_tmp_file() On 32 bit systems 'long' value will overflow in 2038 and become negative. As it is used to index into letters array, and % operation preserves signs, data corruption will then occur. Signed-off-by: Alexander Kanavin --- glib/gfileutils.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/glib/gfileutils.c b/glib/gfileutils.c index 4ed8171cc..205e517b6 100644 --- a/glib/gfileutils.c +++ b/glib/gfileutils.c @@ -1500,7 +1500,7 @@ get_tmp_file (gchar *tmpl, static const char letters[] = "ABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789"; static const int NLETTERS = sizeof (letters) - 1; - glong value; + gint64 value; gint64 now_us; static int counter = 0; @@ -1521,7 +1521,7 @@ get_tmp_file (gchar *tmpl, for (count = 0; count < 100; value += 7777, ++count) { - glong v = value; + gint64 v = value; /* Fill in the random bits. */ XXXXXX[0] = letters[v % NLETTERS];