2005-02-20 Zoltan Varga <vargaz@freemail.hu>
[mono.git] / mono / mini / mini-exceptions.c
index 976c563f14a01902ff8e2fe2ead83bd4b8f3f751..6f4dc154979bd4173dc980ebf1cdafd1a1644110 100644 (file)
@@ -19,6 +19,7 @@
 #include <mono/metadata/exception.h>
 #include <mono/metadata/gc-internal.h>
 #include <mono/metadata/mono-debug.h>
+#include <mono/metadata/mono-debug-debugger.h>
 
 #include "mini.h"
 
@@ -105,6 +106,62 @@ mono_find_jit_info (MonoDomain *domain, MonoJitTlsData *jit_tls, MonoJitInfo *re
 
 #endif /* mono_find_jit_info */
 
+MonoString *
+ves_icall_System_Exception_get_trace (MonoException *ex)
+{
+       MonoDomain *domain = mono_domain_get ();
+       MonoString *res;
+       MonoArray *ta = ex->trace_ips;
+       int i, len;
+       GString *trace_str;
+       char tmpaddr [256];
+
+       if (ta == NULL)
+               /* Exception is not thrown yet */
+               return NULL;
+
+       len = mono_array_length (ta);
+       trace_str = g_string_new ("");
+       for (i = 0; i < len; i++) {
+               MonoJitInfo *ji;
+               gpointer ip = mono_array_get (ta, gpointer, i);
+
+               ji = mono_jit_info_table_find (domain, ip);
+               if (ji == NULL) {
+                       /* Unmanaged frame */
+                       g_string_append_printf (trace_str, "in (unmanaged) %p\n", ip);
+               } else {
+                       char *source_location, *fname;
+                       gint32 address, iloffset;
+
+                       address = (char *)ip - (char *)ji->code_start;
+
+                       source_location = mono_debug_source_location_from_address (ji->method, address, NULL, ex->object.vtable->domain);
+                       iloffset = mono_debug_il_offset_from_address (ji->method, address, ex->object.vtable->domain);
+
+                       if (iloffset < 0)
+                               sprintf (tmpaddr, "<0x%05x>", address);
+                       else
+                               sprintf (tmpaddr, "[0x%05x]", iloffset);
+               
+                       fname = mono_method_full_name (ji->method, TRUE);
+
+                       if (source_location)
+                               g_string_append_printf (trace_str, "in %s (at %s) %s\n", tmpaddr, source_location, fname);
+                       else
+                               g_string_append_printf (trace_str, "in %s %s\n", tmpaddr, fname);
+
+                       g_free (fname);
+                       g_free (source_location);
+               }
+       }
+
+       res = mono_string_new (ex->object.vtable->domain, trace_str->str);
+       g_string_free (trace_str, TRUE);
+
+       return res;
+}
+
 MonoArray *
 ves_icall_get_trace (MonoException *exc, gint32 skip, MonoBoolean need_file_info)
 {
@@ -158,7 +215,6 @@ ves_icall_get_trace (MonoException *exc, gint32 skip, MonoBoolean need_file_info
        return res;
 }
 
-
 /**
  * mono_walk_stack:
  * @domain: starting appdomain
@@ -449,14 +505,14 @@ mono_handle_exception (MonoContext *ctx, gpointer obj, gpointer original_ip, gbo
        static void (*restore_context) (void *);
        MonoJitTlsData *jit_tls = TlsGetValue (mono_jit_tls_id);
        MonoLMF *lmf = jit_tls->lmf;            
+       MonoArray *initial_trace_ips = NULL;
        GList *trace_ips = NULL;
        MonoException *mono_ex;
        gboolean stack_overflow = FALSE;
        MonoContext initial_ctx;
        int frame_count = 0;
        gboolean gc_disabled = FALSE;
-       MonoString *initial_stack_trace = NULL;
-       GString *trace_str = NULL;
+       gboolean has_dynamic_methods = FALSE;
        
        /*
         * This function might execute on an alternate signal stack, and Boehm GC
@@ -495,7 +551,7 @@ mono_handle_exception (MonoContext *ctx, gpointer obj, gpointer original_ip, gbo
 
        if (mono_object_isinst (obj, mono_defaults.exception_class)) {
                mono_ex = (MonoException*)obj;
-               initial_stack_trace = mono_ex->stack_trace;
+               initial_trace_ips = mono_ex->trace_ips;
        } else {
                mono_ex = NULL;
        }
@@ -541,18 +597,10 @@ mono_handle_exception (MonoContext *ctx, gpointer obj, gpointer original_ip, gbo
 
        while (1) {
                MonoContext new_ctx;
-               char *trace = NULL;
-               gboolean need_trace = FALSE;
                guint32 free_stack;
 
-               if (test_only && (frame_count < 1000)) {
-                       need_trace = TRUE;
-                       if (!trace_str)
-                               trace_str = g_string_new ("");
-               }
-
                ji = mono_find_jit_info (domain, jit_tls, &rji, &rji, ctx, &new_ctx, 
-                                                                need_trace ? &trace : NULL, &lmf, NULL, NULL);
+                                                                NULL, &lmf, NULL, NULL);
                if (!ji) {
                        g_warning ("Exception inside function without unwind info");
                        g_assert_not_reached ();
@@ -568,14 +616,14 @@ mono_handle_exception (MonoContext *ctx, gpointer obj, gpointer original_ip, gbo
                                 * rethrown. Also avoid giant stack traces during a stack
                                 * overflow.
                                 */
-                               if (!initial_stack_trace && (frame_count < 1000)) {
+                               if (!initial_trace_ips && (frame_count < 1000)) {
                                        trace_ips = g_list_prepend (trace_ips, MONO_CONTEXT_GET_IP (ctx));
-
-                                       g_string_append (trace_str, trace);
-                                       g_string_append_c (trace_str, '\n');
                                }
                        }
 
+                       if (ji->method->dynamic)
+                               has_dynamic_methods = TRUE;
+
                        if (stack_overflow)
                                free_stack = (guint8*)(MONO_CONTEXT_GET_BP (ctx)) - (guint8*)(MONO_CONTEXT_GET_BP (&initial_ctx));
                        else
@@ -588,23 +636,22 @@ mono_handle_exception (MonoContext *ctx, gpointer obj, gpointer original_ip, gbo
                        if ((free_stack > (64 * 1024)) && ji->num_clauses) {
                                int i;
                                
-                               g_assert (ji->clauses);
-                       
                                for (i = 0; i < ji->num_clauses; i++) {
                                        MonoJitExceptionInfo *ei = &ji->clauses [i];
                                        gboolean filtered = FALSE;
 
+#ifdef __s390__
+                                       if (ei->try_start < MONO_CONTEXT_GET_IP (ctx) && 
+#else
                                        if (ei->try_start <= MONO_CONTEXT_GET_IP (ctx) && 
+#endif
                                            MONO_CONTEXT_GET_IP (ctx) <= ei->try_end) { 
                                                /* catch block */
 
                                                if ((ei->flags == MONO_EXCEPTION_CLAUSE_NONE) || (ei->flags == MONO_EXCEPTION_CLAUSE_FILTER)) {
                                                        /* store the exception object int cfg->excvar */
-                                                       g_assert (ji->exvar_offset);
-                                                       *((gpointer *)((char *)MONO_CONTEXT_GET_BP (ctx) + ji->exvar_offset)) = obj;
-                                                       if (!initial_stack_trace && trace_str) {
-                                                               mono_ex->stack_trace = mono_string_new (domain, trace_str->str);
-                                                       }
+                                                       g_assert (ei->exvar_offset);
+                                                       *((gpointer *)((char *)MONO_CONTEXT_GET_BP (ctx) + ei->exvar_offset)) = obj;
                                                }
 
                                                if (ei->flags == MONO_EXCEPTION_CLAUSE_FILTER) {
@@ -615,17 +662,17 @@ mono_handle_exception (MonoContext *ctx, gpointer obj, gpointer original_ip, gbo
                                                if ((ei->flags == MONO_EXCEPTION_CLAUSE_NONE && 
                                                     mono_object_isinst (obj, ei->data.catch_class)) || filtered) {
                                                        if (test_only) {
-                                                               if (mono_ex) {
+                                                               if (mono_ex && !initial_trace_ips) {
                                                                        trace_ips = g_list_reverse (trace_ips);
                                                                        mono_ex->trace_ips = glist_to_array (trace_ips, mono_defaults.int_class);
+                                                                       if (has_dynamic_methods)
+                                                                               /* These methods could go away anytime, so compute the stack trace now */
+                                                                               mono_ex->stack_trace = ves_icall_System_Exception_get_trace (mono_ex);
                                                                }
                                                                g_list_free (trace_ips);
-                                                               g_free (trace);
 
                                                                if (gc_disabled)
                                                                        mono_gc_enable ();
-                                                               if (trace_str)
-                                                                       g_string_free (trace_str, TRUE);
                                                                return TRUE;
                                                        }
                                                        if (mono_jit_trace_calls != NULL && mono_trace_eval (ji->method))
@@ -633,12 +680,9 @@ mono_handle_exception (MonoContext *ctx, gpointer obj, gpointer original_ip, gbo
                                                        mono_debugger_handle_exception (ei->handler_start, MONO_CONTEXT_GET_SP (ctx), obj);
                                                        MONO_CONTEXT_SET_IP (ctx, ei->handler_start);
                                                        jit_tls->lmf = lmf;
-                                                       g_free (trace);
 
                                                        if (gc_disabled)
                                                                mono_gc_enable ();
-                                                       if (trace_str)
-                                                               g_string_free (trace_str, TRUE);
                                                        return 0;
                                                }
                                                if (!test_only && ei->try_start <= MONO_CONTEXT_GET_IP (ctx) && 
@@ -655,8 +699,6 @@ mono_handle_exception (MonoContext *ctx, gpointer obj, gpointer original_ip, gbo
                        }
                }
 
-               g_free (trace);
-                       
                *ctx = new_ctx;
 
                if ((ji == (gpointer)-1) || MONO_CONTEXT_GET_BP (ctx) >= jit_tls->end_of_stack) {
@@ -670,21 +712,22 @@ mono_handle_exception (MonoContext *ctx, gpointer obj, gpointer original_ip, gbo
                                        /* Switch back to normal stack */
                                        if (stack_overflow)
                                                /* Free up some stack space */
-                                               MONO_CONTEXT_SET_SP (&initial_ctx, (guint32)(MONO_CONTEXT_GET_SP (&initial_ctx)) + (64 * 1024));
-                                       MONO_CONTEXT_SET_IP (&initial_ctx, (unsigned int)jit_tls->abort_func);
+                                               MONO_CONTEXT_SET_SP (&initial_ctx, (gssize)(MONO_CONTEXT_GET_SP (&initial_ctx)) + (64 * 1024));
+                                       MONO_CONTEXT_SET_IP (&initial_ctx, (gssize)jit_tls->abort_func);
                                        restore_context (&initial_ctx);
                                }
                                else
                                        jit_tls->abort_func (obj);
                                g_assert_not_reached ();
                        } else {
-                               if (mono_ex) {
+                               if (mono_ex && !initial_trace_ips) {
                                        trace_ips = g_list_reverse (trace_ips);
                                        mono_ex->trace_ips = glist_to_array (trace_ips, mono_defaults.int_class);
+                                       if (has_dynamic_methods)
+                                               /* These methods could go away anytime, so compute the stack trace now */
+                                               mono_ex->stack_trace = ves_icall_System_Exception_get_trace (mono_ex);
                                }
                                g_list_free (trace_ips);
-                               if (trace_str)
-                                       g_string_free (trace_str, TRUE);
                                return FALSE;
                        }
                }