[runtime] It's spring cleaning in MonoInternalThread. Remove 8 (yes, eight) unused...
[mono.git] / mono / metadata / assembly.c
index 15d60411420a0e041ea0b2fdcfaec9ea6465b01f..a82195d0e171bc51ec60eca87d5cc732aa21228c 100644 (file)
@@ -28,6 +28,7 @@
 #include <mono/io-layer/io-layer.h>
 #include <mono/utils/mono-uri.h>
 #include <mono/metadata/mono-config.h>
+#include <mono/metadata/mono-config-dirs.h>
 #include <mono/utils/mono-digest.h>
 #include <mono/utils/mono-logger-internal.h>
 #include <mono/utils/mono-path.h>
@@ -170,9 +171,9 @@ mono_set_corlib_data (void *data, size_t size)
 #endif
 
 /* This protects loaded_assemblies and image->references */
-#define mono_assemblies_lock() EnterCriticalSection (&assemblies_mutex)
-#define mono_assemblies_unlock() LeaveCriticalSection (&assemblies_mutex)
-static CRITICAL_SECTION assemblies_mutex;
+#define mono_assemblies_lock() mono_mutex_lock (&assemblies_mutex)
+#define mono_assemblies_unlock() mono_mutex_unlock (&assemblies_mutex)
+static mono_mutex_t assemblies_mutex;
 
 /* If defined, points to the bundled assembly information */
 const MonoBundledAssembly **bundles;
@@ -183,7 +184,9 @@ static mono_mutex_t assembly_binding_mutex;
 static GSList *loaded_assembly_bindings = NULL;
 
 static MonoAssembly*
-mono_assembly_invoke_search_hook_internal (MonoAssemblyName *aname, gboolean refonly, gboolean postload);
+mono_assembly_invoke_search_hook_internal (MonoAssemblyName *aname, MonoAssembly *requesting, gboolean refonly, gboolean postload);
+static MonoAssembly*
+mono_assembly_load_full_internal (MonoAssemblyName *aname, MonoAssembly *requesting, const char *basedir, MonoImageOpenStatus *status, gboolean refonly);
 static MonoBoolean
 mono_assembly_is_in_gac (const gchar *filanem);
 
@@ -560,14 +563,10 @@ mono_assembly_getrootdir (void)
 void
 mono_set_dirs (const char *assembly_dir, const char *config_dir)
 {
-#if defined (MONO_ASSEMBLIES)
        if (assembly_dir == NULL)
-               assembly_dir = MONO_ASSEMBLIES;
-#endif
-#if defined (MONO_CFG_DIR)
+               assembly_dir = mono_config_get_assemblies_dir ();
        if (config_dir == NULL)
-               config_dir = MONO_CFG_DIR;
-#endif
+               config_dir = mono_config_get_cfg_dir ();
        mono_assembly_setrootdir (assembly_dir);
        mono_set_config_dir (config_dir);
 }
@@ -582,7 +581,7 @@ compute_base (char *path)
                return NULL;
 
        /* Not a well known Mono executable, we are embedded, cant guess the base  */
-       if (strcmp (p, "/mono") && strcmp (p, "/mono-sgen") && strcmp (p, "/pedump") && strcmp (p, "/monodis") && strcmp (p, "/mint") && strcmp (p, "/monodiet"))
+       if (strcmp (p, "/mono") && strcmp (p, "/mono-boehm") && strcmp (p, "/mono-sgen") && strcmp (p, "/pedump") && strcmp (p, "/monodis"))
                return NULL;
            
        *p = 0;
@@ -599,7 +598,7 @@ compute_base (char *path)
 static void
 fallback (void)
 {
-       mono_set_dirs (MONO_ASSEMBLIES, MONO_CFG_DIR);
+       mono_set_dirs (mono_config_get_assemblies_dir (), mono_config_get_cfg_dir ());
 }
 
 static G_GNUC_UNUSED void
@@ -608,11 +607,14 @@ set_dirs (char *exe)
        char *base;
        char *config, *lib, *mono;
        struct stat buf;
+       const char *bindir;
        
        /*
         * Only /usr prefix is treated specially
         */
-       if (strncmp (exe, MONO_BINDIR, strlen (MONO_BINDIR)) == 0 || (base = compute_base (exe)) == NULL){
+       bindir = mono_config_get_bin_dir ();
+       g_assert (bindir);
+       if (strncmp (exe, bindir, strlen (bindir)) == 0 || (base = compute_base (exe)) == NULL){
                fallback ();
                return;
        }
@@ -739,7 +741,7 @@ mono_assemblies_init (void)
        check_path_env ();
        check_extra_gac_path_env ();
 
-       InitializeCriticalSection (&assemblies_mutex);
+       mono_mutex_init_recursive (&assemblies_mutex);
        mono_mutex_init (&assembly_binding_mutex);
 }
 
@@ -1090,12 +1092,12 @@ mono_assembly_load_reference (MonoImage *image, int index)
        if (image->assembly && image->assembly->ref_only) {
                /* We use the loaded corlib */
                if (!strcmp (aname.name, "mscorlib"))
-                       reference = mono_assembly_load_full (&aname, image->assembly->basedir, &status, FALSE);
+                       reference = mono_assembly_load_full_internal (&aname, image->assembly, image->assembly->basedir, &status, FALSE);
                else {
                        reference = mono_assembly_loaded_full (&aname, TRUE);
                        if (!reference)
                                /* Try a postload search hook */
-                               reference = mono_assembly_invoke_search_hook_internal (&aname, TRUE, TRUE);
+                               reference = mono_assembly_invoke_search_hook_internal (&aname, image->assembly, TRUE, TRUE);
                }
 
                /*
@@ -1111,9 +1113,9 @@ mono_assembly_load_reference (MonoImage *image, int index)
                 * The second load attempt has the basedir set to keep compatibility with the old mono behavior, for
                 * example bug-349190.2.cs and who knows how much more code in the wild.
                 */
-               reference = mono_assembly_load (&aname, NULL, &status);
+               reference = mono_assembly_load_full_internal (&aname, image->assembly, NULL, &status, FALSE);
                if (!reference && image->assembly)
-                       reference = mono_assembly_load (&aname, image->assembly->basedir, &status);
+                       reference = mono_assembly_load_full_internal (&aname, image->assembly, image->assembly->basedir, &status, FALSE);
        }
 
        if (reference == NULL){
@@ -1232,13 +1234,36 @@ struct AssemblySearchHook {
 AssemblySearchHook *assembly_search_hook = NULL;
 
 static MonoAssembly*
-mono_assembly_invoke_search_hook_internal (MonoAssemblyName *aname, gboolean refonly, gboolean postload)
+mono_assembly_invoke_search_hook_internal (MonoAssemblyName *aname, MonoAssembly *requesting, gboolean refonly, gboolean postload)
 {
        AssemblySearchHook *hook;
 
        for (hook = assembly_search_hook; hook; hook = hook->next) {
                if ((hook->refonly == refonly) && (hook->postload == postload)) {
-                       MonoAssembly *ass = hook->func (aname, hook->user_data);
+                       MonoAssembly *ass;
+                       /**
+                         * A little explanation is in order here.
+                         *
+                         * The default postload search hook needs to know the requesting assembly to report it to managed code.
+                         * The embedding API exposes a search hook that doesn't take such argument.
+                         *
+                         * The original fix would call the default search hook before all the registered ones and pass
+                         * the requesting assembly to it. It works but broke a very suddle embedding API aspect that some users
+                         * rely on. Which is the ordering between user hooks and the default runtime hook.
+                         *
+                         * Registering the hook after mono_jit_init would let your hook run before the default one and
+                         * when using it to handle non standard app layouts this could save your app from a massive amount
+                         * of syscalls that the default hook does when probing all sorts of places. Slow targets with horrible IO
+                         * are all using this trick and if we broke this assumption they would be very disapointed at us.
+                         *
+                         * So what's the fix? We register the default hook using regular means and special case it when iterating
+                         * over the registered hooks. This preserves ordering and enables managed resolve hooks to get the requesting
+                         * assembly.
+                         */
+                       if (hook->func == (void*)mono_domain_assembly_postload_search)
+                               ass = mono_domain_assembly_postload_search (aname, requesting, refonly);
+                       else
+                               ass = hook->func (aname, hook->user_data);
                        if (ass)
                                return ass;
                }
@@ -1250,7 +1275,7 @@ mono_assembly_invoke_search_hook_internal (MonoAssemblyName *aname, gboolean ref
 MonoAssembly*
 mono_assembly_invoke_search_hook (MonoAssemblyName *aname)
 {
-       return mono_assembly_invoke_search_hook_internal (aname, FALSE, FALSE);
+       return mono_assembly_invoke_search_hook_internal (aname, NULL, FALSE, FALSE);
 }
 
 static void
@@ -1648,7 +1673,6 @@ mono_assembly_load_friends (MonoAssembly* ass)
                MonoCustomAttrEntry *attr = &attrs->attrs [i];
                MonoAssemblyName *aname;
                const gchar *data;
-               guint slen;
                /* Do some sanity checking */
                if (!attr->ctor || attr->ctor->klass != mono_defaults.internals_visible_class)
                        continue;
@@ -1658,7 +1682,7 @@ mono_assembly_load_friends (MonoAssembly* ass)
                /* 0xFF means null string, see custom attr format */
                if (data [0] != 1 || data [1] != 0 || (data [2] & 0xFF) == 0xFF)
                        continue;
-               slen = mono_metadata_decode_value (data + 2, &data);
+               mono_metadata_decode_value (data + 2, &data);
                aname = g_new0 (MonoAssemblyName, 1);
                /*g_print ("friend ass: %s\n", data);*/
                if (mono_assembly_name_parse_full (data, aname, TRUE, NULL, NULL)) {
@@ -1764,7 +1788,7 @@ mono_assembly_load_from_full (MonoImage *image, const char*fname,
         * assemblies lock.
         */
        if (ass->aname.name) {
-               ass2 = mono_assembly_invoke_search_hook_internal (&ass->aname, refonly, FALSE);
+               ass2 = mono_assembly_invoke_search_hook_internal (&ass->aname, NULL, refonly, FALSE);
                if (ass2) {
                        g_free (ass);
                        g_free (base_dir);
@@ -2367,7 +2391,7 @@ mono_assembly_load_with_partial_name (const char *name, MonoImageOpenStatus *sta
                res->in_gac = TRUE;
        else {
                MonoDomain *domain = mono_domain_get ();
-               MonoReflectionAssembly *refasm = mono_try_assembly_resolve (domain, mono_string_new (domain, name), FALSE);
+               MonoReflectionAssembly *refasm = mono_try_assembly_resolve (domain, mono_string_new (domain, name), NULL, FALSE);
                if (refasm)
                        res = refasm->assembly;
        }
@@ -2947,6 +2971,17 @@ mono_assembly_load_full_nosearch (MonoAssemblyName *aname,
        return result;
 }
 
+MonoAssembly*
+mono_assembly_load_full_internal (MonoAssemblyName *aname, MonoAssembly *requesting, const char *basedir, MonoImageOpenStatus *status, gboolean refonly)
+{
+       MonoAssembly *result = mono_assembly_load_full_nosearch (aname, basedir, status, refonly);
+
+       if (!result)
+               /* Try a postload search hook */
+               result = mono_assembly_invoke_search_hook_internal (aname, requesting, refonly, TRUE);
+       return result;
+}
+
 /**
  * mono_assembly_load_full:
  * @aname: A MonoAssemblyName with the assembly name to load.
@@ -2966,12 +3001,7 @@ mono_assembly_load_full_nosearch (MonoAssemblyName *aname,
 MonoAssembly*
 mono_assembly_load_full (MonoAssemblyName *aname, const char *basedir, MonoImageOpenStatus *status, gboolean refonly)
 {
-       MonoAssembly *result = mono_assembly_load_full_nosearch (aname, basedir, status, refonly);
-       
-       if (!result)
-               /* Try a postload search hook */
-               result = mono_assembly_invoke_search_hook_internal (aname, refonly, TRUE);
-       return result;
+       return mono_assembly_load_full_internal (aname, NULL, basedir, status, refonly);
 }
 
 /**
@@ -2989,9 +3019,9 @@ mono_assembly_load_full (MonoAssemblyName *aname, const char *basedir, MonoImage
 MonoAssembly*
 mono_assembly_load (MonoAssemblyName *aname, const char *basedir, MonoImageOpenStatus *status)
 {
-       return mono_assembly_load_full (aname, basedir, status, FALSE);
+       return mono_assembly_load_full_internal (aname, NULL, basedir, status, FALSE);
 }
-       
+
 MonoAssembly*
 mono_assembly_loaded_full (MonoAssemblyName *aname, gboolean refonly)
 {
@@ -3000,7 +3030,7 @@ mono_assembly_loaded_full (MonoAssemblyName *aname, gboolean refonly)
 
        aname = mono_assembly_remap_version (aname, &maped_aname);
 
-       res = mono_assembly_invoke_search_hook_internal (aname, refonly, FALSE);
+       res = mono_assembly_invoke_search_hook_internal (aname, NULL, refonly, FALSE);
 
        return res;
 }
@@ -3024,7 +3054,7 @@ mono_assembly_release_gc_roots (MonoAssembly *assembly)
        if (assembly == NULL || assembly == REFERENCE_MISSING)
                return;
 
-       if (assembly->dynamic) {
+       if (assembly_is_dynamic (assembly)) {
                int i;
                MonoDynamicImage *dynimg = (MonoDynamicImage *)assembly->image;
                for (i = 0; i < dynimg->image.module_count; ++i)
@@ -3087,7 +3117,7 @@ mono_assembly_close_finish (MonoAssembly *assembly)
        if (assembly->image)
                mono_image_close_finish (assembly->image);
 
-       if (assembly->dynamic) {
+       if (assembly_is_dynamic (assembly)) {
                g_free ((char*)assembly->aname.culture);
        } else {
                g_free (assembly);
@@ -3142,7 +3172,7 @@ mono_assemblies_cleanup (void)
 {
        GSList *l;
 
-       DeleteCriticalSection (&assemblies_mutex);
+       mono_mutex_destroy (&assemblies_mutex);
        mono_mutex_destroy (&assembly_binding_mutex);
 
        for (l = loaded_assembly_bindings; l; l = l->next) {