Remove Assert in Method.MakeGeneric on Invalid Arg (#116788)

The call to mono_class_inflate_generic_method_checked will set error
when there are invalid types (e.g. typeof(void)/typeof(int*)) passed in.
This should not be an assert, it should fall to the "Invalid generic
arguments" message below.   This does lose the error returned by
mono_class_inflate_generic_method_checked but that error isn't user
friendly.  It would report be something like: "MVAR 1 cannot be expanded
with type 0x1" Instead report the more readable error to the user.

Fix: 71339

Author: scott-ferguson-unity

src/mono/mono/metadata/reflection.c

@@ -2728,9 +2728,8 @@ reflection_bind_generic_method_parameters (MonoMethod *method, MonoArrayHandle t
 	tmp_context.method_inst = ginst;
 
 	inflated = mono_class_inflate_generic_method_checked (method, &tmp_context, error);
-	mono_error_assert_ok (error);
 
-	if (!mono_verifier_is_method_valid_generic_instantiation (inflated)) {
+	if (!is_ok(error) || !inflated || !mono_verifier_is_method_valid_generic_instantiation (inflated)) {
 		mono_error_set_argument (error, NULL, "Invalid generic arguments");
 		return NULL;
 	}