fixed.

thanks,
dims

On Tue, 17 Aug 2004 10:02:54 -0400, Sean Mullan <[EMAIL PROTECTED]> wrote:
> There is a bug in XMLSignatureInput.getNodeSet() - it will throw a
> NullPointerException if the nodeset is empty. This is a regression from the 1_1
> release and causes one of the xpath filter 2 interop test vectors to fail:
> merlin-xpath-filter2-three/sign-spec.xml
> 
> Here is a simple patch/code diffs:
> 
> $ cvs diff -c XMLSignatureInput.java
> Index: XMLSignatureInput.java
> ===================================================================
> RCS file:
> /home/cvspublic/xml-security/src/org/apache/xml/security/signature/XMLSignatureInput.java,v
> retrieving revision 1.26
> diff -c -r1.26 XMLSignatureInput.java
> *** XMLSignatureInput.java      12 May 2004 12:00:45 -0000      1.26
> --- XMLSignatureInput.java      17 Aug 2004 13:59:44 -0000
> ***************
> *** 250,256 ****
>                  return this._inputNodeSet;
>            }
>          if (this.isNodeSet()) {
> !
> XMLUtils.circumventBug2650(XMLUtils.getOwnerDocument(this._inputNodeSet));
>             return this._inputNodeSet;
>          } else if (this.isOctetStream()) {
>             DocumentBuilderFactory dfactory = DocumentBuilderFactory.newInstance();
> --- 250,258 ----
>                  return this._inputNodeSet;
>            }
>          if (this.isNodeSet()) {
> !        if (!this._inputNodeSet.isEmpty()) {
> !
> XMLUtils.circumventBug2650(XMLUtils.getOwnerDocument(this._inputNodeSet));
> !        }
>             return this._inputNodeSet;
>          } else if (this.isOctetStream()) {
>             DocumentBuilderFactory dfactory = DocumentBuilderFactory.newInstance();
> 
> 


-- 
Davanum Srinivas - http://webservices.apache.org/~dims/

Reply via email to