ICEpdf
  1. ICEpdf
  2. PDF-354

Font engine not correctly handling CID and GID values that differ

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 4.2.2
    • Fix Version/s: 4.3
    • Component/s: Font Engine
    • Labels:
      None
    • Environment:
      PRO

      Description

      A client sent in a set of PDFs that had embedded font programs but the font engine could not correctly display the glyphs. The glyphs where asian and the font was a CID font with with CIDtoGID that wasn't the standard identity mapping. The font engine and the respective Cmap is incorrectly initialized making it impossible to correctly map a none identity cmap.

      For example consider the cid's below, the current algorithm that organizes and compresses the CID -> GID makes it imposible to retreive the GID value for a given CID because CID and GID are not equal.

      CID GID
      0x0704 = 1796 14
      0x0db3 = 3507 24
      0x08ba = 2234 5
      ox0d15 = 3349 53

        Activity

        Hide
        Patrick Corless added a comment -

        Quite a bit of work was done to address the cid to gid mapping error. QA passed and there are quite a few other documents that had the same problem that now render correctly.

        Marking as resolved.

        Show
        Patrick Corless added a comment - Quite a bit of work was done to address the cid to gid mapping error. QA passed and there are quite a few other documents that had the same problem that now render correctly. Marking as resolved.
        Hide
        Patrick Corless added a comment -

        After a round a QA there where a few remaining issues around how StringObject handle there 2 and 4 bytes cids. Pretty sure I've go this all straightened out for now but will do one more round of qa to be sure.

        Show
        Patrick Corless added a comment - After a round a QA there where a few remaining issues around how StringObject handle there 2 and 4 bytes cids. Pretty sure I've go this all straightened out for now but will do one more round of qa to be sure.

          People

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

            Dates

            • Created:
              Updated:
              Resolved: