You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by ti...@apache.org on 2021/05/06 06:35:56 UTC
svn commit: r1889553 -
/pdfbox/branches/2.0/preflight/src/main/java/org/apache/pdfbox/preflight/process/CatalogValidationProcess.java
Author: tilman
Date: Thu May 6 06:35:55 2021
New Revision: 1889553
URL: http://svn.apache.org/viewvc?rev=1889553&view=rev
Log:
PDFBOX-4892: avoid ClassCastException, as suggested by valerybokov; remove unneeded code
Modified:
pdfbox/branches/2.0/preflight/src/main/java/org/apache/pdfbox/preflight/process/CatalogValidationProcess.java
Modified: pdfbox/branches/2.0/preflight/src/main/java/org/apache/pdfbox/preflight/process/CatalogValidationProcess.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/preflight/src/main/java/org/apache/pdfbox/preflight/process/CatalogValidationProcess.java?rev=1889553&r1=1889552&r2=1889553&view=diff
==============================================================================
--- pdfbox/branches/2.0/preflight/src/main/java/org/apache/pdfbox/preflight/process/CatalogValidationProcess.java (original)
+++ pdfbox/branches/2.0/preflight/src/main/java/org/apache/pdfbox/preflight/process/CatalogValidationProcess.java Thu May 6 06:35:55 2021
@@ -326,11 +326,6 @@ public class CatalogValidationProcess ex
{
try
{
- if (destOutputProfile == null)
- {
- return;
- }
-
// destOutputProfile should be an instance of COSObject because of this is a object reference
if (destOutputProfile instanceof COSObject)
{
@@ -348,6 +343,12 @@ public class CatalogValidationProcess ex
}
// else the profile will be kept in the tmpDestOutputProfile if it is valid
}
+ else
+ {
+ addValidationError(ctx, new ValidationError(ERROR_GRAPHIC_OUTPUT_INTENT_INVALID_ENTRY,
+ "OutputIntent object should be a reference: " + destOutputProfile));
+ return;
+ }
// keep reference to avoid multiple profile definition
mapDestOutputProfile.put(new COSObjectKey((COSObject) destOutputProfile), true);