[lworld] RFR: 8281471: [lworld] check_code.c should not accept Q-signatures as valid

Frederic Parain fparain at openjdk.java.net
Thu Feb 10 18:29:38 UTC 2022


On Thu, 10 Feb 2022 18:10:56 GMT, Harold Seigel <hseigel at openjdk.org> wrote:

> Please review this small change to not allow Q signatures in old class files that are verified using the old verifier.  The change removes checks for Q signatures from the old verifier.
> 
> The fix was tested with Mach5 tiers 1-2 on Linux, Mac OS, and Windows, and Mach5 tiers 3-5 on Linux x64.
> 
> Thanks, Harold

Looks good to me.

Fred

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

Marked as reviewed by fparain (Committer).

PR: https://git.openjdk.java.net/valhalla/pull/645



More information about the valhalla-dev mailing list