Fix ByteEncodingCMapTable.CharacterCodeToGlyphIndex() to account for …

…glyph mapping length, add test and fix #881
UglyToad · Sep 29, 2024 · 8ce6bcc · 8ce6bcc
1 parent ad78532
commit 8ce6bcc
Show file tree

Hide file tree

Showing 3 changed files with 25 additions and 1 deletion.
diff --git a/src/UglyToad.PdfPig.Fonts/TrueType/Tables/CMapSubTables/ByteEncodingCMapTable.cs b/src/UglyToad.PdfPig.Fonts/TrueType/Tables/CMapSubTables/ByteEncodingCMapTable.cs
@@ -46,7 +46,7 @@ public static ByteEncodingCMapTable Load(TrueTypeDataBytes data, TrueTypeCMapPla
 
         public int CharacterCodeToGlyphIndex(int characterCode)
         {
-            if (characterCode < 0 || characterCode >= GlyphMappingLength)
+            if (characterCode < 0 || characterCode >= glyphMapping.Length)
             {
                 return 0;
             }

diff --git a/src/UglyToad.PdfPig.Tests/Integration/Documents/issue_881.pdf b/src/UglyToad.PdfPig.Tests/Integration/Documents/issue_881.pdf
diff --git a/src/UglyToad.PdfPig.Tests/Integration/TrueTypeTests.cs b/src/UglyToad.PdfPig.Tests/Integration/TrueTypeTests.cs
@@ -0,0 +1,24 @@
+namespace UglyToad.PdfPig.Tests.Integration
+{
+    using System.Linq;
+
+    public class TrueTypeTests
+    {
+        [Fact]
+        public void Issue881()
+        {
+            var file = IntegrationHelpers.GetDocumentPath("issue_881.pdf");
+
+            using (var document = PdfDocument.Open(file))
+            {
+                var page = document.GetPage(1);
+                var words = page.GetWords().ToArray();
+                Assert.Equal(4, words.Length);
+                Assert.Equal("IDNR:", words[0].Text);
+                Assert.Equal("4174", words[1].Text);
+                Assert.Equal("/", words[2].Text);
+                Assert.Equal("06.08.2018", words[3].Text);
+            }
+        }
+    }
+}