Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 16 additions & 2 deletions src/core/evaluator.js
Original file line number Diff line number Diff line change
Expand Up @@ -4664,9 +4664,22 @@ class PartialEvaluator {
throw new FormatError("invalid font name");
}

let fontFile, subtype, length1, length2, length3;
let fontFile, fontFileN, subtype, length1, length2, length3;
try {
fontFile = descriptor.get("FontFile", "FontFile2", "FontFile3");
fontFile = descriptor.get("FontFile");
if (fontFile) {
fontFileN = 1;
} else {
fontFile = descriptor.get("FontFile2");
if (fontFile) {
fontFileN = 2;
} else {
fontFile = descriptor.get("FontFile3");
if (fontFile) {
fontFileN = 3;
}
}
}

if (fontFile) {
if (!(fontFile instanceof BaseStream)) {
Expand Down Expand Up @@ -4776,6 +4789,7 @@ class PartialEvaluator {
name: fontName.name,
subtype,
file: fontFile,
fontFileN,
length1,
length2,
length3,
Expand Down
31 changes: 18 additions & 13 deletions src/core/fonts.js
Original file line number Diff line number Diff line change
Expand Up @@ -2682,10 +2682,23 @@ class Font {
}

const isTrueType = !tables["CFF "];
let parsedCff = null;
if (!isTrueType) {
try {
parsedCff = new CFFParser(
new Stream(tables["CFF "].data),
properties,
SEAC_ANALYSIS_ENABLED
).parse();
} catch {
warn("Failed to parse font " + properties.loadedName);
}

// OpenType font (skip composite fonts with non-default glyph mapping).
if (
(header.version === "OTTO" && !properties.composite) ||
(header.version === "OTTO" &&
(!properties.composite ||
(properties.fontFileN === 3 && parsedCff?.isCIDFont))) ||
!tables.head ||
!tables.hhea ||
!tables.maxp ||
Expand Down Expand Up @@ -2725,19 +2738,11 @@ class Font {
}

let numGlyphsFromCFF;
if (!isTrueType) {
if (parsedCff) {
try {
// Trying to repair CFF file
const parser = new CFFParser(
new Stream(tables["CFF "].data),
properties,
SEAC_ANALYSIS_ENABLED
);
const cff = parser.parse();
cff.duplicateFirstGlyph();
const compiler = new CFFCompiler(cff);
tables["CFF "].data = compiler.compile();
numGlyphsFromCFF = cff.charStringCount;
parsedCff.duplicateFirstGlyph();
tables["CFF "].data = new CFFCompiler(parsedCff).compile();
numGlyphsFromCFF = parsedCff.charStringCount;
} catch {
warn("Failed to compile font " + properties.loadedName);
}
Expand Down
1 change: 1 addition & 0 deletions test/pdfs/.gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -921,3 +921,4 @@
!operator_list_cycle.pdf
!knockout_groups_test.pdf
!issue18032.pdf
!Embedded_font.pdf
Binary file added test/pdfs/Embedded_font.pdf
Binary file not shown.
7 changes: 7 additions & 0 deletions test/test_manifest.json
Original file line number Diff line number Diff line change
Expand Up @@ -14260,5 +14260,12 @@
"rounds": 1,
"type": "eq",
"about": "Non-isolated knockout group with a nested non-isolated subgroup over existing text."
},
{
"id": "Embedded_font.pdf",
"file": "pdfs/Embedded_font.pdf",
"md5": "b68dd5a3e6833d1af94e295fe1d60285",
"rounds": 1,
"type": "eq"
}
]
Loading