ICEpdf
  1. ICEpdf
  2. PDF-839

Missing characters (T, A, t) in PDF

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 5.1.1
          • Fix Version/s: 5.1.2
          • Component/s: Core/Parsing
          • Labels:
            None
          • Environment:
            All

            Description

            With the provided PDF file, it is rendering with missing characters (T, A, and t for example).

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              Patrick Corless added a comment -

              The root of the problem is related to the LiteralStringObject.getLiteralStringBuffer() method. The method is responsible converting the underlying string data to a string that can be drawn. In this particular case the font is of CID format which mean we check if each glyph can be represented. by font. We also check to see of the character ID is greater then one which is the root of the problem as the CID glyph id for T is zero in this case.

              The code block in question has a comment that glyphs should be zero or more. I do remember a few corner cases where zero character were not valid. I'll have to run a potential fix though qa to see what happens as this is a high risk change.

              Show
              Patrick Corless added a comment - The root of the problem is related to the LiteralStringObject.getLiteralStringBuffer() method. The method is responsible converting the underlying string data to a string that can be drawn. In this particular case the font is of CID format which mean we check if each glyph can be represented. by font. We also check to see of the character ID is greater then one which is the root of the problem as the CID glyph id for T is zero in this case. The code block in question has a comment that glyphs should be zero or more. I do remember a few corner cases where zero character were not valid. I'll have to run a potential fix though qa to see what happens as this is a high risk change.
              Hide
              Permalink
              Patrick Corless added a comment -

              Applied fix to trunk as well.

              Show
              Patrick Corless added a comment - Applied fix to trunk as well.
              Hide
              Permalink
              Patrick Corless added a comment -

              Fix passed QA, marking as resolved.

              Show
              Patrick Corless added a comment - Fix passed QA, marking as resolved.

                People

                • Assignee:
                  Patrick Corless
                  Reporter:
                  Arran Mccullough
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  2 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: