SEC-2045: AbstractPreAuthenticationFilter afterPropertiesSet invokes super

web/src/main/java/org/springframework/security/web/authentication/preauth/AbstractPreAuthenticatedProcessingFilter.java

@@ -69,6 +69,12 @@ public abstract class AbstractPreAuthenticatedProcessingFilter extends GenericFi
      */
     @Override
     public void afterPropertiesSet() {
+        try {
+            super.afterPropertiesSet();
+        } catch(ServletException e) {
+            // convert to RuntimeException for passivity on afterPropertiesSet signature
+            throw new RuntimeException(e);
+        }
         Assert.notNull(authenticationManager, "An AuthenticationManager must be set");
     }
 

web/src/test/java/org/springframework/security/web/authentication/preauth/AbstractPreAuthenticatedProcessingFilterTests.java

@@ -5,6 +5,7 @@ import static org.mockito.Matchers.any;
 import static org.mockito.Mockito.*;
 
 import javax.servlet.FilterChain;
+import javax.servlet.ServletException;
 import javax.servlet.http.HttpServletRequest;
 
 import org.junit.After;
@@ -77,6 +78,16 @@ public class AbstractPreAuthenticatedProcessingFilterTests {
         }
     }
 
+    // SEC-2045
+    @Test
+    public void testAfterPropertiesSetInvokesSuper() throws Exception {
+        ConcretePreAuthenticatedProcessingFilter filter = new ConcretePreAuthenticatedProcessingFilter();
+        AuthenticationManager am = mock(AuthenticationManager.class);
+        filter.setAuthenticationManager(am);
+        filter.afterPropertiesSet();
+        assertTrue(filter.initFilterBeanInvoked);
+    }
+
     @Test
     public void testDoFilterAuthenticated() throws Exception {
         testDoFilter(true);
@@ -140,12 +151,18 @@ public class AbstractPreAuthenticatedProcessingFilterTests {
 
     private static class ConcretePreAuthenticatedProcessingFilter extends AbstractPreAuthenticatedProcessingFilter {
         private String principal = "testPrincipal";
+        private boolean initFilterBeanInvoked;
         protected Object getPreAuthenticatedPrincipal(HttpServletRequest httpRequest) {
             return principal;
         }
         protected Object getPreAuthenticatedCredentials(HttpServletRequest httpRequest) {
             return "testCredentials";
         }
+        @Override
+        protected void initFilterBean() throws ServletException {
+            super.initFilterBean();
+            initFilterBeanInvoked = true;
+        }
     }
 
 }