You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by fm...@apache.org on 2011/11/29 17:30:04 UTC

svn commit: r1207950 - /sling/trunk/bundles/engine/src/main/java/org/apache/sling/engine/impl/SlingHttpServletRequestImpl.java

Author: fmeschbe
Date: Tue Nov 29 16:30:03 2011
New Revision: 1207950

URL: http://svn.apache.org/viewvc?rev=1207950&view=rev
Log:
SLING-2307 Prevent NullPointerException when called before the ContentData is setup like for example when handling request processing start in the RequestLogger.

Modified:
    sling/trunk/bundles/engine/src/main/java/org/apache/sling/engine/impl/SlingHttpServletRequestImpl.java

Modified: sling/trunk/bundles/engine/src/main/java/org/apache/sling/engine/impl/SlingHttpServletRequestImpl.java
URL: http://svn.apache.org/viewvc/sling/trunk/bundles/engine/src/main/java/org/apache/sling/engine/impl/SlingHttpServletRequestImpl.java?rev=1207950&r1=1207949&r2=1207950&view=diff
==============================================================================
--- sling/trunk/bundles/engine/src/main/java/org/apache/sling/engine/impl/SlingHttpServletRequestImpl.java (original)
+++ sling/trunk/bundles/engine/src/main/java/org/apache/sling/engine/impl/SlingHttpServletRequestImpl.java Tue Nov 29 16:30:03 2011
@@ -47,6 +47,7 @@ import org.apache.sling.api.resource.Res
 import org.apache.sling.api.resource.ResourceResolver;
 import org.apache.sling.engine.impl.helper.NullResourceBundle;
 import org.apache.sling.engine.impl.parameters.ParameterSupport;
+import org.apache.sling.engine.impl.request.ContentData;
 import org.apache.sling.engine.impl.request.RequestData;
 import org.apache.sling.engine.impl.request.SlingRequestDispatcher;
 import org.osgi.service.http.HttpContext;
@@ -95,7 +96,8 @@ public class SlingHttpServletRequestImpl
     }
 
     public Resource getResource() {
-        return getRequestData().getContentData().getResource();
+        final ContentData cd = getRequestData().getContentData();
+        return (cd == null) ? null : cd.getResource();
     }
 
     public ResourceResolver getResourceResolver() {
@@ -206,7 +208,8 @@ public class SlingHttpServletRequestImpl
      * @see org.apache.sling.api.SlingHttpServletRequest#getRequestPathInfo()
      */
     public RequestPathInfo getRequestPathInfo() {
-        return getRequestData().getContentData().getRequestPathInfo();
+        final ContentData cd = getRequestData().getContentData();
+        return (cd == null) ? null : cd.getRequestPathInfo();
     }
 
     /**