[#WODEN-159] InterfaceImpl.getAllInterfaceOperations and getAllInterfaceFaults are not quite recursive

Woden

InterfaceImpl.getAllInterfaceOperations and getAllInterfaceFaults are not quite recursive

Created: 02/May/07 07:53 PM Updated: 13/Feb/08 09:12 PM

Return to search

Component/s:

Parser

Affects Version/s:

M7a

Fix Version/s:

Time Tracking:

Not Specified

Resolution Date:

02/Aug/07 12:17 PM

Description

« Hide

Jacek Kopecky <jacek.kopecky@deri.org> reported the following Woden bug on the W3C mailing list. See http://lists.w3.org/Archives/Public/www-ws-desc/2007May/0001.html.

InterfaceImpl.getAllInterfaceOperations and getAllInterfaceFaults are
not quite recursive, they only go one level of interface extension deep.
This manifests in the flickr test-suite WSDL file when trying to access
the InterfaceOperation of the BindingOperation for
"flickr.groups.pools.add" - the method getInterfaceOperation() returns
null because the actual operation is too deep in the hierarchy of
extended interfaces.

Fix: in both InterfaceImpl.getDerivedInterface*, change getInterface* to
getAllInterface* - this can get into an infinite loop if a circular
interface extension is not detected as an error first.

All

Comments

Work Log

Change History

Subversion Commits

Sort Order:

Repository	Revision	Date	User	Message
ASF	#562092	Thu Aug 02 12:16:16 UTC 2007	jkaputin	~~WODEN-159~~ Fix to detect a circular reference in interface extension and avoid an infinite loop when processing interface faults and operations.
				Files Changed
				MODIFY /incubator/woden/trunk/java/src/org/apache/woden/internal/wsdl20/InterfaceImpl.java