Don't iterate over all empty slots in the xref entries (bug 1980958)

Author: calixteman

src/core/primitives.js

@@ -315,7 +315,7 @@ class Dict {
 
 class Ref {
   constructor(num, gen) {
-    this.num = num;
+    this.num = parseInt(num);
     this.gen = gen;
   }
 

src/core/xref.js

@@ -681,10 +681,14 @@ class XRef {
     // When no trailer dictionary candidate exists, try picking the first
     // dictionary that contains a /Root entry (fixes issue18986.pdf).
     if (!trailerDicts.length) {
-      for (const [num, entry] of this.entries.entries()) {
-        if (!entry) {
+      // In case, this.entries is a sparse array we don't want to
+      // iterate over empty entries so we use the `in` operator instead of
+      // using for..of on entries() or a for with the array length.
+      for (const num in this.entries) {
+        if (!Object.hasOwn(this.entries, num)) {
           continue;
         }
+        const entry = this.entries[num];
         const ref = Ref.get(num, entry.gen);
         let obj;
 
@@ -693,6 +697,7 @@ class XRef {
         } catch {
           continue;
         }
+
         if (obj instanceof BaseStream) {
           obj = obj.dict;
         }

test/pdfs/.gitignore

@@ -740,3 +740,4 @@
 !dates_save.pdf
 !print_protection.pdf
 !tracemonkey_with_annotations.pdf
+!bug1980958.pdf

test/pdfs/bug1980958.pdf

@@ -878,6 +878,20 @@ describe("api", function () {
 
       await loadingTask.destroy();
     });
+
+    it("Doesn't iterate over all empty slots in the xref entries (bug 1980958)", async function () {
+      if (isNodeJS) {
+        pending("Worker is not supported in Node.js.");
+      }
+      const loadingTask = getDocument(buildGetDocumentParams("bug1980958.pdf"));
+      const { promise, resolve } = Promise.withResolvers();
+      setTimeout(() => resolve(null), 1000);
+
+      const pdfDocument = await Promise.race([loadingTask.promise, promise]);
+      expect(pdfDocument?.numPages).toEqual(1);
+
+      loadingTask._worker.destroy();
+    });
   });
 
   describe("PDFWorker", function () {