X-Git-Url: http://www.project-moonshot.org/gitweb/?a=blobdiff_plain;f=saml%2Fbinding%2Fimpl%2FMessageFlowRule.cpp;h=af43c8e61d6d3829c41143b6712d39b3e170c681;hb=9fddf9054c76de6239295b98a559fcc75453effc;hp=ef70bd0ca2ebb1a73ccdb45167f2a7c99ae4b55a;hpb=227bde5e63861fad3a8ce4bbecec6276c4bcb929;p=shibboleth%2Fopensaml2.git diff --git a/saml/binding/impl/MessageFlowRule.cpp b/saml/binding/impl/MessageFlowRule.cpp index ef70bd0..af43c8e 100644 --- a/saml/binding/impl/MessageFlowRule.cpp +++ b/saml/binding/impl/MessageFlowRule.cpp @@ -1,5 +1,5 @@ /* - * Copyright 2001-2006 Internet2 + * Copyright 2001-2007 Internet2 * * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. @@ -22,19 +22,31 @@ #include "internal.h" #include "exceptions.h" -#include "RootObject.h" -#include "binding/MessageFlowRule.h" +#include "binding/SecurityPolicyRule.h" -#include +#include #include -#include +#include using namespace opensaml; +using namespace xmltooling::logging; using namespace xmltooling; -using namespace log4cpp; using namespace std; namespace opensaml { + class SAML_DLLLOCAL MessageFlowRule : public SecurityPolicyRule + { + public: + MessageFlowRule(const DOMElement* e); + virtual ~MessageFlowRule() {} + + void evaluate(const xmltooling::XMLObject& message, const GenericRequest* request, SecurityPolicy& policy) const; + + private: + bool m_checkReplay; + time_t m_expires; + }; + SecurityPolicyRule* SAML_DLLLOCAL MessageFlowRuleFactory(const DOMElement* const & e) { return new MessageFlowRule(e); @@ -57,51 +69,46 @@ MessageFlowRule::MessageFlowRule(const DOMElement* e) } } -pair MessageFlowRule::evaluate( - const GenericRequest& request, - const XMLObject& message, - const saml2md::MetadataProvider* metadataProvider, - const QName* role, - const opensaml::TrustEngine* trustEngine - ) const -{ - try { - const RootObject& obj = dynamic_cast(message); - check(obj.getID(), obj.getIssueInstantEpoch()); - } - catch (bad_cast&) { - throw BindingException("Message was not of a recognized type."); - } - return pair(NULL,NULL); -} - -void MessageFlowRule::check(const XMLCh* id, time_t issueInstant) const +void MessageFlowRule::evaluate(const XMLObject& message, const GenericRequest* request, SecurityPolicy& policy) const { Category& log=Category::getInstance(SAML_LOGCAT".SecurityPolicyRule.MessageFlow"); log.debug("evaluating message flow policy (replay checking %s, expiration %lu)", m_checkReplay ? "on" : "off", m_expires); - - time_t skew = XMLToolingConfig::getConfig().clock_skew_secs; + time_t now = time(NULL); - if (issueInstant > now + skew) { - log.error("rejected not-yet-valid message, timestamp (%lu), now (%lu)", issueInstant, now + skew); - throw BindingException("Message rejected, was issued in the future."); + time_t skew = XMLToolingConfig::getConfig().clock_skew_secs; + time_t issueInstant = policy.getIssueInstant(); + if (issueInstant == 0) { + issueInstant = now; } - else if (issueInstant < now - skew - m_expires) { - log.error("rejected expired message, timestamp (%lu), oldest allowed (%lu)", issueInstant, now - skew - m_expires); - throw BindingException("Message expired, was issued too long ago."); + else { + if (issueInstant > now + skew) { + log.errorStream() << "rejected not-yet-valid message, timestamp (" << issueInstant << + "), newest allowed (" << now + skew << ")" << CategoryStream::ENDLINE; + throw SecurityPolicyException("Message rejected, was issued in the future."); + } + else if (issueInstant < now - skew - m_expires) { + log.errorStream() << "rejected expired message, timestamp (" << issueInstant << + "), oldest allowed (" << (now - skew - m_expires) << ")" << CategoryStream::ENDLINE; + throw SecurityPolicyException("Message expired, was issued too long ago."); + } } // Check replay. if (m_checkReplay) { + const XMLCh* id = policy.getMessageID(); + if (!id || !*id) + return; + ReplayCache* replayCache = XMLToolingConfig::getConfig().getReplayCache(); - if (!replayCache) - throw BindingException("No ReplayCache instance available."); - else if (!id) - throw BindingException("Message did not contain an identifier."); - auto_ptr_char temp(id); - if (!replayCache->check("SAML", temp.get(), issueInstant + skew + m_expires)) { + if (!replayCache) { + log.warn("no ReplayCache available, skipping requested replay check"); + return; + } + + auto_ptr_char temp(id); + if (!replayCache->check("MessageFlow", temp.get(), issueInstant + skew + m_expires)) { log.error("replay detected of message ID (%s)", temp.get()); - throw BindingException("Rejecting replayed message ID ($1).", params(1,temp.get())); + throw SecurityPolicyException("Rejecting replayed message ID ($1).", params(1,temp.get())); } } }