Attachment 139780 Details for Bug 50750 – drop invalid names from encoding table

[patch] drop invalid names from encoding table

0001-GfxFont-Reject-invalid-names-from-encoding-table.patch (text/plain), 1.81 KB, created by Stefan Brüns on 2018-05-26 00:20:03 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Stefan Brüns

Created: 2018-05-26 00:20:03 UTC

Size: 1.81 KB

patch

obsolete

>From 62e5e9525c0dd39856c5157452fd2308e8192603 Mon Sep 17 00:00:00 2001
>From: =?UTF-8?q?Stefan=20Br=C3=BCns?= <stefan.bruens@rwth-aachen.de>
>Date: Thu, 17 May 2018 02:35:32 +0200
>Subject: [PATCH] GfxFont: Reject invalid names from encoding table
>
>Some PDF files contain invalid glyph names in the Encoding /Differences
>array. Putting these e.g. in a Type42 /Encoding array when converting it
>to postscript creates an invalid file, rejected by e.g. ghostscript.
>---
> poppler/GfxFont.cc | 21 ++++++++++++++++++++-
> 1 file changed, 20 insertions(+), 1 deletion(-)
>
>diff --git a/poppler/GfxFont.cc b/poppler/GfxFont.cc
>index 7b4d4b54..9fb2bf24 100644
>--- a/poppler/GfxFont.cc
>+++ b/poppler/GfxFont.cc
>@@ -944,6 +944,25 @@ static GBool testForNumericNames(Dict *fontDict, GBool hex) {
>   return numeric;
> }
> 
>+// Return gTrue if the name is valid, according to
>+// https://github.com/adobe-type-tools/agl-specification
>+static GBool glyphNameIsValid(const char *name)
>+{
>+  // Valid characters: [a-zA-Z0-9_.]
>+  // The AGL spec has more requirements, but this breaks some documents
>+  while (*name != '\0') {
>+    if (islower(*name) || isdigit(*name) ||
>+        isupper(*name) || (*name == '_') ||
>+        (*name == '.')) {
>+      name++;
>+    } else {
>+      return gFalse;
>+    }
>+  }
>+
>+  return gTrue;
>+}
>+
> Gfx8BitFont::Gfx8BitFont(XRef *xref, const char *tagA, Ref idA, GooString *nameA,
> 			 GfxFontType typeA, Ref embFontIDA, Dict *fontDict):
>   GfxFont(tagA, idA, nameA, typeA, embFontIDA) {
>@@ -1221,7 +1240,7 @@ Gfx8BitFont::Gfx8BitFont(XRef *xref, const char *tagA, Ref idA, GooString *nameA
> 	if (obj3.isInt()) {
> 	  code = obj3.getInt();
> 	} else if (obj3.isName()) {
>-	  if (code >= 0 && code < 256) {
>+	  if (glyphNameIsValid(obj3.getName()) && code >= 0 && code < 256) {
> 	    if (encFree[code]) {
> 	      gfree(enc[code]);
> 	    }
>-- 
>2.16.3
>

Actions: View | Splinter Review

Attachments on bug 50750: 62614 | 62615 | 139605 | 139685 | 139780 | 139797 | 139821

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.