Avoid exception when trying to render some glyphs contours

calixteman · calixteman · commit fc4985819f6f · 2025-12-22T23:45:34.000+01:00
It fixes #20513.
diff --git a/src/display/font_loader.js b/src/display/font_loader.js
@@ -436,7 +436,7 @@ class FontFaceObject {
     } catch (ex) {
       warn(`getPathGenerator - ignoring character: "${ex}".`);
     }
-    const path = makePathFromDrawOPS(cmds.path);
+    const path = makePathFromDrawOPS(cmds?.path);
 
     if (!this.fontExtraProperties) {
       // Remove the raw path-string, since we don't need it anymore.
diff --git a/test/pdfs/.gitignore b/test/pdfs/.gitignore
@@ -765,4 +765,5 @@
 !two_paragraphs.pdf
 !paragraph_and_link.pdf
 !issue20225.pdf
+!issue20513.pdf
 !issue20516.pdf
diff --git a/test/pdfs/issue20513.pdf b/test/pdfs/issue20513.pdf
diff --git a/test/test_manifest.json b/test/test_manifest.json
@@ -13099,5 +13099,12 @@
     "enableXfa": true,
     "lastPage": 1,
     "type": "eq"
+  },
+  {
+    "id": "issue20513",
+    "file": "pdfs/issue20513.pdf",
+    "md5": "19a3a347773518242fa3cf1c04a9a1e4",
+    "rounds": 1,
+    "type": "eq"
   }
 ]

Original file line number	Diff line number	Diff line change
`@@ -436,7 +436,7 @@ class FontFaceObject {`
`436`	`436`	`} catch (ex) {`
`437`	`437`	warn(`getPathGenerator - ignoring character: "${ex}".`);
`438`	`438`	`}`
`439`		`- const path = makePathFromDrawOPS(cmds.path);`
	`439`	`+ const path = makePathFromDrawOPS(cmds?.path);`
`440`	`440`
`441`	`441`	`if (!this.fontExtraProperties) {`
`442`	`442`	`// Remove the raw path-string, since we don't need it anymore.`