RFR: 8313226: Redundant condition test in X509CRLImpl

John Jiang jjiang at openjdk.org
Tue Aug 1 13:51:57 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.

Could this patch be reviewed?

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

PR Comment: https://git.openjdk.org/jdk/pull/15051#issuecomment-1660373553



More information about the security-dev mailing list