[java-identity-provider COMMIT] /trunk/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/session/impl/AddLogoutRequ...

noreply at shibboleth.net noreply at shibboleth.net
Thu Nov 5 10:28:02 EST 2015


Author: scantor
Date: Thu Nov  5 10:28:01 2015
New Revision: 7942

URL: http://svn.shibboleth.net/view/java-identity-provider?rev=7942&view=rev
Log:
Drop the clone's DOM to account for encryption.

Modified:
    trunk/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/session/impl/AddLogoutRequest.java

Modified: trunk/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/session/impl/AddLogoutRequest.java
URL: http://svn.shibboleth.net/view/java-identity-provider/trunk/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/session/impl/AddLogoutRequest.java?rev=7942&r1=7941&r2=7942&view=diff
==============================================================================
--- trunk/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/session/impl/AddLogoutRequest.java	(original)
+++ trunk/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/session/impl/AddLogoutRequest.java	Thu Nov  5 10:28:01 2015
@@ -236,7 +236,10 @@
         object.setVersion(SAMLVersion.VERSION_20);
 
         try {
-            final NameID nameId = XMLObjectSupport.cloneXMLObject(saml2Session.getNameID(), true);
+            final NameID nameId = XMLObjectSupport.cloneXMLObject(saml2Session.getNameID());
+            // Drop the cloned DOM so that encryption won't fail later.
+            nameId.releaseDOM();
+            nameId.releaseChildrenDOM(true);
             object.setNameID(nameId);
         } catch (final MarshallingException|UnmarshallingException e) {
             log.error("{} Error cloning NameID for use in LogoutRequest for {}", getLogPrefix(),



More information about the commits mailing list