RFR: 8162431: CatalogUriResolver with circular/self referencing catalog file is not throwing CatalogException as expected.

Joe Wang huizhe.wang at oracle.com
Mon Aug 29 19:18:02 UTC 2016


Hi,

Please review a patch that fixes an issue where circular references were 
not caught if catalogs are pre-loaded. Note that to trigger the issue 
(circular references), the Catalog must be instructed to load additional 
catalogs. This can be done by either setting DEFER to false, or 
providing an uri that does not have a match in the current catalog (thus 
causes the CatalogResolver to load alternative catalogs).  When the uri 
is empty as in the original JCK test, the CatalogResolver will skip the 
resolution and therefore no additional catalogs will be loaded. In the 
unit test, the test (copied from the JCK test) is fixed by adding a 
bogus uri 'anyuri'. The JCK test will need a similar fix.

JBS: https://bugs.openjdk.java.net/browse/JDK-8162431
webrevs: http://cr.openjdk.java.net/~joehw/jdk9/8162431/webrev/

Thanks,
Joe


More information about the core-libs-dev mailing list