Commit 3e8a2283f5119376098a0ec66f876866ff5cea4d

Werner Lemberg 2018-12-11T23:54:17

* src/truetype/ttgload.c (TT_Hint_Glyph): Remove useless test. `control_len' only gets its value from `n_ins' (and vice versa), which is always read as `unsigned short' and thus can't be larger than 0xFFFF.

diff --git a/ChangeLog b/ChangeLog
index d6946b8..4ad8e5c 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+2018-12-11  Werner Lemberg  <wl@gnu.org>
+
+	* src/truetype/ttgload.c (TT_Hint_Glyph): Remove useless test.
+
+	`control_len' only gets its value from `n_ins' (and vice versa),
+	which is always read as `unsigned short' and thus can't be larger
+	than 0xFFFF.
+
 2018-12-04  Werner Lemberg  <wl@gnu.org>
 
 	[bdf] Ignore data after `ENDFONT'.
diff --git a/src/truetype/ttgload.c b/src/truetype/ttgload.c
index 1d84450..ec0d4fc 100644
--- a/src/truetype/ttgload.c
+++ b/src/truetype/ttgload.c
@@ -816,12 +816,6 @@
 
 
 #ifdef TT_USE_BYTECODE_INTERPRETER
-    if ( loader->glyph->control_len > 0xFFFFL )
-    {
-      FT_TRACE1(( "TT_Hint_Glyph: too long instructions" ));
-      FT_TRACE1(( " (0x%lx byte) is truncated\n",
-                  loader->glyph->control_len ));
-    }
     n_ins = loader->glyph->control_len;
 
     /* save original point position in org */