2009-03-02 Zoltan Varga <vargaz@gmail.com>
[mono.git] / mono / metadata / image.c
index f0c4dd290e1d5f87af96c26a6c261a2398655ff0..39766cb0237a0e16c3c5ccca349f1dd6f33911a4 100644 (file)
@@ -6,7 +6,8 @@
  *   Miguel de Icaza (miguel@ximian.com)
  *   Paolo Molaro (lupus@ximian.com)
  *
- * (C) 2001-2003 Ximian, Inc.  http://www.ximian.com
+ * Copyright 2001-2003 Ximian, Inc (http://www.ximian.com)
+ * Copyright 2004-2009 Novell, Inc (http://www.novell.com)
  *
  */
 #include <config.h>
@@ -17,7 +18,6 @@
 #include <string.h>
 #include "image.h"
 #include "cil-coff.h"
-#include "rawbuffer.h"
 #include "mono-endian.h"
 #include "tabledefs.h"
 #include "tokentype.h"
@@ -29,6 +29,7 @@
 #include <mono/io-layer/io-layer.h>
 #include <mono/utils/mono-logger.h>
 #include <mono/utils/mono-path.h>
+#include <mono/utils/mono-mmap.h>
 #include <mono/utils/mono-io-portability.h>
 #include <mono/metadata/class-internals.h>
 #include <mono/metadata/assembly.h>
@@ -460,7 +461,7 @@ mono_image_check_for_module_cctor (MonoImage *image)
        MonoTableInfo *t, *mt;
        t = &image->tables [MONO_TABLE_TYPEDEF];
        mt = &image->tables [MONO_TABLE_METHOD];
-       if (mono_get_runtime_info ()->framework_version [0] == '1') {
+       if (mono_framework_version () == 1) {
                image->checked_module_cctor = TRUE;
                return;
        }
@@ -601,47 +602,20 @@ void
 mono_image_init (MonoImage *image)
 {
        image->mempool = mono_mempool_new_size (512);
-       image->method_cache = g_hash_table_new (NULL, NULL);
        mono_internal_hash_table_init (&image->class_cache,
                                       g_direct_hash,
                                       class_key_extract,
                                       class_next_value);
        image->field_cache = g_hash_table_new (NULL, NULL);
 
-       image->delegate_begin_invoke_cache = 
-               g_hash_table_new ((GHashFunc)mono_signature_hash, 
-                                 (GCompareFunc)mono_metadata_signature_equal);
-       image->delegate_end_invoke_cache = 
-               g_hash_table_new ((GHashFunc)mono_signature_hash, 
-                                 (GCompareFunc)mono_metadata_signature_equal);
-       image->delegate_invoke_cache = 
-               g_hash_table_new ((GHashFunc)mono_signature_hash, 
-                                 (GCompareFunc)mono_metadata_signature_equal);
-       image->runtime_invoke_cache  = 
-               g_hash_table_new ((GHashFunc)mono_signature_hash, 
-                                 (GCompareFunc)mono_metadata_signature_equal);
-       
-       image->runtime_invoke_direct_cache = g_hash_table_new (mono_aligned_addr_hash, NULL);
-       image->managed_wrapper_cache = g_hash_table_new (mono_aligned_addr_hash, NULL);
-       image->native_wrapper_cache = g_hash_table_new (mono_aligned_addr_hash, NULL);
-       image->remoting_invoke_cache = g_hash_table_new (mono_aligned_addr_hash, NULL);
-       image->cominterop_invoke_cache = g_hash_table_new (mono_aligned_addr_hash, NULL);
-       image->cominterop_wrapper_cache = g_hash_table_new (mono_aligned_addr_hash, NULL);
-       image->synchronized_cache = g_hash_table_new (mono_aligned_addr_hash, NULL);
-       image->unbox_wrapper_cache = g_hash_table_new (mono_aligned_addr_hash, NULL);
-
-       image->ldfld_wrapper_cache = g_hash_table_new (mono_aligned_addr_hash, NULL);
-       image->ldflda_wrapper_cache = g_hash_table_new (mono_aligned_addr_hash, NULL);
-       image->stfld_wrapper_cache = g_hash_table_new (mono_aligned_addr_hash, NULL);
-       image->isinst_cache = g_hash_table_new (mono_aligned_addr_hash, NULL);
-       image->castclass_cache = g_hash_table_new (mono_aligned_addr_hash, NULL);
-       image->proxy_isinst_cache = g_hash_table_new (mono_aligned_addr_hash, NULL);
-       image->thunk_invoke_cache = g_hash_table_new (mono_aligned_addr_hash, NULL);
-
        image->typespec_cache = g_hash_table_new (NULL, NULL);
        image->memberref_signatures = g_hash_table_new (NULL, NULL);
        image->helper_signatures = g_hash_table_new (g_str_hash, g_str_equal);
        image->method_signatures = g_hash_table_new (NULL, NULL);
+
+       image->property_hash = mono_property_hash_new ();
+       InitializeCriticalSection (&image->lock);
+       InitializeCriticalSection (&image->szarray_cache_lock);
 }
 
 #if G_BYTE_ORDER != G_LITTLE_ENDIAN
@@ -903,14 +877,13 @@ do_mono_image_open (const char *fname, MonoImageOpenStatus *status,
 {
        MonoCLIImageInfo *iinfo;
        MonoImage *image;
-       FILE *filed;
-       struct stat stat_buf;
+       MonoFileMap *filed;
 
-       if ((filed = fopen (fname, "rb")) == NULL){
+       if ((filed = mono_file_map_open (fname)) == NULL){
                if (IS_PORTABILITY_SET) {
                        gchar *ffname = mono_portability_find_file (fname, TRUE);
                        if (ffname) {
-                               filed = fopen (ffname, "rb");
+                               filed = mono_file_map_open (ffname);
                                g_free (ffname);
                        }
                }
@@ -922,24 +895,24 @@ do_mono_image_open (const char *fname, MonoImageOpenStatus *status,
                }
        }
 
-       if (fstat (fileno (filed), &stat_buf)) {
-               fclose (filed);
+       image = g_new0 (MonoImage, 1);
+       image->raw_buffer_used = TRUE;
+       image->raw_data_len = mono_file_map_size (filed);
+       image->raw_data = mono_file_map (image->raw_data_len, MONO_MMAP_READ|MONO_MMAP_PRIVATE, mono_file_map_fd (filed), 0, &image->raw_data_handle);
+       if (!image->raw_data) {
+               mono_file_map_close (filed);
+               g_free (image);
                if (status)
-                       *status = MONO_IMAGE_ERROR_ERRNO;
+                       *status = MONO_IMAGE_IMAGE_INVALID;
                return NULL;
        }
-       image = g_new0 (MonoImage, 1);
-       image->raw_buffer_used = TRUE;
-       image->raw_data_len = stat_buf.st_size;
-       image->raw_data = mono_raw_buffer_load (fileno (filed), FALSE, 0, stat_buf.st_size);
        iinfo = g_new0 (MonoCLIImageInfo, 1);
        image->image_info = iinfo;
        image->name = mono_path_resolve_symlinks (fname);
        image->ref_only = refonly;
        image->ref_count = 1;
 
-       fclose (filed);
-
+       mono_file_map_close (filed);
        return do_mono_image_load (image, status, care_about_cli);
 }
 
@@ -1079,7 +1052,7 @@ mono_image_open_from_data (char *data, guint32 data_len, gboolean need_copy, Mon
 #ifdef PLATFORM_WIN32
 /* fname is not duplicated. */
 MonoImage*
-mono_image_open_from_module_handle (HMODULE module_handle, char* fname, int ref_count, MonoImageOpenStatus* status)
+mono_image_open_from_module_handle (HMODULE module_handle, char* fname, gboolean has_entry_point, MonoImageOpenStatus* status)
 {
        MonoImage* image;
        MonoCLIImageInfo* iinfo;
@@ -1090,9 +1063,13 @@ mono_image_open_from_module_handle (HMODULE module_handle, char* fname, int ref_
        iinfo = g_new0 (MonoCLIImageInfo, 1);
        image->image_info = iinfo;
        image->name = fname;
-       image->ref_count = ref_count;
+       image->ref_count = has_entry_point ? 0 : 1;
+       image->has_entry_point = has_entry_point;
 
        image = do_mono_image_load (image, status, TRUE);
+       if (image == NULL)
+               return NULL;
+
        return register_image (image);
 }
 #endif
@@ -1121,15 +1098,12 @@ mono_image_open_full (const char *fname, MonoImageOpenStatus *status, gboolean r
                image = g_hash_table_lookup (loaded_images_hash, absfname);
                if (image) {
                        g_assert (image->is_module_handle);
-                       if (image->ref_count == 0) {
-                               MonoCLIImageInfo *iinfo = image->image_info;
-
-                               if (iinfo->cli_header.coff.coff_attributes & COFF_ATTRIBUTE_LIBRARY_IMAGE) {
-                                       /* Increment reference count on images loaded outside of the runtime. */
-                                       fname_utf16 = g_utf8_to_utf16 (absfname, -1, NULL, NULL, NULL);
-                                       module_handle = LoadLibrary (fname_utf16);
-                                       g_assert (module_handle != NULL);
-                               }
+                       if (image->has_entry_point && image->ref_count == 0) {
+                               /* Increment reference count on images loaded outside of the runtime. */
+                               fname_utf16 = g_utf8_to_utf16 (absfname, -1, NULL, NULL, NULL);
+                               /* The image is already loaded because _CorDllMain removes images from the hash. */
+                               module_handle = LoadLibrary (fname_utf16);
+                               g_assert (module_handle == (HMODULE) image->raw_data);
                        }
                        mono_image_addref (image);
                        mono_images_unlock ();
@@ -1140,7 +1114,7 @@ mono_image_open_full (const char *fname, MonoImageOpenStatus *status, gboolean r
                }
 
                fname_utf16 = g_utf8_to_utf16 (absfname, -1, NULL, NULL, NULL);
-               module_handle = LoadLibrary (fname_utf16);
+               module_handle = MonoLoadImage (fname_utf16);
                if (status && module_handle == NULL)
                        last_error = GetLastError ();
 
@@ -1166,11 +1140,12 @@ mono_image_open_full (const char *fname, MonoImageOpenStatus *status, gboolean r
 
                if (image) {
                        g_assert (image->is_module_handle);
+                       g_assert (image->has_entry_point);
                        g_free (absfname);
                        return image;
                }
 
-               return mono_image_open_from_module_handle (module_handle, absfname, 1, status);
+               return mono_image_open_from_module_handle (module_handle, absfname, FALSE, status);
        }
 #endif
 
@@ -1269,12 +1244,12 @@ mono_image_fixup_vtable (MonoImage *image)
                slot_count = vtfixup->count;
                if (slot_type & VTFIXUP_TYPE_32BIT)
                        while (slot_count--) {
-                               *((guint32*) slot) = mono_marshal_get_vtfixup_ftnptr (image, *((guint32*) slot), slot_type);
+                               *((guint32*) slot) = (guint32) mono_marshal_get_vtfixup_ftnptr (image, *((guint32*) slot), slot_type);
                                ((guint32*) slot)++;
                        }
                else if (slot_type & VTFIXUP_TYPE_64BIT)
                        while (slot_count--) {
-                               *((guint64*) slot) = mono_marshal_get_vtfixup_ftnptr (image, *((guint64*) slot), slot_type);
+                               *((guint64*) slot) = (guint64) mono_marshal_get_vtfixup_ftnptr (image, *((guint64*) slot), slot_type);
                                ((guint64*) slot)++;
                        }
                else
@@ -1337,6 +1312,13 @@ mono_dynamic_stream_reset (MonoDynamicStream* stream)
        }
 }
 
+static inline void
+free_hash (GHashTable *hash)
+{
+       if (hash)
+               g_hash_table_destroy (hash);
+}
+
 /**
  * mono_image_close:
  * @image: The image file we wish to close
@@ -1357,19 +1339,15 @@ mono_image_close (MonoImage *image)
                return;
 
 #ifdef PLATFORM_WIN32
-       if (image->is_module_handle) {
-               MonoCLIImageInfo *iinfo = image->image_info;
-
-               if (iinfo->cli_header.coff.coff_attributes & COFF_ATTRIBUTE_LIBRARY_IMAGE) {
-                       mono_images_lock ();
-                       if (image->ref_count == 0) {
-                               /* Image will be closed by _CorDllMain. */
-                               FreeLibrary ((HMODULE) image->raw_data);
-                               mono_images_unlock ();
-                               return;
-                       }
+       if (image->is_module_handle && image->has_entry_point) {
+               mono_images_lock ();
+               if (image->ref_count == 0) {
+                       /* Image will be closed by _CorDllMain. */
+                       FreeLibrary ((HMODULE) image->raw_data);
                        mono_images_unlock ();
+                       return;
                }
+               mono_images_unlock ();
        }
 #endif
 
@@ -1384,10 +1362,11 @@ mono_image_close (MonoImage *image)
         * assemblies, so we can't release these references in mono_assembly_close () since the
         * MonoImage might outlive its associated MonoAssembly.
         */
-       if (image->references) {
+       if (image->references && !image->dynamic) {
+               MonoTableInfo *t = &image->tables [MONO_TABLE_ASSEMBLYREF];
                int i;
 
-               for (i = 0; image->references [i]; i++) {
+               for (i = 0; i < t->rows; i++) {
                        if (image->references [i])
                                mono_assembly_close (image->references [i]);
                }
@@ -1407,19 +1386,15 @@ mono_image_close (MonoImage *image)
                g_hash_table_remove (loaded_images, (char *) image->assembly_name);     
 
 #ifdef PLATFORM_WIN32
-       if (image->is_module_handle) {
-               MonoCLIImageInfo *iinfo = image->image_info;
-
-               if (!(iinfo->cli_header.coff.coff_attributes & COFF_ATTRIBUTE_LIBRARY_IMAGE))
-                       FreeLibrary ((HMODULE) image->raw_data);
-       }
+       if (image->is_module_handle && !image->has_entry_point)
+               FreeLibrary ((HMODULE) image->raw_data);
 #endif
 
        mono_images_unlock ();
 
        if (image->raw_buffer_used) {
                if (image->raw_data != NULL)
-                       mono_raw_buffer_free (image->raw_data);
+                       mono_file_unmap (image->raw_data, image->raw_data_handle);
        }
        
        if (image->raw_data_allocated) {
@@ -1448,44 +1423,49 @@ mono_image_close (MonoImage *image)
                g_free (image->files);
        }
 
-       g_hash_table_destroy (image->method_cache);
+       if (image->method_cache)
+               mono_value_hash_table_destroy (image->method_cache);
+       if (image->methodref_cache)
+               g_hash_table_destroy (image->methodref_cache);
        mono_internal_hash_table_destroy (&image->class_cache);
        g_hash_table_destroy (image->field_cache);
        if (image->array_cache) {
                g_hash_table_foreach (image->array_cache, free_array_cache_entry, NULL);
                g_hash_table_destroy (image->array_cache);
        }
+       if (image->szarray_cache)
+               g_hash_table_destroy (image->szarray_cache);
        if (image->ptr_cache)
                g_hash_table_destroy (image->ptr_cache);
        if (image->name_cache) {
                g_hash_table_foreach (image->name_cache, free_hash_table, NULL);
                g_hash_table_destroy (image->name_cache);
        }
-       g_hash_table_destroy (image->native_wrapper_cache);
-       g_hash_table_destroy (image->managed_wrapper_cache);
-       g_hash_table_destroy (image->delegate_begin_invoke_cache);
-       g_hash_table_destroy (image->delegate_end_invoke_cache);
-       g_hash_table_destroy (image->delegate_invoke_cache);
-       if (image->delegate_abstract_invoke_cache)
-               g_hash_table_destroy (image->delegate_abstract_invoke_cache);
-       g_hash_table_foreach (image->remoting_invoke_cache, free_remoting_wrappers, NULL);
-       g_hash_table_destroy (image->remoting_invoke_cache);
-       g_hash_table_destroy (image->runtime_invoke_cache);
-       g_hash_table_destroy (image->runtime_invoke_direct_cache);
-       g_hash_table_destroy (image->synchronized_cache);
-       g_hash_table_destroy (image->unbox_wrapper_cache);
-       g_hash_table_destroy (image->cominterop_invoke_cache);
-       g_hash_table_destroy (image->cominterop_wrapper_cache);
-       g_hash_table_destroy (image->typespec_cache);
-       g_hash_table_destroy (image->ldfld_wrapper_cache);
-       g_hash_table_destroy (image->ldflda_wrapper_cache);
-       g_hash_table_destroy (image->stfld_wrapper_cache);
-       g_hash_table_destroy (image->isinst_cache);
-       g_hash_table_destroy (image->castclass_cache);
-       g_hash_table_destroy (image->proxy_isinst_cache);
-       g_hash_table_destroy (image->thunk_invoke_cache);
-       if (image->static_rgctx_invoke_cache)
-               g_hash_table_destroy (image->static_rgctx_invoke_cache);
+
+       free_hash (image->native_wrapper_cache);
+       free_hash (image->managed_wrapper_cache);
+       free_hash (image->delegate_begin_invoke_cache);
+       free_hash (image->delegate_end_invoke_cache);
+       free_hash (image->delegate_invoke_cache);
+       free_hash (image->delegate_abstract_invoke_cache);
+       if (image->remoting_invoke_cache)
+               g_hash_table_foreach (image->remoting_invoke_cache, free_remoting_wrappers, NULL);
+       free_hash (image->remoting_invoke_cache);
+       free_hash (image->runtime_invoke_cache);
+       free_hash (image->runtime_invoke_direct_cache);
+       free_hash (image->synchronized_cache);
+       free_hash (image->unbox_wrapper_cache);
+       free_hash (image->cominterop_invoke_cache);
+       free_hash (image->cominterop_wrapper_cache);
+       free_hash (image->typespec_cache);
+       free_hash (image->ldfld_wrapper_cache);
+       free_hash (image->ldflda_wrapper_cache);
+       free_hash (image->stfld_wrapper_cache);
+       free_hash (image->isinst_cache);
+       free_hash (image->castclass_cache);
+       free_hash (image->proxy_isinst_cache);
+       free_hash (image->thunk_invoke_cache);
+       free_hash (image->static_rgctx_invoke_cache);
 
        /* The ownership of signatures is not well defined */
        //g_hash_table_foreach (image->memberref_signatures, free_mr_signatures, NULL);
@@ -1500,6 +1480,9 @@ mono_image_close (MonoImage *image)
        if (image->rgctx_template_hash)
                g_hash_table_destroy (image->rgctx_template_hash);
 
+       if (image->property_hash)
+               mono_property_hash_destroy (image->property_hash);
+
        if (image->interface_bitset) {
                mono_unload_interface_ids (image->interface_bitset);
                mono_bitset_free (image->interface_bitset);
@@ -1524,6 +1507,11 @@ mono_image_close (MonoImage *image)
                g_free (image->modules_loaded);
        if (image->references)
                g_free (image->references);
+       mono_perfcounters->loader_bytes -= mono_mempool_get_allocated (image->mempool);
+
+       DeleteCriticalSection (&image->szarray_cache_lock);
+       DeleteCriticalSection (&image->lock);
+
        /*g_print ("destroy image %p (dynamic: %d)\n", image, image->dynamic);*/
        if (!image->dynamic) {
                if (debug_assembly_unload)
@@ -1583,23 +1571,18 @@ mono_image_walk_resource_tree (MonoCLIImageInfo *info, guint32 res_id,
         * Level 2 holds a directory entry for each language pointing to
         * the actual data.
         */
-       name_offset = GUINT32_FROM_LE (entry->name_offset) & 0x7fffffff;
-       dir_offset = GUINT32_FROM_LE (entry->dir_offset) & 0x7fffffff;
+       is_string = MONO_PE_RES_DIR_ENTRY_NAME_IS_STRING (*entry);
+       name_offset = MONO_PE_RES_DIR_ENTRY_NAME_OFFSET (*entry);
 
-#if G_BYTE_ORDER != G_LITTLE_ENDIAN
-       is_string = (GUINT32_FROM_LE (entry->name_offset) & 0x80000000) != 0;
-       is_dir = (GUINT32_FROM_LE (entry->dir_offset) & 0x80000000) != 0;
-#else
-       is_string = entry->name_is_string;
-       is_dir = entry->is_dir;
-#endif
+       is_dir = MONO_PE_RES_DIR_ENTRY_IS_DIR (*entry);
+       dir_offset = MONO_PE_RES_DIR_ENTRY_DIR_OFFSET (*entry);
 
        if(level==0) {
-               if((is_string==FALSE && name_offset!=res_id) ||
-                  (is_string==TRUE)) {
-                       return(NULL);
-               }
+               if (is_string)
+                       return NULL;
        } else if (level==1) {
+               if (res_id != name_offset)
+                       return NULL;
 #if 0
                if(name!=NULL &&
                   is_string==TRUE && name!=lookup (name_offset)) {
@@ -1607,12 +1590,8 @@ mono_image_walk_resource_tree (MonoCLIImageInfo *info, guint32 res_id,
                }
 #endif
        } else if (level==2) {
-               if ((is_string == FALSE &&
-                   name_offset != lang_id &&
-                   lang_id != 0) ||
-                  (is_string == TRUE)) {
-                       return(NULL);
-               }
+               if (is_string == TRUE || (is_string == FALSE && lang_id != 0 && name_offset != lang_id))
+                       return NULL;
        } else {
                g_assert_not_reached ();
        }
@@ -1679,6 +1658,8 @@ mono_image_lookup_resource (MonoImage *image, guint32 res_id, guint32 lang_id, g
                return(NULL);
        }
 
+       mono_image_ensure_section_idx (image, MONO_SECTION_RSRC);
+
        info=image->image_info;
        if(info==NULL) {
                return(NULL);
@@ -2008,3 +1989,134 @@ mono_image_has_authenticode_entry (MonoImage *image)
        // the Authenticode "pre" (non ASN.1) header is 8 bytes long
        return ((de->rva != 0) && (de->size > 8));
 }
+
+gpointer
+mono_image_alloc (MonoImage *image, guint size)
+{
+       gpointer res;
+
+       mono_perfcounters->loader_bytes += size;
+       mono_image_lock (image);
+       res = mono_mempool_alloc (image->mempool, size);
+       mono_image_unlock (image);
+
+       return res;
+}
+
+gpointer
+mono_image_alloc0 (MonoImage *image, guint size)
+{
+       gpointer res;
+
+       mono_perfcounters->loader_bytes += size;
+       mono_image_lock (image);
+       res = mono_mempool_alloc0 (image->mempool, size);
+       mono_image_unlock (image);
+
+       return res;
+}
+
+char*
+mono_image_strdup (MonoImage *image, const char *s)
+{
+       char *res;
+
+       mono_perfcounters->loader_bytes += strlen (s);
+       mono_image_lock (image);
+       res = mono_mempool_strdup (image->mempool, s);
+       mono_image_unlock (image);
+
+       return res;
+}
+
+GList*
+g_list_prepend_image (MonoImage *image, GList *list, gpointer data)
+{
+       GList *new_list;
+       
+       new_list = mono_image_alloc (image, sizeof (GList));
+       new_list->data = data;
+       new_list->prev = list ? list->prev : NULL;
+    new_list->next = list;
+
+    if (new_list->prev)
+            new_list->prev->next = new_list;
+    if (list)
+            list->prev = new_list;
+
+       return new_list;
+}
+
+GSList*
+g_slist_append_image (MonoImage *image, GSList *list, gpointer data)
+{
+       GSList *new_list;
+
+       new_list = mono_image_alloc (image, sizeof (GSList));
+       new_list->data = data;
+       new_list->next = NULL;
+
+       return g_slist_concat (list, new_list);
+}
+
+void
+mono_image_lock (MonoImage *image)
+{
+       mono_locks_acquire (&image->lock, ImageDataLock);
+}
+
+void
+mono_image_unlock (MonoImage *image)
+{
+       mono_locks_release (&image->lock, ImageDataLock);
+}
+
+
+/**
+ * mono_image_property_lookup:
+ *
+ * Lookup a property on @image. Used to store very rare fields of MonoClass and MonoMethod.
+ *
+ * LOCKING: Takes the image lock
+ */
+gpointer 
+mono_image_property_lookup (MonoImage *image, gpointer subject, guint32 property)
+{
+       gpointer res;
+
+       mono_image_lock (image);
+       res = mono_property_hash_lookup (image->property_hash, subject, property);
+       mono_image_unlock (image);
+
+       return res;
+}
+
+/**
+ * mono_image_property_insert:
+ *
+ * Insert a new property @property with value @value on @subject in @image. Used to store very rare fields of MonoClass and MonoMethod.
+ *
+ * LOCKING: Takes the image lock
+ */
+void
+mono_image_property_insert (MonoImage *image, gpointer subject, guint32 property, gpointer value)
+{
+       mono_image_lock (image);
+       mono_property_hash_insert (image->property_hash, subject, property, value);
+       mono_image_unlock (image);
+}
+
+/**
+ * mono_image_property_remove:
+ *
+ * Remove all properties associated with @subject in @image. Used to store very rare fields of MonoClass and MonoMethod.
+ *
+ * LOCKING: Takes the image lock
+ */
+void
+mono_image_property_remove (MonoImage *image, gpointer subject)
+{
+       mono_image_lock (image);
+       mono_property_hash_remove_object (image->property_hash, subject);
+       mono_image_unlock (image);
+}