[
https://issues.apache.org/jira/browse/IMAGING-178?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15097732#comment-15097732
]
emopers commented on IMAGING-178:
---------------------------------
Yes, I have already submitted PR on github.
https://github.com/apache/commons-imaging/pull/20
Let me know if it is the correct way to do.
> PnmImageParser does not check the validity of input PAM header
> --------------------------------------------------------------
>
> Key: IMAGING-178
> URL: https://issues.apache.org/jira/browse/IMAGING-178
> Project: Commons Imaging
> Issue Type: Bug
> Components: Format: PNM
> Reporter: emopers
>
> PnmImageParser.java directly calls tokenizer.nextToken() at line no 160, 163,
> 166, 169 and 172 on java.util.StringTokenizer tokenizer without checking if
> there are more tokens. Because tokenizer is built from the bytes string that
> can be invalid, this can lead to a runtime exception without a useful error
> message. This can be easily fixed by calling tokenizer.hasMoreTokens()
> before calling tokenizer.nextToken() at each line number mentioned before and
> throwing useful error message.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
