2006-10-06 Zoltan Varga <vargaz@gmail.com>
[mono.git] / mono / metadata / metadata.c
index 516709de8d2d3c4cc39804b390d5e41208f9684d..5b41c7add2d2e18e18dd797419dbab13e2e7f780 100644 (file)
@@ -27,6 +27,7 @@ static gboolean do_mono_metadata_parse_type (MonoType *type, MonoImage *m, MonoG
 
 static gboolean do_mono_metadata_type_equal (MonoType *t1, MonoType *t2, gboolean signature_only);
 static gboolean mono_metadata_class_equal (MonoClass *c1, MonoClass *c2, gboolean signature_only);
+static gboolean mono_metadata_fnptr_equal (MonoMethodSignature *s1, MonoMethodSignature *s2, gboolean signature_only);
 static gboolean _mono_metadata_generic_class_equal (const MonoGenericClass *g1, const MonoGenericClass *g2,
                                                    gboolean signature_only);
 
@@ -174,7 +175,11 @@ const static unsigned char TableSchemas [] = {
        MONO_MT_TABLE_IDX,  /* "EventType" }, TypeDef or TypeRef  */
        MONO_MT_END,
 
-#define EXPORTED_TYPE_SCHEMA_OFFSET EVENT_SCHEMA_OFFSET + 4
+#define EVENT_POINTER_SCHEMA_OFFSET EVENT_SCHEMA_OFFSET + 4
+       MONO_MT_TABLE_IDX,  /* "Event" }, */
+       MONO_MT_END,
+
+#define EXPORTED_TYPE_SCHEMA_OFFSET EVENT_POINTER_SCHEMA_OFFSET + 2
        MONO_MT_UINT32,     /* "Flags" }, */
        MONO_MT_TABLE_IDX,  /* "TypeDefId" }, */
        MONO_MT_STRING_IDX, /* "TypeName" }, */
@@ -203,7 +208,11 @@ const static unsigned char TableSchemas [] = {
        MONO_MT_TABLE_IDX,  /* "Field:Field" }, */
        MONO_MT_END,
 
-#define FILE_SCHEMA_OFFSET FIELD_RVA_SCHEMA_OFFSET + 3
+#define FIELD_POINTER_SCHEMA_OFFSET FIELD_RVA_SCHEMA_OFFSET + 3
+       MONO_MT_TABLE_IDX,  /* "Field" }, */
+       MONO_MT_END,
+
+#define FILE_SCHEMA_OFFSET FIELD_POINTER_SCHEMA_OFFSET + 2
        MONO_MT_UINT32,     /* "Flags" }, */
        MONO_MT_STRING_IDX, /* "Name" }, */
        MONO_MT_BLOB_IDX,   /* "Value" },  */
@@ -255,7 +264,11 @@ const static unsigned char TableSchemas [] = {
        MONO_MT_HS_IDX,     /* "Association" }, */
        MONO_MT_END,
 
-#define MODULE_SCHEMA_OFFSET METHOD_SEMA_SCHEMA_OFFSET + 4
+#define METHOD_POINTER_SCHEMA_OFFSET METHOD_SEMA_SCHEMA_OFFSET + 4
+       MONO_MT_TABLE_IDX,  /* "Method" }, */
+       MONO_MT_END,
+
+#define MODULE_SCHEMA_OFFSET METHOD_POINTER_SCHEMA_OFFSET + 2
        MONO_MT_UINT16,     /* "Generation" }, */
        MONO_MT_STRING_IDX, /* "Name" }, */
        MONO_MT_GUID_IDX,   /* "MVID" }, */
@@ -278,13 +291,21 @@ const static unsigned char TableSchemas [] = {
        MONO_MT_STRING_IDX, /* "Name" }, */
        MONO_MT_END,
 
-#define PROPERTY_SCHEMA_OFFSET PARAM_SCHEMA_OFFSET + 4
+#define PARAM_POINTER_SCHEMA_OFFSET PARAM_SCHEMA_OFFSET + 4
+       MONO_MT_TABLE_IDX,  /* "Param" }, */
+       MONO_MT_END,
+
+#define PROPERTY_SCHEMA_OFFSET PARAM_POINTER_SCHEMA_OFFSET + 2
        MONO_MT_UINT16,     /* "Flags" }, */
        MONO_MT_STRING_IDX, /* "Name" }, */
        MONO_MT_BLOB_IDX,   /* "Type" }, */
        MONO_MT_END,
 
-#define PROPERTY_MAP_SCHEMA_OFFSET PROPERTY_SCHEMA_OFFSET + 4
+#define PROPERTY_POINTER_SCHEMA_OFFSET PROPERTY_SCHEMA_OFFSET + 4
+       MONO_MT_TABLE_IDX, /* "Property" }, */
+       MONO_MT_END,
+
+#define PROPERTY_MAP_SCHEMA_OFFSET PROPERTY_POINTER_SCHEMA_OFFSET + 2
        MONO_MT_TABLE_IDX,  /* "Parent:TypeDef" }, */
        MONO_MT_TABLE_IDX,  /* "PropertyList:Property" }, */
        MONO_MT_END,
@@ -339,11 +360,11 @@ table_description [] = {
        MODULE_SCHEMA_OFFSET,
        TYPEREF_SCHEMA_OFFSET,
        TYPEDEF_SCHEMA_OFFSET,
-       NULL_SCHEMA_OFFSET,
+       FIELD_POINTER_SCHEMA_OFFSET,
        FIELD_SCHEMA_OFFSET,
-       NULL_SCHEMA_OFFSET,
+       METHOD_POINTER_SCHEMA_OFFSET,
        METHOD_SCHEMA_OFFSET,
-       NULL_SCHEMA_OFFSET,
+       PARAM_POINTER_SCHEMA_OFFSET,
        PARAM_SCHEMA_OFFSET,
        IFACEMAP_SCHEMA_OFFSET,
        MEMBERREF_SCHEMA_OFFSET, /* 0xa */
@@ -355,10 +376,10 @@ table_description [] = {
        FIELD_LAYOUT_SCHEMA_OFFSET, /* 0x10 */
        STDALON_SIG_SCHEMA_OFFSET,
        EVENTMAP_SCHEMA_OFFSET,
-       NULL_SCHEMA_OFFSET,
+       EVENT_POINTER_SCHEMA_OFFSET,
        EVENT_SCHEMA_OFFSET,
        PROPERTY_MAP_SCHEMA_OFFSET,
-       NULL_SCHEMA_OFFSET,
+       PROPERTY_POINTER_SCHEMA_OFFSET,
        PROPERTY_SCHEMA_OFFSET,
        METHOD_SEMA_SCHEMA_OFFSET,
        METHOD_IMPL_SCHEMA_OFFSET,
@@ -511,6 +532,9 @@ mono_metadata_compute_size (MonoImage *meta, int tableindex, guint32 *result_bit
                                field_size = MAX (idx_size (MONO_TABLE_TYPEDEF), idx_size(MONO_TABLE_TYPEREF));
                                field_size = MAX (field_size, idx_size(MONO_TABLE_TYPESPEC));
                                break;
+                       case MONO_TABLE_EVENT_POINTER:
+                               g_assert (i == 0);
+                               field_size = idx_size (MONO_TABLE_EVENT); break;
                        case MONO_TABLE_EXPORTEDTYPE:
                                g_assert (i == 1);
                                /* the index is in another metadata file, so it must be 4 */
@@ -521,6 +545,9 @@ mono_metadata_compute_size (MonoImage *meta, int tableindex, guint32 *result_bit
                        case MONO_TABLE_FIELDRVA:
                                g_assert (i == 1);
                                field_size = idx_size (MONO_TABLE_FIELD); break;
+                       case MONO_TABLE_FIELD_POINTER:
+                               g_assert (i == 0);
+                               field_size = idx_size (MONO_TABLE_FIELD); break;
                        case MONO_TABLE_IMPLMAP:
                                g_assert (i == 3);
                                field_size = idx_size (MONO_TABLE_MODULEREF); break;
@@ -536,14 +563,23 @@ mono_metadata_compute_size (MonoImage *meta, int tableindex, guint32 *result_bit
                        case MONO_TABLE_METHODSEMANTICS:
                                g_assert (i == 1);
                                field_size = idx_size (MONO_TABLE_METHOD); break;
+                       case MONO_TABLE_METHOD_POINTER:
+                               g_assert (i == 0);
+                               field_size = idx_size (MONO_TABLE_METHOD); break;
                        case MONO_TABLE_NESTEDCLASS:
                                g_assert (i == 0 || i == 1);
                                field_size = idx_size (MONO_TABLE_TYPEDEF); break;
+                       case MONO_TABLE_PARAM_POINTER:
+                               g_assert (i == 0);
+                               field_size = idx_size (MONO_TABLE_PARAM); break;
                        case MONO_TABLE_PROPERTYMAP:
                                g_assert (i == 0 || i == 1);
                                field_size = i ? idx_size (MONO_TABLE_PROPERTY):
                                        idx_size(MONO_TABLE_TYPEDEF); 
                                break;
+                       case MONO_TABLE_PROPERTY_POINTER:
+                               g_assert (i == 0);
+                               field_size = idx_size (MONO_TABLE_PROPERTY); break;
                        case MONO_TABLE_TYPEDEF:
                                g_assert (i == 4 || i == 5);
                                field_size = i == 4 ? idx_size (MONO_TABLE_FIELD):
@@ -966,6 +1002,7 @@ mono_metadata_decode_row_col (const MonoTableInfo *t, int idx, guint col)
        }
        return 0;
 }
+
 /**
  * mono_metadata_decode_blob_size:
  * @ptr: pointer to a blob object
@@ -1067,6 +1104,81 @@ mono_metadata_decode_signed_value (const char *ptr, const char **rptr)
        return ival - 0x20000000;
 }
 
+/* 
+ * Translates the given 1-based index into the Method, Field, Event, or Param tables
+ * using the *Ptr tables in uncompressed metadata, if they are available.
+ *
+ * FIXME: The caller is not forced to call this function, which is error-prone, since 
+ * forgetting to call it would only show up as a bug on uncompressed metadata.
+ */
+guint32
+mono_metadata_translate_token_index (MonoImage *image, int table, guint32 idx)
+{
+       if (!image->uncompressed_metadata)
+               return idx;
+
+       switch (table) {
+       case MONO_TABLE_METHOD:
+               if (image->tables [MONO_TABLE_METHOD_POINTER].rows)
+                       return mono_metadata_decode_row_col (&image->tables [MONO_TABLE_METHOD_POINTER], idx - 1, MONO_METHOD_POINTER_METHOD);
+               else
+                       return idx;
+       case MONO_TABLE_FIELD:
+               if (image->tables [MONO_TABLE_FIELD_POINTER].rows)
+                       return mono_metadata_decode_row_col (&image->tables [MONO_TABLE_FIELD_POINTER], idx - 1, MONO_FIELD_POINTER_FIELD);
+               else
+                       return idx;
+       case MONO_TABLE_EVENT:
+               if (image->tables [MONO_TABLE_EVENT_POINTER].rows)
+                       return mono_metadata_decode_row_col (&image->tables [MONO_TABLE_EVENT_POINTER], idx - 1, MONO_EVENT_POINTER_EVENT);
+               else
+                       return idx;
+       case MONO_TABLE_PROPERTY:
+               if (image->tables [MONO_TABLE_PROPERTY_POINTER].rows)
+                       return mono_metadata_decode_row_col (&image->tables [MONO_TABLE_PROPERTY_POINTER], idx - 1, MONO_PROPERTY_POINTER_PROPERTY);
+               else
+                       return idx;
+       case MONO_TABLE_PARAM:
+               if (image->tables [MONO_TABLE_PARAM_POINTER].rows)
+                       return mono_metadata_decode_row_col (&image->tables [MONO_TABLE_PARAM_POINTER], idx - 1, MONO_PARAM_POINTER_PARAM);
+               else
+                       return idx;
+       default:
+               return idx;
+       }
+}
+
+/**
+ * mono_metadata_decode_table_row:
+ *
+ *   Same as mono_metadata_decode_row, but takes an IMAGE+TABLE ID pair, and takes
+ * uncompressed metadata into account, so it should be used to access the
+ * Method, Field, Param and Event tables.
+ */
+void
+mono_metadata_decode_table_row (MonoImage *image, int table, int idx, guint32 *res, int res_size)
+{
+       if (image->uncompressed_metadata)
+               idx = mono_metadata_translate_token_index (image, table, idx + 1) - 1;
+
+       mono_metadata_decode_row (&image->tables [table], idx, res, res_size);
+}
+
+/**
+ * mono_metadata_decode_table_row_col:
+ *
+ *   Same as mono_metadata_decode_row_col, but takes an IMAGE+TABLE ID pair, and takes
+ * uncompressed metadata into account, so it should be used to access the
+ * Method, Field, Param and Event tables.
+ */
+guint32 mono_metadata_decode_table_row_col (MonoImage *image, int table, int idx, guint col)
+{
+       if (image->uncompressed_metadata)
+               idx = mono_metadata_translate_token_index (image, table, idx + 1) - 1;
+
+       return mono_metadata_decode_row_col (&image->tables [table], idx, col);
+}
+
 /*
  * mono_metadata_parse_typedef_or_ref:
  * @m: a metadata context.
@@ -1517,16 +1629,16 @@ mono_metadata_get_param_attrs (MonoImage *m, int def)
        MonoTableInfo *paramt = &m->tables [MONO_TABLE_PARAM];
        MonoTableInfo *methodt = &m->tables [MONO_TABLE_METHOD];
        guint32 cols [MONO_PARAM_SIZE];
-       guint lastp, i, param_index = mono_metadata_decode_row_col (methodt, def - 1, MONO_METHOD_PARAMLIST);
+       guint lastp, i, param_index = mono_metadata_decode_table_row_col (m, MONO_TABLE_METHOD, def - 1, MONO_METHOD_PARAMLIST);
        int *pattrs = NULL;
 
        if (def < methodt->rows)
-               lastp = mono_metadata_decode_row_col (methodt, def, MONO_METHOD_PARAMLIST);
+               lastp = mono_metadata_decode_table_row_col (m, MONO_TABLE_METHOD, def, MONO_METHOD_PARAMLIST);
        else
                lastp = paramt->rows + 1;
 
        for (i = param_index; i < lastp; ++i) {
-               mono_metadata_decode_row (paramt, i - 1, cols, MONO_PARAM_SIZE);
+               mono_metadata_decode_table_row (m, MONO_TABLE_PARAM, i - 1, cols, MONO_PARAM_SIZE);
                if (cols [MONO_PARAM_FLAGS]) {
                        if (!pattrs)
                                pattrs = g_new0 (int, 1 + (lastp - param_index));
@@ -1687,6 +1799,8 @@ mono_metadata_parse_method_signature_full (MonoImage *m, MonoGenericContainer *c
                if (*ptr == MONO_TYPE_SENTINEL) {
                        if (method->call_convention != MONO_CALL_VARARG || def)
                                g_error ("found sentinel for methoddef or no vararg method");
+                       if (method->sentinelpos >= 0)
+                               g_error ("found sentinel twice in the same signature");
                        method->sentinelpos = i;
                        ptr++;
                }
@@ -1700,6 +1814,10 @@ mono_metadata_parse_method_signature_full (MonoImage *m, MonoGenericContainer *c
                        is_open = mono_class_is_open_constructed_type (method->params [i]);
        }
 
+       /* The sentinel could be missing if the caller does not pass any additional arguments */
+       if (!def && method->call_convention == MONO_CALL_VARARG && method->sentinelpos < 0)
+               method->sentinelpos = method->param_count;
+
        method->has_type_parameters = is_open;
 
        if (def && (method->call_convention == MONO_CALL_VARARG))
@@ -1880,7 +1998,7 @@ mono_metadata_parse_generic_inst (MonoImage *m, MonoGenericContainer *container,
        return mono_metadata_lookup_generic_inst (ginst);
 }
 
-static void
+static gboolean
 do_mono_metadata_parse_generic_class (MonoType *type, MonoImage *m, MonoGenericContainer *container,
                                      const char *ptr, const char **rptr)
 {
@@ -1907,6 +2025,8 @@ do_mono_metadata_parse_generic_class (MonoType *type, MonoImage *m, MonoGenericC
        igclass->klass = g_new0 (MonoClass, 1);
 
        gtype = mono_metadata_parse_type (m, MONO_PARSE_TYPE, 0, ptr, &ptr);
+       if (gtype == NULL)
+               return FALSE;
        gclass->container_class = gklass = mono_class_from_mono_type (gtype);
 
        g_assert (gklass->generic_container);
@@ -1919,6 +2039,10 @@ do_mono_metadata_parse_generic_class (MonoType *type, MonoImage *m, MonoGenericC
        if (rptr)
                *rptr = ptr;
 
+       /* If we failed to parse, return, the error has been flagged. */
+       if (gclass->inst == NULL)
+               return FALSE;
+       
        /*
         * We may be called multiple times on different metadata to create the same
         * instantiated type.  This happens for instance if we're part of a method or
@@ -1939,7 +2063,7 @@ do_mono_metadata_parse_generic_class (MonoType *type, MonoImage *m, MonoGenericC
                g_free (gclass);
 
                type->data.generic_class = cached;
-               return;
+               return TRUE;
        } else {
                g_hash_table_insert (generic_class_cache, gclass, gclass);
 
@@ -1948,6 +2072,7 @@ do_mono_metadata_parse_generic_class (MonoType *type, MonoImage *m, MonoGenericC
                        sizeof (MonoGenericContext) +
                        gclass->inst->type_argc * sizeof (MonoType);
        }
+       return TRUE;
 }
 
 /*
@@ -1965,17 +2090,14 @@ select_container (MonoGenericContainer *gc, MonoTypeEnum type)
 
        g_assert (is_var || type == MONO_TYPE_MVAR);
 
-       if (is_var && gc->parent)
-               /*
-                * The current MonoGenericContainer is a generic method -> its `parent'
-                * points to the containing class'es container.
-                */
-               gc = gc->parent;
-
-       /*
-        * Ensure that we have the correct type of GenericContainer.
-        */
-       g_assert (is_var == !gc->is_method);
+       if (is_var) {
+               if (gc->is_method || gc->parent)
+                       /*
+                        * The current MonoGenericContainer is a generic method -> its `parent'
+                        * points to the containing class'es container.
+                        */
+                       return gc->parent;
+       }
 
        return gc;
 }
@@ -2032,6 +2154,7 @@ static gboolean
 do_mono_metadata_parse_type (MonoType *type, MonoImage *m, MonoGenericContainer *container,
                             const char *ptr, const char **rptr)
 {
+       gboolean ok = TRUE;
        type->type = mono_metadata_decode_value (ptr, &ptr);
        
        switch (type->type){
@@ -2087,7 +2210,7 @@ do_mono_metadata_parse_type (MonoType *type, MonoImage *m, MonoGenericContainer
                type->data.generic_param = mono_metadata_parse_generic_param (m, container, type->type, ptr, &ptr);
                break;
        case MONO_TYPE_GENERICINST:
-               do_mono_metadata_parse_generic_class (type, m, container, ptr, &ptr);
+               ok = do_mono_metadata_parse_generic_class (type, m, container, ptr, &ptr);
                break;
        default:
                g_error ("type 0x%02x not handled in do_mono_metadata_parse_type", type->type);
@@ -2095,7 +2218,7 @@ do_mono_metadata_parse_type (MonoType *type, MonoImage *m, MonoGenericContainer
        
        if (rptr)
                *rptr = ptr;
-       return TRUE;
+       return ok;
 }
 
 /*
@@ -2336,7 +2459,6 @@ mono_metadata_parse_mh_full (MonoImage *m, MonoGenericContainer *container, cons
                        mh->locals [i] = mono_metadata_parse_type_full (
                                m, container, MONO_PARSE_LOCAL, 0, locals_ptr, &locals_ptr);
                        if (!mh->locals [i]) {
-                               g_free (mh);
                                return NULL;
                        }
                }
@@ -2775,11 +2897,16 @@ mono_metadata_interfaces_from_typedef_full (MonoImage *meta, guint32 index, Mono
 
        pos = start;
        while (pos < tdef->rows) {
+               MonoClass *iface;
+               
                mono_metadata_decode_row (tdef, pos, cols, MONO_INTERFACEIMPL_SIZE);
                if (cols [MONO_INTERFACEIMPL_CLASS] != loc.idx)
                        break;
-               result [pos - start] = mono_class_get_full (
+               iface = mono_class_get_full (
                        meta, mono_metadata_token_from_dor (cols [MONO_INTERFACEIMPL_INTERFACE]), context);
+               if (iface == NULL)
+                       return FALSE;
+               result [pos - start] = iface;
                ++pos;
        }
        *count = pos - start;
@@ -3067,8 +3194,8 @@ mono_type_size (MonoType *t, gint *align)
                MonoClass *container_class;
 
                gclass = mono_get_inflated_generic_class (t->data.generic_class);
-               g_assert (!gclass->generic_class.inst->is_open);
-               g_assert (!gclass->klass->generic_container);
+               // g_assert (!gclass->generic_class.inst->is_open);
+               // g_assert (!gclass->klass->generic_container);
 
                container_class = gclass->generic_class.container_class;
 
@@ -3311,6 +3438,37 @@ mono_metadata_class_equal (MonoClass *c1, MonoClass *c2, gboolean signature_only
        return FALSE;
 }
 
+static gboolean
+mono_metadata_fnptr_equal (MonoMethodSignature *s1, MonoMethodSignature *s2, gboolean signature_only)
+{
+       gpointer iter1 = 0, iter2 = 0;
+
+       if (s1 == s2)
+               return TRUE;
+       if (s1->call_convention != s2->call_convention)
+               return FALSE;
+       if (s1->sentinelpos != s2->sentinelpos)
+               return FALSE;
+       if (s1->hasthis != s2->hasthis)
+               return FALSE;
+       if (s1->explicit_this != s2->explicit_this)
+               return FALSE;
+       if (! do_mono_metadata_type_equal (s1->ret, s2->ret, signature_only))
+               return FALSE;
+       if (s1->param_count != s2->param_count)
+               return FALSE;
+
+       while (TRUE) {
+               MonoType *t1 = mono_signature_get_params (s1, &iter1);
+               MonoType *t2 = mono_signature_get_params (s2, &iter2);
+
+               if (t1 == NULL || t2 == NULL)
+                       return (t1 == t2);
+               if (! do_mono_metadata_type_equal (t1, t2, signature_only))
+                       return FALSE;
+       }
+}
+
 /*
  * mono_metadata_type_equal:
  * @t1: a type
@@ -3364,6 +3522,8 @@ do_mono_metadata_type_equal (MonoType *t1, MonoType *t2, gboolean signature_only
        case MONO_TYPE_MVAR:
                return mono_metadata_generic_param_equal (
                        t1->data.generic_param, t2->data.generic_param, signature_only);
+       case MONO_TYPE_FNPTR:
+               return mono_metadata_fnptr_equal (t1->data.method, t2->data.method, signature_only);
        default:
                g_error ("implement type compare for %0x!", t1->type);
                return FALSE;
@@ -3420,6 +3580,31 @@ mono_metadata_signature_equal (MonoMethodSignature *sig1, MonoMethodSignature *s
        return TRUE;
 }
 
+/**
+ * mono_metadata_type_dup_mp:
+ * @image: image type is defined in
+ * @original: type to duplicate
+ *
+ * Returns: copy of type allocated from mempool.
+ */
+MonoType *
+mono_metadata_type_dup_mp (MonoImage *image, const MonoType *original)
+{
+       MonoType *r = NULL;
+       mono_loader_lock ();
+       r = mono_mempool_alloc0 (image->mempool, sizeof(MonoType));
+       mono_loader_unlock ();
+       *r = *original;
+       /* FIXME: we don't handle these yet because they need to duplicate memory
+        * but the current routines used are not using the mempools
+        */
+       if (original->type == MONO_TYPE_PTR || 
+               original->type == MONO_TYPE_ARRAY || 
+               original->type == MONO_TYPE_FNPTR)
+               g_assert_not_reached ();
+       return r;
+}
+
 guint
 mono_signature_hash (MonoMethodSignature *sig)
 {
@@ -4114,7 +4299,8 @@ mono_class_get_overrides_full (MonoImage *image, guint32 type_token, MonoMethod
        gint32 i, num;
        guint32 cols [MONO_METHODIMPL_SIZE];
        MonoMethod **result;
-
+       gint32 ok = TRUE;
+       
        *overrides = NULL;
        if (num_overrides)
                *num_overrides = 0;
@@ -4149,17 +4335,25 @@ mono_class_get_overrides_full (MonoImage *image, guint32 type_token, MonoMethod
        num = end - start;
        result = g_new (MonoMethod*, num * 2);
        for (i = 0; i < num; ++i) {
+               MonoMethod *method;
+
                mono_metadata_decode_row (tdef, start + i, cols, MONO_METHODIMPL_SIZE);
-               result [i * 2] = method_from_method_def_or_ref (
+               method = method_from_method_def_or_ref (
                        image, cols [MONO_METHODIMPL_DECLARATION], generic_context);
-               result [i * 2 + 1] = method_from_method_def_or_ref (
+               if (method == NULL)
+                       ok = FALSE;
+               result [i * 2] = method;
+               method = method_from_method_def_or_ref (
                        image, cols [MONO_METHODIMPL_BODY], generic_context);
+               if (method == NULL)
+                       ok = FALSE;
+               result [i * 2 + 1] = method;
        }
 
        *overrides = result;
        if (num_overrides)
                *num_overrides = num;
-       return TRUE;
+       return ok;
 }
 
 /**
@@ -4372,6 +4566,13 @@ mono_type_get_array_type (MonoType *type)
        return type->data.array;
 }
 
+/* For MONO_TYPE_PTR */
+MonoType*
+mono_type_get_ptr_type (MonoType *type)
+{
+       return type->data.type;
+}
+
 MonoClass*
 mono_type_get_modifiers (MonoType *type, gboolean *is_required, gpointer *iter)
 {