[java-opensaml COMMIT] /trunk/opensaml-security-impl/src/main/java/org/opensaml/security/messaging/impl/BaseTrustEngi...

[noreply at shibboleth.net]

Author: putmanb
Date: Fri May 24 19:00:46 2013
New Revision: 3346

URL: http://svn.shibboleth.net/view/java-opensaml?rev=3346&view=rev
Log:
Add constraint and init checks.

Modified:
    trunk/opensaml-security-impl/src/main/java/org/opensaml/security/messaging/impl/BaseTrustEngineSecurityHandler.java

Modified: trunk/opensaml-security-impl/src/main/java/org/opensaml/security/messaging/impl/BaseTrustEngineSecurityHandler.java
URL: http://svn.shibboleth.net/view/java-opensaml/trunk/opensaml-security-impl/src/main/java/org/opensaml/security/messaging/impl/BaseTrustEngineSecurityHandler.java?rev=3346&r1=3345&r2=3346&view=diff
==============================================================================
--- trunk/opensaml-security-impl/src/main/java/org/opensaml/security/messaging/impl/BaseTrustEngineSecurityHandler.java (original)
+++ trunk/opensaml-security-impl/src/main/java/org/opensaml/security/messaging/impl/BaseTrustEngineSecurityHandler.java Fri May 24 19:00:46 2013
@@ -18,6 +18,7 @@
 package org.opensaml.security.messaging.impl;
 
 import net.shibboleth.utilities.java.support.component.ComponentInitializationException;
+import net.shibboleth.utilities.java.support.component.ComponentSupport;
 import net.shibboleth.utilities.java.support.logic.Constraint;
 import net.shibboleth.utilities.java.support.resolver.CriteriaSet;
 
@@ -35,7 +36,8 @@
  * @param <TokenType> type of token which is being evaluated by the underlying trust engine
  * @param <MessageType> type of message contained in the message context being evaluated
  */
-public abstract class BaseTrustEngineSecurityHandler<TokenType, MessageType> extends AbstractMessageHandler<MessageType> {
+public abstract class BaseTrustEngineSecurityHandler<TokenType, MessageType> 
+        extends AbstractMessageHandler<MessageType> {
 
     /** Logger. */
     private final Logger log = LoggerFactory.getLogger(BaseTrustEngineSecurityHandler.class);
@@ -58,7 +60,8 @@
      * @param engine engine used to validate the untrusted token
      */
     public void setTrustEngine(TrustEngine<TokenType> engine) {
-        trustEngine = engine;
+        ComponentSupport.ifInitializedThrowUnmodifiabledComponentException(this);
+        trustEngine = Constraint.isNotNull(engine, "TrustEngine may not be null");
     }
 
     /** {@inheritDoc} */