summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorWerner Lemberg <wl@gnu.org>2017-07-03 06:27:52 +0200
committerWerner Lemberg <wl@gnu.org>2017-07-03 06:27:52 +0200
commitca799e9be52526739691f120285a27b91fd15d68 (patch)
tree0c330fc35706def34a35c0d660325f39a0ebaf26
parentabeb28f161bc11be4fcdb8af50ab4b736d34a3e5 (diff)
downloadfreetype2-ca799e9be52526739691f120285a27b91fd15d68.tar.gz
[truetype] Integer overflow.
Reported as https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=2455 * src/truetype/ttinterp.c (Ins_SCFS): Use SUB_LONG.
-rw-r--r--ChangeLog10
-rw-r--r--src/truetype/ttinterp.c2
2 files changed, 11 insertions, 1 deletions
diff --git a/ChangeLog b/ChangeLog
index 30cf6304a..c6e8229af 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,13 @@
+2017-07-03 Werner Lemberg <wl@gnu.org>
+
+ [truetype] Integer overflow.
+
+ Reported as
+
+ https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=2455
+
+ * src/truetype/ttinterp.c (Ins_SCFS): Use SUB_LONG.
+
2017-07-01 Alexei Podtelezhnikov <apodtele@gmail.com>
* src/sfnt/sfobjs.c (sfnt_load_face): Ignore No_Unicode_Glyph_Name.
diff --git a/src/truetype/ttinterp.c b/src/truetype/ttinterp.c
index 24318bf1b..f8bf4af71 100644
--- a/src/truetype/ttinterp.c
+++ b/src/truetype/ttinterp.c
@@ -4803,7 +4803,7 @@
K = FAST_PROJECT( &exc->zp2.cur[L] );
- exc->func_move( exc, &exc->zp2, L, args[1] - K );
+ exc->func_move( exc, &exc->zp2, L, SUB_LONG( args[1], K ) );
/* UNDOCUMENTED! The MS rasterizer does that with */
/* twilight points (confirmed by Greg Hitchcock) */