[ 
https://jira.codehaus.org/browse/MJAVADOC-342?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=300776#comment-300776
 ] 

Marcin Domanski commented on MJAVADOC-342:
------------------------------------------

This is true p.i.t.a. Out of 3 projects on javadoc:fix (v2.8.1)
* 1 failed with NPE on:
** {code}java.lang.NullPointerException
        at 
org.apache.maven.plugin.javadoc.AbstractFixJavadocMojo.writeParamTag(AbstractFixJavadocMojo.java:1994){code}
* the other failed with (seems to be fixed in Qdox already):
** {code}java.lang.ArrayIndexOutOfBoundsException: 500
        at com.thoughtworks.qdox.parser.impl.Parser.val_push(Parser.java:142)
        at com.thoughtworks.qdox.parser.impl.Parser.yyparse(Parser.java:1303)
        at com.thoughtworks.qdox.parser.impl.Parser.parse(Parser.java:999)
        at 
com.thoughtworks.qdox.JavaDocBuilder.addSource(JavaDocBuilder.java:353)
        at 
com.thoughtworks.qdox.JavaDocBuilder.addSource(JavaDocBuilder.java:381)
        at 
com.thoughtworks.qdox.JavaDocBuilder.addSource(JavaDocBuilder.java:377)
        at 
org.apache.maven.plugin.javadoc.AbstractFixJavadocMojo.getQdoxClasses(AbstractFixJavadocMojo.java:855){code}
* last one was successful
                
> An incomplete fix for the NPE bugs in AbstractJavadocMojo.java
> --------------------------------------------------------------
>
>                 Key: MJAVADOC-342
>                 URL: https://jira.codehaus.org/browse/MJAVADOC-342
>             Project: Maven 2.x Javadoc Plugin
>          Issue Type: Bug
>            Reporter: Guangtai Liang
>            Priority: Critical
>
> The fix revision 554202 was aimed to remove an NPE bug on the  returned value 
> of  "getJavadocDirectory()" in the method "getSourcePaths " of the file 
> "/maven/plugins/trunk/maven-javadoc-plugin/src/main/java/org/apache/maven/plugin/javadoc/AbstractJavadocMojo.java"
>  , but it is incomplete. 
> Since the returned value  "getJavadocDirectory()" could be null during the 
> runtime execution, its value should also be null-checked before being 
> dereferenced in other methods. 
> The buggy code locations the same fix needs to be applied at are as bellows: 
> Line 2401 of the method "copyJavadocResources";
> Line 1505 of the method "getSourcePaths". 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://jira.codehaus.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to