Author: markt
Date: Tue Feb 14 09:23:57 2017
New Revision: 1782934

URL: http://svn.apache.org/viewvc?rev=1782934&view=rev
Log:
Fix https://bz.apache.org/bugzilla/show_bug.cgi?id=60694
Prevent NPE during authentication when no JASPIC AuthConfigFactory is available.

Modified:
    tomcat/trunk/java/org/apache/catalina/authenticator/AuthenticatorBase.java
    tomcat/trunk/webapps/docs/changelog.xml

Modified: 
tomcat/trunk/java/org/apache/catalina/authenticator/AuthenticatorBase.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/authenticator/AuthenticatorBase.java?rev=1782934&r1=1782933&r2=1782934&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/authenticator/AuthenticatorBase.java 
(original)
+++ tomcat/trunk/java/org/apache/catalina/authenticator/AuthenticatorBase.java 
Tue Feb 14 09:23:57 2017
@@ -1194,14 +1194,21 @@ public abstract class AuthenticatorBase
         return provider.orElse(null);
     }
 
+
     private Optional<AuthConfigProvider> findJaspicProvider() {
         AuthConfigFactory factory = AuthConfigFactory.getFactory();
-        Optional<AuthConfigProvider> provider =
-                Optional.ofNullable(factory.getConfigProvider("HttpServlet", 
jaspicAppContextID, this));
+        Optional<AuthConfigProvider> provider;
+        if (factory == null) {
+            provider = Optional.empty();
+        } else {
+            provider = Optional.ofNullable(
+                    factory.getConfigProvider("HttpServlet", 
jaspicAppContextID, this));
+        }
         jaspicProvider = provider;
         return provider;
     }
 
+
     @Override
     public void notify(String layer, String appContext) {
         findJaspicProvider();

Modified: tomcat/trunk/webapps/docs/changelog.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/webapps/docs/changelog.xml?rev=1782934&r1=1782933&r2=1782934&view=diff
==============================================================================
--- tomcat/trunk/webapps/docs/changelog.xml (original)
+++ tomcat/trunk/webapps/docs/changelog.xml Tue Feb 14 09:23:57 2017
@@ -80,6 +80,10 @@
         code. (markt)
       </fix>
       <fix>
+        <bug>60694</bug>: Prevent NPE during authentication when no JASPIC
+        <code>AuthConfigFactory</code> is available. (markt)
+      </fix>
+      <fix>
         <bug>60720</bug>: Replace "WWW-Authenticate" literal with static final
         AUTH_HEADER_NAME in SpnegoAuthenticator. Patch provided by Michael
         Osipov. (violetagg)



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to