Blame SOURCES/0566-font-Fix-integer-overflow-in-ensure_comb_space.patch

235a57
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
235a57
From: Zhang Boyang <zhangboyang.id@gmail.com>
235a57
Date: Fri, 5 Aug 2022 02:27:05 +0800
235a57
Subject: [PATCH] font: Fix integer overflow in ensure_comb_space()
235a57
235a57
In fact it can't overflow at all because glyph_id->ncomb is only 8-bit
235a57
wide. But let's keep safe if somebody changes the width of glyph_id->ncomb
235a57
in the future. This patch also fixes the inconsistency between
235a57
render_max_comb_glyphs and render_combining_glyphs when grub_malloc()
235a57
returns NULL.
235a57
235a57
Signed-off-by: Zhang Boyang <zhangboyang.id@gmail.com>
235a57
Reviewed-by: Daniel Kiper <daniel.kiper@oracle.com>
235a57
(cherry picked from commit b2740b7e4a03bb8331d48b54b119afea76bb9d5f)
235a57
(cherry picked from commit f66ea1e60c347408e92b6695d5105c7e0f24d568)
235a57
(cherry picked from commit 0e07159c24cdbb62a9d19fba8199065b049e03c7)
235a57
---
235a57
 grub-core/font/font.c | 14 +++++++++-----
235a57
 1 file changed, 9 insertions(+), 5 deletions(-)
235a57
235a57
diff --git a/grub-core/font/font.c b/grub-core/font/font.c
235a57
index 347e9dfa29..1367e44743 100644
235a57
--- a/grub-core/font/font.c
235a57
+++ b/grub-core/font/font.c
235a57
@@ -1468,14 +1468,18 @@ ensure_comb_space (const struct grub_unicode_glyph *glyph_id)
235a57
   if (glyph_id->ncomb <= render_max_comb_glyphs)
235a57
     return;
235a57
 
235a57
-  render_max_comb_glyphs = 2 * glyph_id->ncomb;
235a57
-  if (render_max_comb_glyphs < 8)
235a57
+  if (grub_mul (glyph_id->ncomb, 2, &render_max_comb_glyphs))
235a57
+    render_max_comb_glyphs = 0;
235a57
+  if (render_max_comb_glyphs > 0 && render_max_comb_glyphs < 8)
235a57
     render_max_comb_glyphs = 8;
235a57
   grub_free (render_combining_glyphs);
235a57
-  render_combining_glyphs = grub_malloc (render_max_comb_glyphs
235a57
-					 * sizeof (render_combining_glyphs[0]));
235a57
+  render_combining_glyphs = (render_max_comb_glyphs > 0) ?
235a57
+    grub_calloc (render_max_comb_glyphs, sizeof (render_combining_glyphs[0])) : NULL;
235a57
   if (!render_combining_glyphs)
235a57
-    grub_errno = 0;
235a57
+    {
235a57
+      render_max_comb_glyphs = 0;
235a57
+      grub_errno = GRUB_ERR_NONE;
235a57
+    }
235a57
 }
235a57
 
235a57
 int