diff options
author | Jay Berkenbilt <jberkenbilt@users.noreply.github.com> | 2023-09-03 14:54:12 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-09-03 14:54:12 +0200 |
commit | 2b4dcb33aa3cc130241894df78e2b67bbe5c99b8 (patch) | |
tree | 12e3ed0c43b214a852b683cf0c40e81c824d3f44 /libqpdf | |
parent | fec75b44b4d5b0ee84c5bf98fee6139dceacfde3 (diff) | |
parent | b0c3ea2bb6e0cab1d09726f3e66742eaf45c5745 (diff) | |
download | qpdf-2b4dcb33aa3cc130241894df78e2b67bbe5c99b8.tar.zst |
Merge pull request #1014 from m-holger/i1011
Change QPDF::copyForeignObject to return a null object when called wi…
Diffstat (limited to 'libqpdf')
-rw-r--r-- | libqpdf/QPDF.cc | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/libqpdf/QPDF.cc b/libqpdf/QPDF.cc index e39f9b7a..95ed39ec 100644 --- a/libqpdf/QPDF.cc +++ b/libqpdf/QPDF.cc @@ -2034,13 +2034,13 @@ QPDF::copyForeignObject(QPDFObjectHandle foreign) } obj_copier.to_copy.clear(); - auto& result = obj_copier.object_map[foreign.getObjGen()]; - if (!result.isInitialized()) { - result = QPDFObjectHandle::newNull(); - warn(damagedPDF("Unexpected reference to /Pages object while copying foreign object. " - "Replacing with Null object.")); + auto og = foreign.getObjGen(); + if (!obj_copier.object_map.count(og)) { + warn(damagedPDF("unexpected reference to /Pages object while copying foreign object; " + "replacing with null")); + return QPDFObjectHandle::newNull(); } - return result; + return obj_copier.object_map[foreign.getObjGen()]; } void |