[jetty-dev] [jira] (JETTY-1490) NPE in TagLibraryInfoImpl createTagFileInfo

"Jan Bartel (JIRA)" <[email protected]>
Newsgroups gmane.comp.java.jetty.general
Message-ID <1181388865.45750.1330650484681.JavaMail.j2ee-jira@codehaus01.managed.contegix.com>
     [ https://jira.codehaus.org/browse/JETTY-1490?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jan Bartel reassigned JETTY-1490:
---------------------------------

    Assignee: Jan Bartel
    
> NPE in TagLibraryInfoImpl createTagFileInfo
> -------------------------------------------
>
>                 Key: JETTY-1490
>                 URL: https://jira.codehaus.org/browse/JETTY-1490
>             Project: Jetty
>          Issue Type: Bug
>          Components: JSP
>    Affects Versions: 8.0.4
>            Reporter: Marc Vanbrabant
>            Assignee: Jan Bartel
>
> If you deploy a web exploded application with a TLD to a tag-file you will get a NPE, because the jarFileUrl argument is "null".
> The NPE will occur on:
> private TagFileInfo createTagFileInfo(TreeNode elem, String uri, URL jarFileUrl) throws JasperException {
>  ... ctxt.getTagFileJarUrls().put(path, jarFileUrl);
>  ...
>  }
> where the second "jarFileUrl" cannot be NULL when putting it onto the hashmap.
> Also see the tomcat patch over at:
> http://tomcat.10.n6.nabble.com/svn-commit-r492263-tomcat-jasper-tc5-5-x-src-share-org-apache-jasper-compiler-TagLibraryInfoImpl-java-td2219260.html
> This is a shameless copy/paste of the patch:
> if (path.startsWith("/META-INF/tags")) {
> // Tag file packaged in JAR
> ctxt.getTagFileJarUrls().put(path, jarFileUrl);
> + // check if jarFileUrl is not null before adding it, or this will
> + // cause a NPE.
> + if (jarFileUrl != null) { + ctxt.getTagFileJarUrls().put(path, jarFileUrl); + }
> } else if (!path.startsWith("/WEB-INF/tags")) { err.jspError("jsp.error.tagfile.illegalPath", path); }
> Also tracking this over at glassfish:
> http://java.net/jira/browse/GLASSFISH-18391

--
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

        

---------------------------------------------------------------------
To unsubscribe from this list, please visit:

    http://xircles.codehaus.org/manage_email
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.