Skip to Content.
Sympa Menu

mace-opensaml-users - Re: Possible bug in AuthzDecisionStatementImpl

Subject: OpenSAML user discussion

List archive

Re: Possible bug in AuthzDecisionStatementImpl


Chronological Thread 
  • From: Chad La Joie <>
  • To:
  • Subject: Re: Possible bug in AuthzDecisionStatementImpl
  • Date: Fri, 26 Oct 2007 19:17:02 +0200
  • Organization: SWITCH

Yes, this could be done but if you're code isn't checking for nulls in this list you're going to have problems. Lists can have null values and if some one replaced one of our shipped classes with their own implementation they might do something that resulted in nulls as well. You're going to have to check for them.

Fátima Milla Olaya wrote:
Hi, I am new in opensaml but in the schema of the AuthzDecisionStatement the element Evidence maybe doesn't exist (minOccurs="0") so in the method getOrderedChildren of AuthzDecisionStatementImpl should there be an ·if· for verifying its existence?
public List<XMLObject> getOrderedChildren() {
ArrayList<XMLObject> children = new ArrayList<XMLObject>();

children.addAll(actions);
* if(evidence!=null)*
children.add(evidence);
return Collections.unmodifiableList(children);
}
}

--
SWITCH
Serving Swiss Universities
--------------------------
Chad La Joie, Software Engineer, Security
Werdstrasse 2, P.O. Box, 8021 Zurich, Switzerland
phone +41 44 268 15 75, fax +41 44 268 15 68
,
http://www.switch.ch



Archive powered by MHonArc 2.6.16.

Top of Page