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 2021/05/24 16:15:43 UTC
svn commit: r1890166 -
/pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/FileSystemFontProvider.java
Author: tilman
Date: Mon May 24 16:15:43 2021
New Revision: 1890166
URL: http://svn.apache.org/viewvc?rev=1890166&view=rev
Log:
PDFBOX-4892: avoid NPE, as suggested by valerybokov
Modified:
pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/FileSystemFontProvider.java
Modified: pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/FileSystemFontProvider.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/FileSystemFontProvider.java?rev=1890166&r1=1890165&r2=1890166&view=diff
==============================================================================
--- pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/FileSystemFontProvider.java (original)
+++ pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/FileSystemFontProvider.java Mon May 24 16:15:43 2021
@@ -779,7 +779,13 @@ final class FileSystemFontProvider exten
try
{
Type1Font type1 = Type1Font.createWithPFB(input);
- if (type1.getName() != null && type1.getName().contains("|"))
+ if (type1.getName() == null)
+ {
+ fontInfoList.add(new FSIgnored(pfbFile, FontFormat.PFB, "*skipnoname*"));
+ LOG.warn("Missing 'name' entry for PostScript name in font " + pfbFile);
+ return;
+ }
+ if (type1.getName().contains("|"))
{
fontInfoList.add(new FSIgnored(pfbFile, FontFormat.PFB, "*skippipeinname*"));
LOG.warn("Skipping font with '|' in name " + type1.getName() + " in file " + pfbFile);