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

Ensure that PDFDocument.checkFirstPage waits for cleanup to complete (PR 10392 follow-up)

Browse Source 
Given how this method is currently used there shouldn't be any fonts loaded at the point in time where it's called, but it does seem like a bad idea to assume that that's always going to be the case. Since `PDFDocument.checkFirstPage` is already asynchronous, it's easy enough to simply await `Catalog.cleanup` here.

(The patch also makes a tiny simplification in a loop in `Catalog.cleanup`.)
This commit is contained in:
Jonas Jenwald 2019-12-07 12:20:26 +01:00
parent 514b500a6c
commit a02122e984
2 changed files with 4 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/core/document.js
View File

@ -664,13 +664,13 @@ class PDFDocument {
}
checkFirstPage() {
return this.getPage(0).catch((reason) => {
return this.getPage(0).catch(async (reason) => {
if (reason instanceof XRefEntryException) {
// Clear out the various caches to ensure that we haven't stored any
// inconsistent and/or incorrect state, since that could easily break
// subsequent `this.getPage` calls.
this._pagePromises.length = 0;
this.cleanup();
await this.cleanup();
throw new XRefParseException();
}

5
src/core/obj.js
View File

@ -671,9 +671,8 @@ class Catalog {
});
return Promise.all(promises).then((translatedFonts) => {
for (let i = 0, ii = translatedFonts.length; i < ii; i++) {
const font = translatedFonts[i].dict;
delete font.translated;
for (const { dict, } of translatedFonts) {
delete dict.translated;
}
this.fontCache.clear();
this.builtInCMapCache.clear();