You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by le...@apache.org on 2021/01/31 12:06:37 UTC

svn commit: r1886078 - /pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cmap/CMapParser.java

Author: lehmi
Date: Sun Jan 31 12:06:37 2021
New Revision: 1886078

URL: http://svn.apache.org/viewvc?rev=1886078&view=rev
Log:
PDFBOX-5090: fix issue with bfrange for identity encoding

Modified:
    pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cmap/CMapParser.java

Modified: pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cmap/CMapParser.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cmap/CMapParser.java?rev=1886078&r1=1886077&r2=1886078&view=diff
==============================================================================
--- pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cmap/CMapParser.java (original)
+++ pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cmap/CMapParser.java Sun Jan 31 12:06:37 2021
@@ -407,10 +407,11 @@ public class CMapParser
                     {
                         for (int i = 0; i < 256; i++)
                         {
-                            startCode[1] = (byte) i;
-                            tokenBytes[1] = (byte) i;
-                            addMappingFrombfrange(result, startCode, 0xff, tokenBytes);
-
+                            startCode[0] = (byte) i;
+                            startCode[1] = 0;
+                            tokenBytes[0] = (byte) i;
+                            tokenBytes[1] = 0;
+                            addMappingFrombfrange(result, startCode, 256, tokenBytes);
                         }
                     }
                     else
@@ -439,12 +440,12 @@ public class CMapParser
         {
             String value = createStringFromBytes(tokenBytes);
             cmap.addCharMapping(startCode, value);
-            increment(startCode, startCode.length - 1, false);
             if (!increment(tokenBytes, tokenBytes.length - 1, strictMode))
             {
                 // overflow detected -> stop adding further mappings
                 break;
             }
+            increment(startCode, startCode.length - 1, false);
         }
     }