#include <mono/metadata/debug-helpers.h>
#include <mono/metadata/reflection.h>
#include <mono/utils/mono-logger.h>
+#include <mono/metadata/exception.h>
MonoDefaults mono_defaults;
loader_error_thread_id = TlsAlloc ();
}
+void
+mono_loader_cleanup (void)
+{
+ TlsFree (loader_error_thread_id);
+
+ DeleteCriticalSection (&loader_mutex);
+}
+
/*
* Handling of type load errors should be done as follows:
*
set_loader_error (MonoLoaderError *error)
{
TlsSetValue (loader_error_thread_id, error);
-}
+}
-/*
+/**
+ * mono_loader_set_error_assembly_load:
+ *
+ * Set the loader error for this thread.
+ */
+void
+mono_loader_set_error_assembly_load (const char *assembly_name, gboolean ref_only)
+{
+ MonoLoaderError *error;
+
+ if (mono_loader_get_last_error ())
+ return;
+
+ error = g_new0 (MonoLoaderError, 1);
+ error->kind = MONO_LOADER_ERROR_ASSEMBLY;
+ error->assembly_name = g_strdup (assembly_name);
+ error->ref_only = ref_only;
+
+ /*
+ * This is not strictly needed, but some (most) of the loader code still
+ * can't deal with load errors, and this message is more helpful than an
+ * assert.
+ */
+ g_warning ("Could not load file or assembly '%s' or one of its dependencies.", assembly_name);
+
+ set_loader_error (error);
+}
+
+/**
* mono_loader_set_error_type_load:
*
- * Set the loader error for this thread. CLASS_NAME and ASSEMBLY_NAME should be
- * dynamically allocated strings whose ownership is passed to this function.
+ * Set the loader error for this thread.
*/
void
-mono_loader_set_error_type_load (char *class_name, char *assembly_name)
+mono_loader_set_error_type_load (const char *class_name, const char *assembly_name)
{
MonoLoaderError *error;
- if (mono_loader_get_last_error ()) {
- g_free (class_name);
- g_free (assembly_name);
+ if (mono_loader_get_last_error ())
return;
- }
error = g_new0 (MonoLoaderError, 1);
error->kind = MONO_LOADER_ERROR_TYPE;
- error->class_name = class_name;
- error->assembly_name = assembly_name;
+ error->class_name = g_strdup (class_name);
+ error->assembly_name = g_strdup (assembly_name);
/*
* This is not strictly needed, but some (most) of the loader code still
* assert.
*/
g_warning ("The class %s could not be loaded, used in %s", class_name, assembly_name);
-
+
set_loader_error (error);
}
* inside metadata.
*/
void
-mono_loader_set_error_method_load (MonoClass *klass, const char *member_name)
+mono_loader_set_error_method_load (const char *class_name, const char *member_name)
{
MonoLoaderError *error;
error = g_new0 (MonoLoaderError, 1);
error->kind = MONO_LOADER_ERROR_METHOD;
- error->klass = klass;
+ error->class_name = g_strdup (class_name);
error->member_name = member_name;
- g_warning ("Missing member %s in type %s, assembly %s", member_name, mono_class_get_name (klass), klass->image->name);
-
set_loader_error (error);
}
/*
* mono_loader_set_error_field_load:
*
- * Set the loader error for this thread. MEMBER_NAME should point to a string
+ * Set the loader error for this thread. MEMBER_NAME should point to a string
* inside metadata.
*/
void
error->kind = MONO_LOADER_ERROR_FIELD;
error->klass = klass;
error->member_name = member_name;
-
+
set_loader_error (error);
}
return (MonoLoaderError*)TlsGetValue (loader_error_thread_id);
}
+/**
+ * mono_loader_clear_error:
+ *
+ * Disposes any loader error messages on this thread
+ */
void
mono_loader_clear_error (void)
{
MonoLoaderError *ex = (MonoLoaderError*)TlsGetValue (loader_error_thread_id);
- g_assert (ex);
-
- g_free (ex->class_name);
- g_free (ex->assembly_name);
+ g_free (ex->class_name);
+ g_free (ex->assembly_name);
g_free (ex);
-
+
TlsSetValue (loader_error_thread_id, NULL);
}
+/**
+ * mono_loader_error_prepare_exception:
+ * @error: The MonoLoaderError to turn into an exception
+ *
+ * This turns a MonoLoaderError into an exception that can be thrown
+ * and resets the Mono Loader Error state during this process.
+ *
+ */
+MonoException *
+mono_loader_error_prepare_exception (MonoLoaderError *error)
+{
+ MonoException *ex = NULL;
+
+ switch (error->kind) {
+ case MONO_LOADER_ERROR_TYPE: {
+ char *cname = g_strdup (error->class_name);
+ char *aname = g_strdup (error->assembly_name);
+ MonoString *class_name;
+
+ mono_loader_clear_error ();
+
+ class_name = mono_string_new (mono_domain_get (), cname);
+
+ ex = mono_get_exception_type_load (class_name, aname);
+ g_free (cname);
+ g_free (aname);
+ break;
+ }
+ case MONO_LOADER_ERROR_METHOD: {
+ char *cname = g_strdup (error->class_name);
+ char *aname = g_strdup (error->member_name);
+
+ mono_loader_clear_error ();
+ ex = mono_get_exception_missing_method (cname, aname);
+ g_free (cname);
+ g_free (aname);
+ break;
+ }
+
+ case MONO_LOADER_ERROR_FIELD: {
+ char *cnspace = g_strdup (*error->klass->name_space ? error->klass->name_space : "");
+ char *cname = g_strdup (error->klass->name);
+ char *cmembername = g_strdup (error->member_name);
+ char *class_name;
+
+ mono_loader_clear_error ();
+ class_name = g_strdup_printf ("%s%s%s", cnspace, cnspace ? "." : "", cname);
+
+ ex = mono_get_exception_missing_field (class_name, cmembername);
+ g_free (class_name);
+ g_free (cname);
+ g_free (cmembername);
+ g_free (cnspace);
+ break;
+ }
+
+ case MONO_LOADER_ERROR_ASSEMBLY: {
+ char *msg;
+
+ if (error->ref_only)
+ msg = g_strdup_printf ("Cannot resolve dependency to assembly '%s' because it has not been preloaded. When using the ReflectionOnly APIs, dependent assemblies must be pre-loaded or loaded on demand through the ReflectionOnlyAssemblyResolve event.", error->assembly_name);
+ else
+ msg = g_strdup_printf ("Could not load file or assembly '%s' or one of its dependencies.", error->assembly_name);
+
+ mono_loader_clear_error ();
+ ex = mono_get_exception_file_not_found2 (msg, mono_string_new (mono_domain_get (), error->assembly_name));
+ g_free (msg);
+ break;
+ }
+
+ default:
+ g_assert_not_reached ();
+ }
+
+ return ex;
+}
+
static MonoClassField*
field_from_memberref (MonoImage *image, guint32 token, MonoClass **retklass,
MonoGenericContext *context)
class = cols [MONO_MEMBERREF_CLASS] & MONO_MEMBERREF_PARENT_MASK;
fname = mono_metadata_string_heap (image, cols [MONO_MEMBERREF_NAME]);
-
+
ptr = mono_metadata_blob_heap (image, cols [MONO_MEMBERREF_SIGNATURE]);
mono_metadata_decode_blob_size (ptr, &ptr);
/* we may want to check the signature here... */
for (i = 0; i < sig1->sentinelpos; i++) {
MonoType *p1 = sig1->params[i];
MonoType *p2 = sig2->params[i];
-
+
/*if (p1->attrs != p2->attrs)
return FALSE;
*/
}
static MonoMethod *
-find_method_in_class (MonoClass *klass, const char *name, const char *qname, const char *fqname, MonoMethodSignature *sig)
+find_method_in_class (MonoClass *in_class, const char *name, const char *qname, const char *fqname,
+ MonoMethodSignature *sig, MonoClass *from_class)
{
int i;
- mono_class_setup_methods (klass);
- for (i = 0; i < klass->method.count; ++i) {
- MonoMethod *m = klass->methods [i];
+ mono_class_setup_methods (in_class);
+ for (i = 0; i < in_class->method.count; ++i) {
+ MonoMethod *m = in_class->methods [i];
if (!((fqname && !strcmp (m->name, fqname)) ||
(qname && !strcmp (m->name, qname)) || !strcmp (m->name, name)))
if (sig->call_convention == MONO_CALL_VARARG) {
if (mono_metadata_signature_vararg_match (sig, mono_method_signature (m)))
- return m;
+ break;
} else {
- MonoMethodSignature *msig = mono_method_signature (m);
- if (mono_metadata_signature_equal (sig, msig))
- return m;
+ if (mono_metadata_signature_equal (sig, mono_method_signature (m)))
+ break;
}
}
+
+ if (i < in_class->method.count) {
+ mono_class_setup_methods (from_class);
+ g_assert (from_class->method.count == in_class->method.count);
+ return from_class->methods [i];
+ }
return NULL;
}
static MonoMethod *
-find_method (MonoClass *klass, MonoClass *ic, const char* name, MonoMethodSignature *sig)
+find_method (MonoClass *in_class, MonoClass *ic, const char* name, MonoMethodSignature *sig, MonoClass *from_class)
{
int i;
char *qname, *fqname, *class_name;
gboolean is_interface;
MonoMethod *result = NULL;
- is_interface = MONO_CLASS_IS_INTERFACE (klass);
+ is_interface = MONO_CLASS_IS_INTERFACE (in_class);
if (ic) {
class_name = mono_type_get_name_full (&ic->byval_arg, MONO_TYPE_NAME_FORMAT_IL);
} else
class_name = qname = fqname = NULL;
- while (klass) {
- result = find_method_in_class (klass, name, qname, fqname, sig);
+ while (in_class) {
+ g_assert (from_class);
+ result = find_method_in_class (in_class, name, qname, fqname, sig, from_class);
if (result)
goto out;
if (name [0] == '.' && (!strcmp (name, ".ctor") || !strcmp (name, ".cctor")))
break;
- for (i = 0; i < klass->interface_count; i++) {
- MonoClass *ic = klass->interfaces [i];
+ g_assert (from_class->interface_count == in_class->interface_count);
+ for (i = 0; i < in_class->interface_count; i++) {
+ MonoClass *ic = in_class->interfaces [i];
+ MonoClass *from_ic = from_class->interfaces [i];
- result = find_method_in_class (ic, name, qname, fqname, sig);
+ result = find_method_in_class (ic, name, qname, fqname, sig, from_ic);
if (result)
goto out;
}
- klass = klass->parent;
+ in_class = in_class->parent;
+ from_class = from_class->parent;
}
+ g_assert (!in_class == !from_class);
if (is_interface)
- result = find_method_in_class (mono_defaults.object_class, name, qname, fqname, sig);
+ result = find_method_in_class (mono_defaults.object_class, name, qname, fqname, sig, mono_defaults.object_class);
out:
g_free (class_name);
return result;
}
+static MonoMethodSignature*
+inflate_generic_signature (MonoImage *image, MonoMethodSignature *sig, MonoGenericContext *context)
+{
+ MonoMethodSignature *res;
+ gboolean is_open;
+ int i;
+
+ if (!context)
+ return sig;
+
+ res = mono_metadata_signature_alloc (image, sig->param_count);
+ res->ret = mono_class_inflate_generic_type (sig->ret, context);
+ is_open = mono_class_is_open_constructed_type (res->ret);
+ for (i = 0; i < sig->param_count; ++i) {
+ res->params [i] = mono_class_inflate_generic_type (sig->params [i], context);
+ if (!is_open)
+ is_open = mono_class_is_open_constructed_type (res->params [i]);
+ }
+ res->hasthis = sig->hasthis;
+ res->explicit_this = sig->explicit_this;
+ res->call_convention = sig->call_convention;
+ res->pinvoke = sig->pinvoke;
+ res->generic_param_count = sig->generic_param_count;
+ res->sentinelpos = sig->sentinelpos;
+ res->has_type_parameters = is_open;
+ res->is_inflated = 1;
+ return res;
+}
+
+static MonoMethodHeader*
+inflate_generic_header (MonoMethodHeader *header, MonoGenericContext *context)
+{
+ MonoMethodHeader *res;
+ int i;
+ res = g_malloc0 (sizeof (MonoMethodHeader) + sizeof (gpointer) * header->num_locals);
+ res->code = header->code;
+ res->code_size = header->code_size;
+ res->max_stack = header->max_stack;
+ res->num_clauses = header->num_clauses;
+ res->init_locals = header->init_locals;
+ res->num_locals = header->num_locals;
+ res->clauses = header->clauses;
+ for (i = 0; i < header->num_locals; ++i)
+ res->locals [i] = mono_class_inflate_generic_type (header->locals [i], context);
+ return res;
+}
+
/*
* token is the method_ref or method_def token used in a call IL instruction.
*/
mono_loader_unlock ();
if (!sig) {
mono_metadata_decode_row (&image->tables [MONO_TABLE_MEMBERREF], idx-1, cols, MONO_MEMBERREF_SIZE);
-
+
ptr = mono_metadata_blob_heap (image, cols [MONO_MEMBERREF_SIGNATURE]);
mono_metadata_decode_blob_size (ptr, &ptr);
sig = mono_metadata_parse_method_signature_full (
mono_loader_unlock ();
}
- sig = mono_class_inflate_generic_signature (image, sig, context);
+ sig = inflate_generic_signature (image, sig, context);
return sig;
}
static MonoMethod *
method_from_memberref (MonoImage *image, guint32 idx, MonoGenericContext *typespec_context)
{
- MonoGenericContext *context = NULL;
MonoClass *klass = NULL;
MonoMethod *method = NULL;
MonoTableInfo *tables = image->tables;
if (!klass) {
char *name = mono_class_name_from_token (image, MONO_TOKEN_TYPE_REF | nindex);
g_warning ("Missing method %s in assembly %s, type %s", mname, image->name, name);
+ mono_loader_set_error_method_load (name, mname);
g_free (name);
return NULL;
}
if (!klass) {
char *name = mono_class_name_from_token (image, MONO_TOKEN_TYPE_SPEC | nindex);
g_warning ("Missing method %s in assembly %s, type %s", mname, image->name, name);
+ mono_loader_set_error_method_load (name, mname);
g_free (name);
return NULL;
}
if (!klass) {
char *name = mono_class_name_from_token (image, MONO_TOKEN_TYPE_DEF | nindex);
g_warning ("Missing method %s in assembly %s, type %s", mname, image->name, name);
+ mono_loader_set_error_method_load (name, mname);
g_free (name);
return NULL;
}
break;
case MONO_MEMBERREF_PARENT_METHODDEF:
return mono_get_method (image, MONO_TOKEN_METHOD_DEF | nindex, NULL);
+
default:
- g_error ("Memberref parent unknown: class: %d, index %d", class, nindex);
- g_assert_not_reached ();
+ {
+ /* This message leaks */
+ char *message = g_strdup_printf ("Memberref parent unknown: class: %d, index %d", class, nindex);
+ mono_loader_set_error_method_load ("", message);
+ return NULL;
+ }
+
}
g_assert (klass);
mono_class_init (klass);
mono_metadata_decode_blob_size (ptr, &ptr);
sig = mono_metadata_parse_method_signature (image, 0, ptr, NULL);
-
- /* If the class is a generic instantiation, ensure that all VAR references in the signature
- are replaced by the corresponding type argument. Note that MVARs are left alone. */
- if (klass->generic_class && sig->has_type_parameters) {
- MonoMethodSignature *old_sig = sig;
- sig = mono_class_inflate_generic_signature (image, sig, klass->generic_class->context);
- g_assert (sig != old_sig);
- mono_metadata_free_method_signature (old_sig);
- }
+ if (sig == NULL)
+ return NULL;
switch (class) {
case MONO_MEMBERREF_PARENT_TYPEREF:
- method = find_method (klass, NULL, mname, sig);
- if (!method)
- mono_loader_set_error_method_load (klass, mname);
- mono_metadata_free_method_signature (sig);
+ case MONO_MEMBERREF_PARENT_TYPEDEF:
+ method = find_method (klass, NULL, mname, sig, klass);
break;
+
case MONO_MEMBERREF_PARENT_TYPESPEC: {
MonoType *type;
MonoMethod *result;
type = &klass->byval_arg;
if (type->type != MONO_TYPE_ARRAY && type->type != MONO_TYPE_SZARRAY) {
- method = find_method (klass, NULL, mname, sig);
- if (!method)
- g_warning ("Missing method %s in assembly %s, type %s", mname, image->name, mono_class_get_name (klass));
- else if (klass->generic_class && (klass != method->klass))
- method = mono_class_inflate_generic_method (
- method, klass->generic_class->context);
- mono_metadata_free_method_signature (sig);
+ MonoClass *in_class = klass->generic_class ? klass->generic_class->container_class : klass;
+ method = find_method (in_class, NULL, mname, sig, klass);
break;
}
result = (MonoMethod *)g_new0 (MonoMethodPInvoke, 1);
result->klass = klass;
result->iflags = METHOD_IMPL_ATTRIBUTE_INTERNAL_CALL;
+ result->flags = METHOD_ATTRIBUTE_PUBLIC;
result->signature = sig;
result->name = mname;
/* we special-case this in the runtime. */
return result;
}
-
+
if (!strcmp (mname, "Set")) {
g_assert (sig->hasthis);
g_assert (type->data.array->rank + 1 == sig->param_count);
g_assert_not_reached ();
break;
}
- case MONO_MEMBERREF_PARENT_TYPEDEF:
- method = find_method (klass, NULL, mname, sig);
- if (!method)
- g_warning (
- "Missing method %s::%s(%s) in assembly %s",
- mono_type_get_name (&klass->byval_arg), mname, mono_signature_get_desc (sig, FALSE), image->name);
- mono_metadata_free_method_signature (sig);
- break;
default:
g_error ("Memberref parent unknown: class: %d, index %d", class, nindex);
g_assert_not_reached ();
}
+ if (!method) {
+ char *msig = mono_signature_get_desc (sig, FALSE);
+ char * class_name = mono_type_get_name (&klass->byval_arg);
+ GString *s = g_string_new (mname);
+ if (sig->generic_param_count)
+ g_string_append_printf (s, "<[%d]>", sig->generic_param_count);
+ g_string_append_printf (s, "(%s)", msig);
+ g_free (msig);
+ msig = g_string_free (s, FALSE);
+
+ g_warning (
+ "Missing method %s::%s in assembly %s, referenced in assembly %s",
+ class_name, msig, klass->image->name, image->name);
+ mono_loader_set_error_method_load (class_name, mname);
+ g_free (msig);
+ g_free (class_name);
+ }
+ mono_metadata_free_method_signature (sig);
+
return method;
}
nindex = token >> MONO_METHODDEFORREF_BITS;
ptr = mono_metadata_blob_heap (image, cols [MONO_METHODSPEC_SIGNATURE]);
-
+
mono_metadata_decode_value (ptr, &ptr);
ptr++;
param_count = mono_metadata_decode_value (ptr, &ptr);
* ie. instantiate the method as `Foo.Hello<float>.
*/
- gmethod->inst = mono_metadata_parse_generic_inst (image, context, param_count, ptr, &ptr);
+ gmethod->inst = mono_metadata_parse_generic_inst (image, context ? context->container : NULL, param_count, ptr, &ptr);
- if (context)
+ if (context && gmethod->inst->is_open)
gmethod->inst = mono_metadata_inflate_generic_inst (gmethod->inst, context);
if (!container->method_hash)
mono_stats.generics_metadata_size += sizeof (MonoGenericMethod) +
sizeof (MonoGenericContext) + param_count * sizeof (MonoType);
- inflated = mono_class_inflate_generic_method (method, new_context);
+ inflated = mono_class_inflate_generic_method_full (method, method->klass, new_context);
g_hash_table_insert (container->method_hash, gmethod, inflated);
- new_context->gclass = inflated->klass->generic_class;
return inflated;
}
return 0;
}
*rdll = map->target? map->target: dll;
-
+
for (tmp = map->next; tmp; tmp = tmp->next) {
if (strcmp (func, tmp->name) == 0) {
*rfunc = tmp->name;
/* we allow a special name to dlopen from the running process namespace */
if (strcmp (new_scope, "__Internal") == 0)
gmodule = g_module_open (NULL, G_MODULE_BIND_LAZY);
-
+
/*
* Try loading the module using a variety of names
*/
#ifdef PLATFORM_WIN32
if (mangle_charset == 0)
mangled_name = g_strconcat (import, "W", NULL);
+#else
+ /* Try the mangled name last */
+ if (mangle_charset == 1)
+ mangled_name = g_strconcat (import, "A", NULL);
#endif
break;
case PINVOKE_ATTRIBUTE_CHAR_SET_ANSI:
return piinfo->addr;
}
+MonoGenericMethod *
+mono_get_shared_generic_method (MonoGenericContainer *container)
+{
+ MonoGenericMethod *gmethod = g_new0 (MonoGenericMethod, 1);
+ gmethod->container = container;
+ gmethod->generic_class = container->context.gclass;
+ gmethod->inst = mono_get_shared_generic_inst (container);
+
+ return gmethod;
+}
+
static MonoMethod *
mono_get_method_from_token (MonoImage *image, guint32 token, MonoClass *klass,
MonoGenericContext *context)
if ((cols [2] & METHOD_ATTRIBUTE_PINVOKE_IMPL) ||
(cols [1] & METHOD_IMPL_ATTRIBUTE_INTERNAL_CALL))
- result = (MonoMethod *)g_new0 (MonoMethodPInvoke, 1);
- else
- result = (MonoMethod *)g_new0 (MonoMethodNormal, 1);
+ result = (MonoMethod *)mono_mempool_alloc0 (image->mempool, sizeof (MonoMethodPInvoke));
+ else
+ result = (MonoMethod *)mono_mempool_alloc0 (image->mempool, sizeof (MonoMethodNormal));
if (!klass) {
guint32 type = mono_metadata_typedef_from_method (image, token);
klass = mono_class_get (image, MONO_TOKEN_TYPE_DEF | type);
+ if (klass == NULL)
+ return NULL;
}
-
+
result->slot = -1;
result->klass = klass;
result->flags = cols [2];
container = klass->generic_container;
generic_container = mono_metadata_load_generic_params (image, token, container);
if (generic_container) {
+ MonoGenericContext *context = &generic_container->context;
+ if (container)
+ context->gclass = container->context.gclass;
+ context->gmethod = mono_get_shared_generic_method (generic_container);
mono_metadata_load_generic_param_constraints (image, token, generic_container);
for (i = 0; i < generic_container->type_argc; i++) {
if (!sig) /* already taken from the methodref */
sig = mono_metadata_blob_heap (image, cols [4]);
size = mono_metadata_decode_blob_size (sig, &sig);
-
- /* there are generic params, or a container. FIXME: be lazy here for generics*/
- if (* sig & 0x10 || container) {
- result->signature = mono_metadata_parse_method_signature_full (
- image, container, idx, sig, NULL);
-
- /* Verify metadata consistency */
- if (result->signature->generic_param_count) {
- if (!container || !container->is_method)
- g_error ("Signature claims method has generic parameters, but generic_params table says it doesn't");
- if (container->type_argc != result->signature->generic_param_count)
- g_error ("Inconsistent generic parameter count. Signature says %d, generic_params table says %d",
- result->signature->generic_param_count, container->type_argc);
- } else if (container && container->is_method && container->type_argc)
- g_error ("generic_params table claims method has generic parameters, but signature says it doesn't");
-
- if (cols [1] & METHOD_IMPL_ATTRIBUTE_INTERNAL_CALL)
- result->signature->pinvoke = 1;
- }
if (cols [1] & METHOD_IMPL_ATTRIBUTE_INTERNAL_CALL) {
if (result->klass == mono_defaults.string_class && !strcmp (result->name, ".ctor"))
} else if ((cols [2] & METHOD_ATTRIBUTE_PINVOKE_IMPL) && (!(cols [1] & METHOD_IMPL_ATTRIBUTE_NATIVE))) {
MonoMethodPInvoke *piinfo = (MonoMethodPInvoke *)result;
MonoTableInfo *im = &tables [MONO_TABLE_IMPLMAP];
-
+
piinfo->implmap_idx = mono_metadata_implmap_from_method (image, idx - 1);
piinfo->piflags = mono_metadata_decode_row_col (im, piinfo->implmap_idx - 1, MONO_IMPLMAP_FLAGS);
}
-
+
/* FIXME: lazyness for generics too, but how? */
- if (!result->klass->dummy && !(result->flags & METHOD_ATTRIBUTE_ABSTRACT) &&
+ if (!(result->flags & METHOD_ATTRIBUTE_ABSTRACT) &&
!(cols [1] & METHOD_IMPL_ATTRIBUTE_INTERNAL_CALL) &&
!(result->iflags & METHOD_IMPL_ATTRIBUTE_RUNTIME) && container) {
gpointer loc = mono_image_rva_map (image, cols [0]);
g_assert (loc);
- ((MonoMethodNormal *) result)->header = mono_metadata_parse_mh_full (
- image, (MonoGenericContext *) container, loc);
+ ((MonoMethodNormal *) result)->header = mono_metadata_parse_mh_full (image, container, loc);
}
-
+
result->generic_container = generic_container;
return result;
{
MonoMethod *method, *result;
MonoClass *ic = NULL;
- MonoGenericClass *gclass = NULL;
+ MonoGenericContext *class_context = NULL, *method_context = NULL;
+ MonoMethodSignature *sig;
mono_loader_lock ();
mono_class_init (constrained_class);
method = mono_get_inflated_method (method);
+ sig = mono_method_signature (method);
+
+ if (method->is_inflated && sig->generic_param_count) {
+ MonoMethodInflated *imethod = (MonoMethodInflated *) method;
+ sig = mono_method_signature (imethod->declaring);
+ method_context = imethod->context;
+ }
if ((constrained_class != method->klass) && (method->klass->interface_id != 0))
ic = method->klass;
if (constrained_class->generic_class)
- gclass = constrained_class->generic_class;
+ class_context = constrained_class->generic_class->context;
- result = find_method (constrained_class, ic, method->name, mono_method_signature (method));
- if (!result)
- g_warning ("Missing method %s in assembly %s token %x", method->name,
- image->name, token);
+ result = find_method (constrained_class, ic, method->name, sig, constrained_class);
+ if (!result) {
+ g_warning ("Missing method %s.%s.%s in assembly %s token %x", method->klass->name_space,
+ method->klass->name, method->name, image->name, token);
+ mono_loader_unlock ();
+ return NULL;
+ }
- if (gclass)
- result = mono_class_inflate_generic_method (result, gclass->context);
+ if (class_context)
+ result = mono_class_inflate_generic_method (result, class_context);
+ if (method_context)
+ result = mono_class_inflate_generic_method (result, method_context);
mono_loader_unlock ();
return result;
* locals are shared.
*/
/* mono_metadata_free_method_signature (method->signature); */
- g_free (method->signature);
+ /* g_free (method->signature); */
}
-
+
if (method->dynamic) {
MonoMethodWrapper *mw = (MonoMethodWrapper*)method;
-
+
g_free ((char*)method->name);
if (mw->method.header)
g_free ((char*)mw->method.header->code);
g_free (mw->method_data);
}
- if (!(method->iflags & METHOD_IMPL_ATTRIBUTE_INTERNAL_CALL) && ((MonoMethodNormal *)method)->header) {
+ if (method->dynamic && !(method->iflags & METHOD_IMPL_ATTRIBUTE_INTERNAL_CALL) && ((MonoMethodNormal *)method)->header) {
/* FIXME: Ditto */
/* mono_metadata_free_mh (((MonoMethodNormal *)method)->header); */
g_free (((MonoMethodNormal*)method)->header);
}
- g_free (method);
+ if (method->dynamic)
+ g_free (method);
}
void
if (idx > 0) {
guint32 cols [MONO_PARAM_SIZE];
guint param_index = mono_metadata_decode_row_col (methodt, idx - 1, MONO_METHOD_PARAMLIST);
-
+
if (idx + 1 < methodt->rows)
lastp = mono_metadata_decode_row_col (methodt, idx, MONO_METHOD_PARAMLIST);
else
LeaveCriticalSection (&loader_mutex);
}
-MonoMethodSignature*
+/**
+ * mono_method_signature:
+ *
+ * Return the signature of the method M. On failure, returns NULL.
+ */
+MonoMethodSignature*
mono_method_signature (MonoMethod *m)
-{
- m = mono_get_inflated_method (m);
- return mono_method_signature_full (m, NULL);
-}
-
-MonoMethodSignature*
-mono_method_signature_full (MonoMethod *m, MonoGenericContainer *container)
{
int idx;
int size;
MonoImage* img;
const char *sig;
-
+ gboolean can_cache_signature;
+ MonoGenericContainer *container;
+ int *pattrs;
+
if (m->signature)
return m->signature;
-
+
mono_loader_lock ();
-
+
if (m->signature) {
mono_loader_unlock ();
return m->signature;
}
-
+
+ if (m->is_inflated) {
+ MonoMethodInflated *imethod = (MonoMethodInflated *) m;
+ MonoMethodSignature *signature;
+ /* the lock is recursive */
+ signature = mono_method_signature (imethod->declaring);
+ m->signature = inflate_generic_signature (imethod->declaring->klass->image, signature, imethod->context);
+ mono_loader_unlock ();
+ return m->signature;
+ }
+
g_assert (mono_metadata_token_table (m->token) == MONO_TABLE_METHOD);
idx = mono_metadata_token_index (m->token);
img = m->klass->image;
-
+
sig = mono_metadata_blob_heap (img, mono_metadata_decode_row_col (&img->tables [MONO_TABLE_METHOD], idx - 1, MONO_METHOD_SIGNATURE));
- size = mono_metadata_decode_blob_size (sig, &sig);
-
- m->signature = mono_metadata_parse_method_signature_full (img, container, idx, sig, NULL);
-
+
+ g_assert (!m->klass->generic_class);
+ container = m->generic_container;
+ if (!container)
+ container = m->klass->generic_container;
+
+ /* Generic signatures depend on the container so they cannot be cached */
+ /* icall/pinvoke signatures cannot be cached cause we modify them below */
+ can_cache_signature = !(m->iflags & METHOD_IMPL_ATTRIBUTE_INTERNAL_CALL) && !(m->flags & METHOD_ATTRIBUTE_PINVOKE_IMPL) && !container;
+
+ /* If the method has parameter attributes, that can modify the signature */
+ pattrs = mono_metadata_get_param_attrs (img, idx);
+ if (pattrs) {
+ can_cache_signature = FALSE;
+ g_free (pattrs);
+ }
+
+ if (can_cache_signature)
+ m->signature = g_hash_table_lookup (img->method_signatures, sig);
+
+ if (!m->signature) {
+ const char *sig_body;
+
+ size = mono_metadata_decode_blob_size (sig, &sig_body);
+
+ m->signature = mono_metadata_parse_method_signature_full (img, container, idx, sig_body, NULL);
+ if (!m->signature) {
+ mono_loader_unlock ();
+ return NULL;
+ }
+
+ if (can_cache_signature)
+ g_hash_table_insert (img->method_signatures, (gpointer)sig, m->signature);
+ }
+
+ /* Verify metadata consistency */
+ if (m->signature->generic_param_count) {
+ if (!container || !container->is_method)
+ g_error ("Signature claims method has generic parameters, but generic_params table says it doesn't");
+ if (container->type_argc != m->signature->generic_param_count)
+ g_error ("Inconsistent generic parameter count. Signature says %d, generic_params table says %d",
+ m->signature->generic_param_count, container->type_argc);
+ } else if (container && container->is_method && container->type_argc)
+ g_error ("generic_params table claims method has generic parameters, but signature says it doesn't");
+
if (m->iflags & METHOD_IMPL_ATTRIBUTE_INTERNAL_CALL)
m->signature->pinvoke = 1;
else if ((m->flags & METHOD_ATTRIBUTE_PINVOKE_IMPL) && (!(m->iflags & METHOD_IMPL_ATTRIBUTE_NATIVE))) {
MonoCallConvention conv = 0;
MonoMethodPInvoke *piinfo = (MonoMethodPInvoke *)m;
m->signature->pinvoke = 1;
-
+
switch (piinfo->piflags & PINVOKE_ATTRIBUTE_CALL_CONV_MASK) {
case 0: /* no call conv, so using default */
case PINVOKE_ATTRIBUTE_CALL_CONV_WINAPI:
default:
g_warning ("unsupported calling convention : 0x%04x", piinfo->piflags);
g_assert_not_reached ();
- }
+ }
m->signature->call_convention = conv;
}
-
+
mono_loader_unlock ();
return m->signature;
}
return method->token;
}
-MonoMethodHeader*
+MonoMethodHeader*
mono_method_get_header (MonoMethod *method)
{
int idx;
MonoImage* img;
gpointer loc;
MonoMethodNormal* mn = (MonoMethodNormal*) method;
-
- if (method->klass->dummy || (method->flags & METHOD_ATTRIBUTE_ABSTRACT) || (method->iflags & METHOD_IMPL_ATTRIBUTE_RUNTIME) || (method->iflags & METHOD_IMPL_ATTRIBUTE_INTERNAL_CALL) || (method->flags & METHOD_ATTRIBUTE_PINVOKE_IMPL))
+
+ if ((method->flags & METHOD_ATTRIBUTE_ABSTRACT) || (method->iflags & METHOD_IMPL_ATTRIBUTE_RUNTIME) || (method->iflags & METHOD_IMPL_ATTRIBUTE_INTERNAL_CALL) || (method->flags & METHOD_ATTRIBUTE_PINVOKE_IMPL))
return NULL;
-
+
#ifdef G_LIKELY
if (G_LIKELY (mn->header))
#else
if (mn->header)
#endif
return mn->header;
-
+
mono_loader_lock ();
-
+
if (mn->header) {
mono_loader_unlock ();
return mn->header;
}
-
+
+ if (method->is_inflated) {
+ MonoMethodInflated *imethod = (MonoMethodInflated *) method;
+ MonoMethodHeader *header;
+ /* the lock is recursive */
+ header = mono_method_get_header (imethod->declaring);
+ mn->header = inflate_generic_header (header, imethod->context);
+ mono_loader_unlock ();
+ return mn->header;
+ }
+
g_assert (mono_metadata_token_table (method->token) == MONO_TABLE_METHOD);
idx = mono_metadata_token_index (method->token);
img = method->klass->image;
rva = mono_metadata_decode_row_col (&img->tables [MONO_TABLE_METHOD], idx - 1, MONO_METHOD_RVA);
loc = mono_image_rva_map (img, rva);
-
+
g_assert (loc);
-
- mn->header = mono_metadata_parse_mh_full (img, (MonoGenericContext *) method->generic_container, loc);
-
+
+ mn->header = mono_metadata_parse_mh_full (img, method->generic_container, loc);
+
mono_loader_unlock ();
return mn->header;
}
projects / mono.git / blobdiff