You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by le...@apache.org on 2014/10/20 19:06:36 UTC
svn commit: r1633187 -
/pdfbox/branches/1.8/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/NonSequentialPDFParser.java
Author: lehmi
Date: Mon Oct 20 17:06:35 2014
New Revision: 1633187
URL: http://svn.apache.org/r1633187
Log:
PDFBOX-2250: include key for "Invalid object stream xref object reference" IOException, reat fileOffset == 0 like fileOffset == null (merged from trunk)
Modified:
pdfbox/branches/1.8/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/NonSequentialPDFParser.java (contents, props changed)
Modified: pdfbox/branches/1.8/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/NonSequentialPDFParser.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/1.8/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/NonSequentialPDFParser.java?rev=1633187&r1=1633186&r2=1633187&view=diff
==============================================================================
--- pdfbox/branches/1.8/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/NonSequentialPDFParser.java (original)
+++ pdfbox/branches/1.8/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/NonSequentialPDFParser.java Mon Oct 20 17:06:35 2014
@@ -1157,7 +1157,7 @@ public class NonSequentialPDFParser exte
Long fileOffset = xrefTrailerResolver.getXrefTable().get(objKey);
// it is allowed that object references point to null,
// thus we have to test
- if (fileOffset != null)
+ if (fileOffset != null && fileOffset != 0)
{
if (fileOffset > 0)
{
@@ -1171,7 +1171,9 @@ public class NonSequentialPDFParser exte
fileOffset = xrefTrailerResolver.getXrefTable().get(new COSObjectKey(-fileOffset, 0));
if ((fileOffset == null) || (fileOffset <= 0))
{
- throw new IOException("Invalid object stream xref object reference: " + fileOffset);
+ throw new IOException(
+ "Invalid object stream xref object reference for key '" + objKey + "': "
+ + fileOffset);
}
List<COSObject> stmObjects = objToBeParsed.get(fileOffset);
Propchange: pdfbox/branches/1.8/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/NonSequentialPDFParser.java
------------------------------------------------------------------------------
Merged /pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/NonSequentialPDFParser.java:r1619296-1619481