Added checks to ensure object is properly initialized

core/src/main/java/org/acegisecurity/providers/ldap/LdapAuthenticationProvider.java

@@ -148,7 +148,8 @@ public class LdapAuthenticationProvider extends AbstractUserDetailsAuthenticatio
         this.authenticator = authenticator;
     }
 
-    public LdapAuthenticator getAuthenticator() {
+    protected LdapAuthenticator getAuthenticator() {
+        Assert.notNull(authenticator, "You must set the authenticator before using this instance.");
         return authenticator;
     }
 
@@ -157,7 +158,8 @@ public class LdapAuthenticationProvider extends AbstractUserDetailsAuthenticatio
         this.authoritiesPopulator = authoritiesPopulator;
     }
 
-    public LdapAuthoritiesPopulator getAuthoritiesPopulator() {
+    protected LdapAuthoritiesPopulator getAuthoritiesPopulator() {
+        Assert.notNull(authoritiesPopulator, "You must set the authoritiesPopulator before using this instance.");
         return authoritiesPopulator;
     }
 

core/src/main/java/org/acegisecurity/providers/ldap/authenticator/AbstractLdapAuthenticator.java

@@ -111,6 +111,7 @@ public abstract class AbstractLdapAuthenticator implements LdapAuthenticator, In
     }
 
     public InitialDirContextFactory getInitialDirContextFactory() {
+        Assert.notNull(initialDirContextFactory, "You must set the initialDirContextFactory before using this instance.");
         return initialDirContextFactory;
     }
 

core/src/main/java/org/acegisecurity/providers/ldap/populator/DefaultLdapAuthoritiesPopulator.java

@@ -248,6 +248,7 @@ public class DefaultLdapAuthoritiesPopulator implements LdapAuthoritiesPopulator
     }
 
     protected InitialDirContextFactory getInitialDirContextFactory() {
+        Assert.notNull(initialDirContextFactory, "You must set the initialDirContextFactory before using this instance.");
         return initialDirContextFactory;
     }
 
@@ -280,6 +281,7 @@ public class DefaultLdapAuthoritiesPopulator implements LdapAuthoritiesPopulator
     }
 
     protected String getGroupSearchBase() {
+        Assert.notNull(groupSearchBase, "You must set the groupSearchBase before using this instance.");
         return groupSearchBase;
     }