You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by ti...@apache.org on 2024/11/27 16:47:02 UTC

svn commit: r1922174 - /pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/PDFont.java

Author: tilman
Date: Wed Nov 27 16:47:02 2024
New Revision: 1922174

URL: http://svn.apache.org/viewvc?rev=1922174&view=rev
Log:
PDFBOX-5384: use workaround only if /ToUnicode is a stream due to failure with sample_fonts_solidconvertor.pdf

Modified:
    pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/PDFont.java

Modified: pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/PDFont.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/PDFont.java?rev=1922174&r1=1922173&r2=1922174&view=diff
==============================================================================
--- pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/PDFont.java (original)
+++ pdfbox/branches/3.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/PDFont.java Wed Nov 27 16:47:02 2024
@@ -146,7 +146,8 @@ public abstract class PDFont implements
             {
                 String name = getName();
                 LOG.warn("Invalid ToUnicode CMap in font " + name);
-                if (name != null && (name.startsWith("Tahoma") || name.startsWith("Verdana")))
+                if (toUnicode instanceof COSStream && name != null &&
+                        (name.startsWith("Tahoma") || name.startsWith("Verdana")))
                 {
                     // PDFBOX-5384: workaround inspired by PDF.js 15719 and 11242
                     return null;