Fix potential NPE - SIDP-508
authorlajoie <lajoie@ab3bd59b-922f-494d-bb5f-6f0a3c29deca>
Mon, 10 Oct 2011 18:55:32 +0000 (18:55 +0000)
committerlajoie <lajoie@ab3bd59b-922f-494d-bb5f-6f0a3c29deca>
Mon, 10 Oct 2011 18:55:32 +0000 (18:55 +0000)
git-svn-id: https://subversion.switch.ch/svn/shibboleth/java-idp/branches/REL_2@3076 ab3bd59b-922f-494d-bb5f-6f0a3c29deca

doc/RELEASE-NOTES.txt
src/main/java/edu/internet2/middleware/shibboleth/idp/authn/AuthenticationEngine.java

index 9e0ded3..be83744 100644 (file)
@@ -1,5 +1,6 @@
 Changes in Release 2.3.4
 =============================================
+[SIDP-508] - NullPointerException in AuthenticationEngine
 [SIDP-510] - Error with stack trace when passive cannot be honored
 [SIDP-511] - ExternalAuthnSystemLoginHandler does not support forceAuthn/isPassive
 [SIDP-513] - idpui taglib could look for more languages matches
index a1fdf79..436e42b 100644 (file)
@@ -164,6 +164,11 @@ public class AuthenticationEngine extends HttpServlet {
             LOG.warn("No login context available, unable to return to profile handler");
             forwardRequest("/error.jsp", httpRequest, httpResponse);
         }
+        
+        if (loginContext.getProfileHandlerURL() == null) {
+            LOG.warn("Login context did not contain a profile handler path, unable to return to profile handler");
+            forwardRequest("/error.jsp", httpRequest, httpResponse);
+        }
 
         String profileUrl = HttpServletHelper.getContextRelativeUrl(httpRequest, loginContext.getProfileHandlerURL())
                 .buildURL();