You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@wookie.apache.org by "Darren Jones (JIRA)" <ji...@apache.org> on 2014/01/06 16:07:50 UTC
[jira] [Updated] (WOOKIE-424) NPE with Wookie.war deployed on
WildFly 8.0.0.CR1
[ https://issues.apache.org/jira/browse/WOOKIE-424?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Darren Jones updated WOOKIE-424:
--------------------------------
Affects Version/s: (was: 0.15.0)
0.14.0
> NPE with Wookie.war deployed on WildFly 8.0.0.CR1
> -------------------------------------------------
>
> Key: WOOKIE-424
> URL: https://issues.apache.org/jira/browse/WOOKIE-424
> Project: Wookie
> Issue Type: Bug
> Components: Server
> Affects Versions: 0.14.0
> Environment: WildFly 8.0.0.CR1, Undertow web server component 1.0.0.Beta31-SNAPSHOT
> Reporter: Darren Jones
>
> In LocalizedResourceFilter, there is code that looks for a localized copy of a resource. Here is an example snippet that gives rise to the exception (around line 185):
> String filePath = filterConfig.getServletContext().getRealPath(path);
> if (new File(filePath).exists())
> return context + path;
> According to the javadoc for ServletContext.getRealPath(), it is allowed to return null if the file does not exist. As my widgets are not localized, this is the case, and a NullPointerException occurs because null is then passed to new File(filePath).
> I'm not sure why this has not been found before - perhaps other web servers have been returning non-null if a resource does not exist.
> I've fixed it temporarily with a local patch that does an extra null check before calling new File():
> String filePath = filterConfig.getServletContext().getRealPath(path);
> if (filePath != null && new File(filePath).exists())
> return context + path;
> I will attach a patch file since there are 3 places in the file using getRealPath().
--
This message was sent by Atlassian JIRA
(v6.1.5#6160)