Skip to content

Commit f2a9d02

Browse files
committed
8256370: Add asserts to Reference.getInactive()
Reviewed-by: mchung
1 parent c255b18 commit f2a9d02

File tree

2 files changed

+4
-1
lines changed

2 files changed

+4
-1
lines changed

src/java.base/share/classes/java/lang/ref/Finalizer.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,8 @@ private void runFinalizer(JavaLangAccess jla) {
8383

8484
try {
8585
Object finalizee = this.getInactive();
86-
if (finalizee != null && !(finalizee instanceof java.lang.Enum)) {
86+
assert finalizee != null;
87+
if (!(finalizee instanceof java.lang.Enum)) {
8788
jla.invokeFinalize(finalizee);
8889

8990
// Clear stack slot containing this variable, to decrease

src/java.base/share/classes/java/lang/ref/Reference.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -353,6 +353,8 @@ public T get() {
353353
* null, and would subsequently not finalize the referent/finalizee.
354354
*/
355355
T getInactive() {
356+
assert this instanceof FinalReference;
357+
assert next == this; // I.e. FinalReference is inactive
356358
return this.referent;
357359
}
358360

0 commit comments

Comments
 (0)