RFR: 8292182: [TESTLIB] Enhance JAXPPolicyManager to setup required permissions for jtreg version 7 jar [v2]
Jaikiran Pai
jpai at openjdk.org
Sat Aug 13 03:43:13 UTC 2022
On Fri, 12 Aug 2022 20:41:24 GMT, Christian Stein <cstein at 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
More information about the core-libs-dev
mailing list