From fb504ce913b49399b7a1243dc53bc2dd6977fae8 Mon Sep 17 00:00:00 2001 From: m-holger Date: Tue, 5 Sep 2023 13:34:03 +0100 Subject: Fix #1028 --- libqpdf/QPDF.cc | 22 +++++++++++----------- 1 file changed, 11 insertions(+), 11 deletions(-) (limited to 'libqpdf') diff --git a/libqpdf/QPDF.cc b/libqpdf/QPDF.cc index 9027c7fd..bd591a51 100644 --- a/libqpdf/QPDF.cc +++ b/libqpdf/QPDF.cc @@ -2058,27 +2058,27 @@ QPDF::reserveObjects(QPDFObjectHandle foreign, ObjCopier& obj_copier, bool top) if (foreign.isIndirect()) { QPDFObjGen foreign_og(foreign.getObjGen()); - if (obj_copier.object_map.count(foreign_og) > 0) { - QTC::TC("qpdf", "QPDF already reserved object"); - if (obj_copier.visiting.count(foreign_og)) { - QTC::TC("qpdf", "QPDF loop reserving objects"); - } - return; - } if (!obj_copier.visiting.add(foreign_og)) { + QTC::TC("qpdf", "QPDF loop reserving objects"); return; } - QTC::TC("qpdf", "QPDF copy indirect"); - if (obj_copier.object_map.count(foreign_og) == 0) { + if (obj_copier.object_map.count(foreign_og) > 0) { + QTC::TC("qpdf", "QPDF already reserved object"); + if (!(top && foreign.isPageObject() && obj_copier.object_map[foreign_og].isNull())) { + obj_copier.visiting.erase(foreign); + return; + } + } else { + QTC::TC("qpdf", "QPDF copy indirect"); obj_copier.object_map[foreign_og] = foreign.isStream() ? newStream() : newIndirectNull(); if ((!top) && foreign.isPageObject()) { QTC::TC("qpdf", "QPDF not crossing page boundary"); - obj_copier.visiting.erase(foreign); + obj_copier.visiting.erase(foreign_og); return; } - obj_copier.to_copy.push_back(foreign); } + obj_copier.to_copy.push_back(foreign); } if (foreign_tc == ::ot_array) { -- cgit v1.2.3-54-g00ecf