Merge pull request #15170 from calixteman/js_rm_null

[JS] Embedded JS scripts can have some null chars
mozilla · Jul 15, 2022 · de7d1d2 · de7d1d2
2 parents 75b8647 + 5f0c95e
commit de7d1d2
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/core/catalog.js b/src/core/catalog.js
@@ -982,7 +982,8 @@ class Catalog {
       if (javaScript === null) {
         javaScript = new Map();
       }
-      javaScript.set(name, stringToPDFString(js));
+      js = stringToPDFString(js).replace(/\u0000/g, "");
+      javaScript.set(name, js);
     }
 
     if (obj instanceof Dict && obj.has("JavaScript")) {

diff --git a/src/core/core_utils.js b/src/core/core_utils.js
@@ -339,7 +339,7 @@ function _collectJS(entry, xref, list, parents) {
       } else if (typeof js === "string") {
         code = js;
       }
-      code = code && stringToPDFString(code);
+      code = code && stringToPDFString(code).replace(/\u0000/g, "");
       if (code) {
         list.push(code);
       }