Add a null check before invoking finalizer. Before, we would do a callvirt Finalize() on the object and if it didn't have a finalizer, it would call Object::Finalize(). However, since now we use mono_method_invoke with a concrete method, we crash if it's null. So we just

Author: TautvydasZilys

mono/metadata/gc.c

@@ -327,14 +327,6 @@ mono_gc_run_finalize (void *obj, void *data)
 		return;
 	}
 
-	/* 
-	 * To avoid the locking plus the other overhead of mono_runtime_invoke_checked (),
-	 * create and precompile a wrapper which calls the finalize method using
-	 * a CALLVIRT.
-	 */
-	if (log_finalizers)
-		g_log ("mono-gc-finalizers", G_LOG_LEVEL_MESSAGE, "<%s at %p> Compiling finalizer.", o->vtable->klass->name, o);
-
 	mono_runtime_class_init_full (o->vtable, &error);
 	goto_if_nok (&error, unhandled_error);
 
@@ -343,18 +335,20 @@ mono_gc_run_finalize (void *obj, void *data)
 				o->vtable->klass->name_space, o->vtable->klass->name);
 	}
 
-	if (log_finalizers)
-		g_log ("mono-gc-finalizers", G_LOG_LEVEL_MESSAGE, "<%s at %p> Calling finalizer.", o->vtable->klass->name, o);
+	if (finalizer) {
+		if (log_finalizers)
+			g_log ("mono-gc-finalizers", G_LOG_LEVEL_MESSAGE, "<%s at %p> Calling finalizer.", o->vtable->klass->name, o);
 
-	MONO_PROFILER_RAISE (gc_finalizing_object, (o));
+		MONO_PROFILER_RAISE (gc_finalizing_object, (o));
 
-	gpointer params[] = { NULL };
-	mono_runtime_try_invoke (finalizer, o, params, &exc, &error);
+		gpointer params[] = { NULL };
+		mono_runtime_try_invoke (finalizer, o, params, &exc, &error);
 
-	MONO_PROFILER_RAISE (gc_finalized_object, (o));
+		MONO_PROFILER_RAISE (gc_finalized_object, (o));
 
-	if (log_finalizers)
-		g_log ("mono-gc-finalizers", G_LOG_LEVEL_MESSAGE, "<%s at %p> Returned from finalizer.", o->vtable->klass->name, o);
+		if (log_finalizers)
+			g_log ("mono-gc-finalizers", G_LOG_LEVEL_MESSAGE, "<%s at %p> Returned from finalizer.", o->vtable->klass->name, o);
+	}
 
 unhandled_error:
 	if (!is_ok (&error))