diff --git a/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/COSParser.java b/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/COSParser.java index fc9d2de8cad..f334a431bda 100644 --- a/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/COSParser.java +++ b/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/COSParser.java @@ -1159,6 +1159,7 @@ private boolean validateXrefOffsets(Map xrefOffset) throws I return true; } Map correctedKeys = new HashMap<>(); + Map validKeys = new HashMap<>(); for (Entry objectEntry : xrefOffset.entrySet()) { COSObjectKey objectKey = objectEntry.getKey(); @@ -1178,13 +1179,18 @@ else if (foundObjectKey != objectKey) { // Generation was fixed - need to update map later, after iteration correctedKeys.put(objectKey, foundObjectKey); + } else { + validKeys.put(objectKey, foundObjectKey); } } } for (Entry correctedKeyEntry : correctedKeys.entrySet()) { - xrefOffset.put(correctedKeyEntry.getValue(), - xrefOffset.remove(correctedKeyEntry.getKey())); + if(!validKeys.containsKey(correctedKeyEntry.getValue())) { + // Only replacy entries, if the original entry does not point to a valid object + xrefOffset.put(correctedKeyEntry.getValue(), + xrefOffset.remove(correctedKeyEntry.getKey())); + } } return true; } @@ -1239,6 +1245,7 @@ private COSObjectKey findObjectKey(COSObjectKey objectKey, long offset) throws I if(objectKey.getNumber() != foundObjectNumber){ LOG.warn("found wrong object number. expected [" + objectKey.getNumber() +"] found ["+ foundObjectNumber + "]"); if(!isLenient) return null; + else objectKey = new COSObjectKey(foundObjectNumber, objectKey.getGeneration()); } int genNumber = readGenerationNumber();