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

Harold Seigel hseigel at openjdk.java.net
Fri Feb 11 13:26:50 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

This pull request has now been integrated.

Changeset: c99c872f
Author:    Harold Seigel <hseigel at openjdk.org>
URL:       https://git.openjdk.java.net/valhalla/commit/c99c872f9fcdcb5874b0ef046b1e645a187dba40
Stats:     146 lines in 4 files changed: 131 ins; 3 del; 12 mod

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

Reviewed-by: fparain

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

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



More information about the valhalla-dev mailing list