RFR: 8273826: Correct Manifest file name and NPE checks [v2]

Sean Mullan mullan at openjdk.java.net
Thu Oct 7 15:02:10 UTC 2021


On Thu, 7 Oct 2021 10:44:39 GMT, Sean Coffey <coffeys at openjdk.org> wrote:

>> Use correct manifest file name in the Manifest verifier checks. 
>> Also - extra null check
>> 
>> The test doesn't reproduce the exact issue reported but should prevent future regressions in this area.
>
> Sean Coffey has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Make variables final

Marked as reviewed by mullan (Reviewer).

test/jdk/sun/security/tools/jarsigner/warnings/LowerCaseManifest.java line 38:

> 36:  * @library /test/lib ../
> 37:  * @build jdk.test.lib.util.JarUtils
> 38:  * @run main LowerCaseManifest

You don't need this line as this is the default @run setting.

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

PR: https://git.openjdk.java.net/jdk/pull/5841



More information about the security-dev mailing list