Sakurai/pdf.js
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

[MessageHandler] Add a non-PRODUCTION/TESTING check to ensure that wrapReason is called with a valid reason

Browse Source 
There shouldn't be any situation where `reason` isn't either an `Error`, or a cloned "Error" sent via `postMessage`.
This commit is contained in:
Jonas Jenwald 2019-10-06 14:07:53 +02:00
parent 9201c8dad4
commit eabedab38e
1 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/shared/message_handler.js
View File

@ -31,7 +31,13 @@ const StreamKind = {
}; };
function wrapReason(reason) { function wrapReason(reason) {
if (typeof reason !== 'object') { if (typeof PDFJSDev === 'undefined' ||
PDFJSDev.test('!PRODUCTION || TESTING')) {
assert(reason instanceof Error ||
(typeof reason === 'object' && reason !== null),
'wrapReason: Expected "reason" to be a (possibly cloned) Error.');
}
if (typeof reason !== 'object' || reason === null) {
return reason; return reason;
} }
switch (reason.name) { switch (reason.name) {