8526b3
From 24ddcfc7e37c0ce3b0f1852042ee431a53fd774c Mon Sep 17 00:00:00 2001
97dcb9
From: Robin Watts <Robin.Watts@artifex.com>
97dcb9
Date: Mon, 27 Jan 2020 10:12:24 -0800
97dcb9
Subject: [PATCH] Fix OSS-Fuzz issue 20332: buffer overflow in
97dcb9
 jbig2_image_compose.
97dcb9
97dcb9
With extreme values of x/y/w/h we can get overflow. Test for this
97dcb9
and exit safely.
97dcb9
97dcb9
Thanks for OSS-Fuzz for reporting.
97dcb9
---
97dcb9
 jbig2_image.c | 13 +++++++++++++
97dcb9
 1 file changed, 13 insertions(+)
97dcb9
97dcb9
diff --git a/jbig2_image.c b/jbig2_image.c
8526b3
index 22e21ef..f036cef 100644
97dcb9
--- a/jbig2_image.c
97dcb9
+++ b/jbig2_image.c
8526b3
@@ -34,6 +34,10 @@
8526b3
 #define INT32_MAX  0x7fffffff
8526b3
 #endif
97dcb9
 
97dcb9
+#if !defined (UINT32_MAX)
97dcb9
+#define UINT32_MAX  0xffffffffu
97dcb9
+#endif
97dcb9
+
97dcb9
 /* allocate a Jbig2Image structure and its associated bitmap */
97dcb9
 Jbig2Image *
97dcb9
 jbig2_image_new(Jbig2Ctx *ctx, uint32_t width, uint32_t height)
8526b3
@@ -255,6 +259,15 @@ jbig2_image_compose(Jbig2Ctx *ctx, Jbig2Image *dst, Jbig2Image *src, int x, int
97dcb9
     uint8_t *d, *dd;
97dcb9
     uint8_t mask, rightmask;
97dcb9
 
97dcb9
+    if ((UINT32_MAX - src->width  < (x > 0 ? x : -x)) ||
97dcb9
+        (UINT32_MAX - src->height < (y > 0 ? y : -y)))
97dcb9
+    {
97dcb9
+#ifdef JBIG2_DEBUG
97dcb9
+        jbig2_error(ctx, JBIG2_SEVERITY_DEBUG, -1, "overflow in compose_image");
97dcb9
+#endif
97dcb9
+        return 0;
97dcb9
+    }
97dcb9
+
8526b3
     if (src == NULL)
8526b3
         return 0;
8526b3
 
97dcb9
-- 
97dcb9
2.26.2
97dcb9