[java-identity-provider COMMIT] /trunk/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/saml2/profile/delegation/i...

[noreply at shibboleth.net]

Author: putmanb
Date: Fri Sep 25 19:45:31 2015
New Revision: 7786

URL: http://svn.shibboleth.net/view/java-identity-provider?rev=7786&view=rev
Log:
When cloning XMLObject, rather than rooting in new Document, just leave in existing but then drop the DOM.

It will be re-marshalled anyway, so save the expense of the new Document creation.

Modified:
    trunk/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/saml2/profile/delegation/impl/AddAuthnStatementToAssertionFromInboundAssertionToken.java

Modified: trunk/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/saml2/profile/delegation/impl/AddAuthnStatementToAssertionFromInboundAssertionToken.java
URL: http://svn.shibboleth.net/view/java-identity-provider/trunk/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/saml2/profile/delegation/impl/AddAuthnStatementToAssertionFromInboundAssertionToken.java?rev=7786&r1=7785&r2=7786&view=diff
==============================================================================
--- trunk/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/saml2/profile/delegation/impl/AddAuthnStatementToAssertionFromInboundAssertionToken.java	(original)
+++ trunk/idp-saml-impl/src/main/java/net/shibboleth/idp/saml/saml2/profile/delegation/impl/AddAuthnStatementToAssertionFromInboundAssertionToken.java	Fri Sep 25 19:45:31 2015
@@ -259,7 +259,9 @@
      */
     @Nullable protected AuthnStatement getNewAuthnStatement() {
         try {
-            return XMLObjectSupport.cloneXMLObject(sourceStatement, true);
+            AuthnStatement statement = XMLObjectSupport.cloneXMLObject(sourceStatement);
+            statement.releaseChildrenDOM(true);
+            return statement;
         } catch (MarshallingException | UnmarshallingException e) {
             log.error("{} Error cloning Assertion AuthnStatement", getLogPrefix(), e);
             return null;