[8u] RFR (S) 8207779: Method::is_valid_method() compares 'this' with NULL

Hohensee, Paul hohensee at amazon.com
Tue Apr 27 20:54:48 UTC 2021

I'd add the modified assert in restore_unshareable_info() in method.cpp, even though it's not present right now. Otherwise lgtm.


-----Original Message-----
From: jdk8u-dev <jdk8u-dev-retn at openjdk.java.net> on behalf of Aleksey Shipilev <shade at redhat.com>
Date: Thursday, April 15, 2021 at 8:25 AM
To: "jdk8u-dev at openjdk.java.net" <jdk8u-dev at openjdk.java.net>
Subject: [8u] RFR (S) 8207779: Method::is_valid_method() compares 'this' with NULL

Original bug:

I was looking at backporting this to 8u as the pre-requisite for JDK-8231949. I have suggested
JDK-8231949 without this pre-requisite, which also matches how 11u backported without this fix.

But, I think the fix is actually simple enough to backport and it dodges UB as a good bonus. The
patch does not apply to 8u cleanly, and requires adjustments to fit the context differences. I
grepped the source for is_valid_method to catch all other places where this needs to be adjusted.

8u webrev:

Testing: Linux x86_64 fastdebug, {tier1, jdk_jfr}


More information about the jdk8u-dev mailing list