[SHALE-494] WebResourceProcessor getResourceURL throws and catches NPE

• Type: Bug
• Status: Open
• Priority: Minor
• Resolution: Unresolved
• Affects Version/s: 1.0.4
• Component/s: Remoting
• Environment: WinXP, glassfish v2ur1
• Assignee: Unassigned
• Reporter: David Cooke
• Created: 06/Jun/08 14:43
• Updated: 06/Jun/08 14:43

Description

In the try catch block where the resource ID is converted to a URL using reflection, the code looks to see if debug logging is enabled in order to log the conversion. However, log is null at this point, causing an NPE which is caught in the surrounding try/catch (there for reflection exceptions). It is then logged as a resource exception, and null is returned. This seconds set of logging uses the log() private method which assigns to the variable log.

if (log.isDebugEnabled())

{ log.debug("getResource(" + resourceId + ") --> " + url); }

should be

if (log().isDebugEnabled())

{ log().debug("getResource(" + resourceId + ") --> " + url); }