RFR: 8281748: runtime/logging/RedefineClasses.java failed "assert(addr != __null) failed: invariant"

[Stefan Johansson]

On Tue, 15 Feb 2022 14:06:30 GMT, Albert Mingkun Yang <ayang at openjdk.org> wrote:

> Approved since it fixes CI failure and it's equivalent to what we had on master. However, I believe `G1CMIsAliveClosure::do_object_b` should not accept `null` as its input. Taking `ReferenceProcessor::process_discovered_list_work` as an example, `null` referents are filtered out before asking if a referent is live. Therefore, IMO, the proper fix should be to check `null` before calling this closure.

Thanks for review. I agree, I've filed [JDK-8281823](https://bugs.openjdk.java.net/browse/JDK-8281823) for this. I currently don't have time to properly look into that and wanted a quick fix since I'm not sure how many failures this could generate.

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

PR: https://git.openjdk.java.net/jdk/pull/7476