Remove mcid overriding

veraPDF · Feb 19, 2024 · 0cffb5d · 0cffb5d
1 parent 842ea70
commit 0cffb5d
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 0 deletions.
diff --git a/src/core/bounding_boxes.js b/src/core/bounding_boxes.js
@@ -16,6 +16,8 @@ var BoundingBoxesCalculator = (function PartialEvaluatorClosure() {
     this.boundingBoxes = {};
     this.ignoreCalculations = ignoreCalculations;
     this.operationArray = [];
+    this.mcidArray = [];
+    this.sameMcidDepth = 0;
     this.operationIndex = -1;
   }
 
@@ -410,6 +412,22 @@ var BoundingBoxesCalculator = (function PartialEvaluatorClosure() {
         this.boundingBoxesStack.inc();
       }
 
+      // Prevent parsing of Marked content with same MCIDs
+      if (fn === OPS.beginMarkedContentProps && isDict(args[1]) && args[1].has('MCID')) {
+        const mcid = args[1].get('MCID');
+        if (this.mcidArray.includes(mcid)) {
+          this.sameMcidDepth++;
+          return;
+        } else {
+          this.mcidArray.push(mcid);
+        }
+      }
+
+      if (fn === OPS.endMarkedContent && this.sameMcidDepth !== 0) {
+        this.sameMcidDepth--;
+        return;
+      }
+
       switch (fn | 0) {
         case OPS.restore:
           this.graphicsStateManager.restore();

diff --git a/src/core/catalog.js b/src/core/catalog.js
@@ -1637,6 +1637,7 @@ class ExtendedCatalog extends Catalog {
         name: name ? stringToUTF8String(name) : null,
         roleName: roleName ? stringToUTF8String(roleName) : null,
         children: this.getTreeElement(el.get('K'), page, el.getRaw('K')),
+        pageIndex: page,
         ref: ref
       };
     }
@@ -1695,6 +1696,7 @@ class ExtendedCatalog extends Catalog {
         name: name ? stringToUTF8String(name) : null,
         roleName: roleName ? stringToUTF8String(roleName) : null,
         children: [],
+        pageIndex: page,
         ref: ref
       };
     }