RFR: 8339280: jarsigner -verify performs cross-checking between CEN and LOC [v17]

Hai-May Chao hchao at openjdk.org
Thu Mar 27 19:01:12 UTC 2025


On Thu, 27 Mar 2025 13:28:57 GMT, Sean Mullan <mullan at openjdk.org> wrote:

>> Hai-May Chao has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Not split warning in rb.getString() accross multi-lines for Usages.java
>
> src/jdk.jartool/share/classes/sun/security/tools/jarsigner/Main.java line 1137:
> 
>> 1135: 
>> 1136:     private void crossCheckEntries(String jarName) throws Exception {
>> 1137:         List<String> locEntries = new ArrayList<>();
> 
> I think this can be a `HashSet` now that you don't care about the order when checking against the `cenEntries` later.

Changed to `HashSet`.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/23532#discussion_r2017446934


More information about the security-dev mailing list