You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pdfbox.apache.org by "Tilman Hausherr (JIRA)" <ji...@apache.org> on 2015/06/22 19:48:02 UTC

[jira] [Commented] (PDFBOX-2834) Violation in PDOutputIntent.getDestOutputProfile() method

    [ https://issues.apache.org/jira/browse/PDFBOX-2834?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14596319#comment-14596319 ] 

Tilman Hausherr commented on PDFBOX-2834:
-----------------------------------------

I think this is a one line change, i.e. use getDictionaryObject instead of getItem.

> Violation in PDOutputIntent.getDestOutputProfile() method
> ---------------------------------------------------------
>
>                 Key: PDFBOX-2834
>                 URL: https://issues.apache.org/jira/browse/PDFBOX-2834
>             Project: PDFBox
>          Issue Type: Bug
>          Components: PDModel
>    Affects Versions: 2.0.0
>         Environment: Windows 7, jdk 1.7.0_51
>            Reporter: Evgeniy Muravitskiy
>            Assignee: Andreas Lehmkühler
>             Fix For: 2.0.0
>
>         Attachments: OutputIntentTest.pdf
>
>
> Inside this method an Object (the result of dictionary.getItem(COSName.DEST_OUTPUT_PROFILE) method call) is casted to COSStream, but it can be also a COSObject containing COSStream which leads to class cast exception.
> {code}
> public COSStream getDestOutputIntent()
> {
>      return (COSStream) dictionary.getItem(COSName.DEST_OUTPUT_PROFILE);
> }
> {code}
> Possible solutions :
> 1) Check class instance and handle them in proper way*
> 2) Usage of visitor pattern
> *{code}
> public COSStream getDestOutputIntent()
> {
>         final COSBase item = dictionary.getItem(COSName.DEST_OUTPUT_PROFILE);
>         return getDestOutputIntent(item);
> }
> private COSStream getDestOutputIntent(COSBase object) {
>         if (object instanceof COSStream) {
>             return (COSStream) object;
>         } else if (object instanceof COSObject) {
>             return getDestOutputIntent(((COSObject) object).getObject());
>         } else {
>             return null;
>         }
> }
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@pdfbox.apache.org
For additional commands, e-mail: dev-help@pdfbox.apache.org