Merge pull request #17690 from calixteman/issue17689

Avoid to access to a missing cidSystemInfo property
This commit is contained in:
calixteman 2024-02-19 12:04:07 +01:00 committed by GitHub
commit 99fa713fba
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
3 changed files with 15 additions and 1 deletions

View File

@ -3719,7 +3719,9 @@ class PartialEvaluator {
properties.composite && properties.composite &&
((properties.cMap.builtInCMap && ((properties.cMap.builtInCMap &&
!(properties.cMap instanceof IdentityCMap)) || !(properties.cMap instanceof IdentityCMap)) ||
(properties.cidSystemInfo.registry === "Adobe" && // The font is supposed to have a CIDSystemInfo dictionary, but some
// PDFs don't include it (fixes issue 17689), hence the `?'.
(properties.cidSystemInfo?.registry === "Adobe" &&
(properties.cidSystemInfo.ordering === "GB1" || (properties.cidSystemInfo.ordering === "GB1" ||
properties.cidSystemInfo.ordering === "CNS1" || properties.cidSystemInfo.ordering === "CNS1" ||
properties.cidSystemInfo.ordering === "Japan1" || properties.cidSystemInfo.ordering === "Japan1" ||

View File

@ -0,0 +1,2 @@
https://github.com/mozilla/pdf.js/files/14328616/2023_12_13.pdf

View File

@ -9736,5 +9736,15 @@
"md5": "407695b36b4611bc7c501a891c8a4b28", "md5": "407695b36b4611bc7c501a891c8a4b28",
"rounds": 1, "rounds": 1,
"type": "eq" "type": "eq"
},
{
"id": "issue17689",
"file": "pdfs/issue17689.pdf",
"md5": "2d260d3795e551fa047eef7c3dbab45b",
"rounds": 1,
"link": true,
"firstPage": 2,
"lastPage": 2,
"type": "eq"
} }
] ]