From a60b98412f5457f838e6e369b64674a04a6d273c Mon Sep 17 00:00:00 2001
From: Jonas Jenwald <jonas.jenwald@gmail.com>
Date: Thu, 10 Mar 2022 16:33:10 +0100
Subject: [PATCH] Validate the `cMapUrl`/`standardFontDataUrl` parameters in
 `getDocument`

These changes make sense for two reasons:
 - Given that the parameters are potentially passed to the worker-thread, depending on the `useWorkerFetch` parameter, we need to prevent errors if the user provides values that aren't clonable.
 - By ensuring that the default values are indeed `null`, we'll trigger main-thread fetching (of CMaps and Standard fonts) as intended in the `PartialEvaluator` and thus potentially provide better Error messages.
---
 src/display/api.js | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/src/display/api.js b/src/display/api.js
index 241db612e..2411fa6cf 100644
--- a/src/display/api.js
+++ b/src/display/api.js
@@ -339,6 +339,12 @@ function getDocument(src) {
   if (!Number.isInteger(params.maxImageSize)) {
     params.maxImageSize = -1;
   }
+  if (typeof params.cMapUrl !== "string") {
+    params.cMapUrl = null;
+  }
+  if (typeof params.standardFontDataUrl !== "string") {
+    params.standardFontDataUrl = null;
+  }
   if (typeof params.useWorkerFetch !== "boolean") {
     params.useWorkerFetch =
       params.CMapReaderFactory === DOMCMapReaderFactory &&