On Fri, 12 Aug 2022 20:41:24 GMT, Christian Stein <cst...@openjdk.org> wrote:
>> Enhance `JAXPPolicyManager` to setup required permissions for `jtreg` 7 by >> accept version information in JAR file names. >> >> Find details in https://bugs.openjdk.org/browse/JDK-8292182 > > Christian Stein has updated the pull request incrementally with one > additional commit since the last revision: > > Remove unused import statement test/jaxp/javax/xml/jaxp/libs/jaxp/library/JAXPPolicyManager.java line 217: > 215: URL loc = (cs == null) ? null : cs.getLocation(); > 216: String path = (loc == null) ? null : loc.getPath(); > 217: String name = (path == null) ? null : > path.substring(path.lastIndexOf('/') + 1); Is there a chance that this path could end with `/` and thus lead to a `IndexOutOfBoundsException` on this call to `substring`? ------------- PR: https://git.openjdk.org/jdk/pull/9857
