freetype-commit
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[freetype2] master 3774fc0: * src/psaux/psobjs.c (t1_builder_close_conto


From: Werner LEMBERG
Subject: [freetype2] master 3774fc0: * src/psaux/psobjs.c (t1_builder_close_contour): Add safety guard.
Date: Sun, 26 Mar 2017 02:33:18 -0400 (EDT)

branch: master
commit 3774fc08b502c3e685afca098b6e8a195aded6a0
Author: Werner Lemberg <address@hidden>
Commit: Werner Lemberg <address@hidden>

    * src/psaux/psobjs.c (t1_builder_close_contour): Add safety guard.
    
    Reported as
    
      https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=941
---
 ChangeLog          | 8 ++++++++
 src/psaux/psobjs.c | 8 ++++++++
 2 files changed, 16 insertions(+)

diff --git a/ChangeLog b/ChangeLog
index 01e9a7b..3d782e9 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,11 @@
+2017-03-26  Werner Lemberg  <address@hidden>
+
+       * src/psaux/psobjs.c (t1_builder_close_contour): Add safety guard.
+
+       Reported as
+
+         https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=941
+
 2017-03-23  Werner Lemberg  <address@hidden>
 
        [psaux] Better protect `flex' handling.
diff --git a/src/psaux/psobjs.c b/src/psaux/psobjs.c
index d18e821..0baf836 100644
--- a/src/psaux/psobjs.c
+++ b/src/psaux/psobjs.c
@@ -1718,6 +1718,14 @@
     first = outline->n_contours <= 1
             ? 0 : outline->contours[outline->n_contours - 2] + 1;
 
+    /* in malformed fonts it can happen that a contour was started */
+    /* but no points were added                                    */
+    if ( outline->n_contours && first == outline->n_points )
+    {
+      outline->n_contours--;
+      return;
+    }
+
     /* We must not include the last point in the path if it */
     /* is located on the first point.                       */
     if ( outline->n_points > 1 )



reply via email to

[Prev in Thread] Current Thread [Next in Thread]