[lworld] RFR: 8242402: multianewarray is missing checks on the bottom class [v2]

Frederic Parain fparain at openjdk.java.net
Mon Jul 6 20:43:02 UTC 2020


> Please review this patch which adds a missing check in array class resolution.
> 
> When an array class is resolved and the bottom type is not a primitive type, the envelope of the bottom type signature
> is removed in order to extract the class name and load it. Currently, there's no check that the kind of the loaded
> class matches the enveloped stripped from the signature.  The patch adds this check and includes an unit test.
> 
> Tested on all all platforms with mach5, tiers 1 to 3.
> 
> Thank you,
> 
> Fred

Frederic Parain has updated the pull request incrementally with one additional commit since the last revision:

  Add message, copyrights and comments

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

Changes:
  - all: https://git.openjdk.java.net/valhalla/pull/105/files
  - new: https://git.openjdk.java.net/valhalla/pull/105/files/4616eb98..663cbc0c

Webrevs:
 - full: https://webrevs.openjdk.java.net/valhalla/105/webrev.01
 - incr: https://webrevs.openjdk.java.net/valhalla/105/webrev.00-01

  Stats: 84 lines in 4 files changed: 82 ins; 0 del; 2 mod
  Patch: https://git.openjdk.java.net/valhalla/pull/105.diff
  Fetch: git fetch https://git.openjdk.java.net/valhalla pull/105/head:pull/105

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


More information about the valhalla-dev mailing list