On Fri, 28 Jul 2023 19:14:09 GMT, Hai-May Chao <hc...@openjdk.org> wrote:
>> src/java.base/share/classes/sun/security/util/SignatureFileVerifier.java >> line 852: >> >>> 850: Integer tmp = GetIntegerAction.privilegedGetProperty( >>> 851: "jdk.jar.maxSignatureFileSize", 16000000); >>> 852: if (tmp < 0 || tmp > MAX_ARRAY_SIZE) { >> >> Can't `tmp` be `null`? After all, it is an `Integer` object. The rest of >> your code changes look fine to me. > > `tmp` will not be null. `GetIntegerAction.privilegedGetProperty()` will > return the default value when jdk.jar.maxSignatureFileSize=null You're right. Didn't look close enough. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/15072#discussion_r1278003455