Integrated: 8313226: Redundant condition test in X509CRLImpl

John Jiang jjiang at openjdk.org
Tue Aug 1 22:38:53 UTC 2023


On Thu, 27 Jul 2023 04:00:21 GMT, John Jiang <jjiang at openjdk.org> wrote:

> if ((nextByte == DerValue.tag_SequenceOf)
>         && (! ((nextByte & 0x0c0) == 0x080))) {
>     ...
>     ...
> }
> 
> If `nextByte` is `DerValue.tag_SequenceOf`, exactly `0x30`, then the test after `&&` should always be true.

This pull request has now been integrated.

Changeset: 28be34c1
Author:    John Jiang <jjiang at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/28be34c1b9179e21c8ec5d2f9b05e3f842bb30a1
Stats:     3 lines in 1 file changed: 0 ins; 1 del; 2 mod

8313226: Redundant condition test in X509CRLImpl

Reviewed-by: jnimeh

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

PR: https://git.openjdk.org/jdk/pull/15051



More information about the security-dev mailing list