do not display backtrace capturing stuff in the displayed backtrace (plus, be eclipse...

[simgrid.git] / src / xbt / backtrace_linux.c

diff --git a/src/xbt/backtrace_linux.c b/src/xbt/backtrace_linux.c
index 63ca68e..b4fe6be 100644 (file)
--- a/src/xbt/backtrace_linux.c
+++ b/src/xbt/backtrace_linux.c
@@ -9,12 +9,24 @@
 /* This program is free software; you can redistribute it and/or modify it
  * under the terms of the license (GNU LGPL) which comes with this package. */
 
 /* This program is free software; you can redistribute it and/or modify it
  * under the terms of the license (GNU LGPL) which comes with this package. */
 

+/* This file is to be included in ex.c, so the following headers are not mandatory, but it's to make sure that eclipse see them too */
+#include "xbt/ex.h"
+#include "xbt/str.h"
+#include "xbt/module.h" /* xbt_binary_name */
+#include "xbt_modinter.h" /* backtrace initialization headers */
+/* end of "useless" inclusions */
+

 extern char **environ;          /* the environment, as specified by the opengroup */
 
 /* Module creation/destruction: nothing to do on linux */
 void xbt_backtrace_init(void) { }
 void xbt_backtrace_exit(void) { }
 
 extern char **environ;          /* the environment, as specified by the opengroup */
 
 /* Module creation/destruction: nothing to do on linux */
 void xbt_backtrace_init(void) { }
 void xbt_backtrace_exit(void) { }
 

+void xbt_backtrace_current(xbt_ex_t * e) {
+  e->used = backtrace((void **) e->bt, XBT_BACKTRACE_SIZE);
+}
+
+

 void xbt_ex_setup_backtrace(xbt_ex_t * e)
 {
   int i;
 void xbt_ex_setup_backtrace(xbt_ex_t * e)
 {
   int i;


@@ -39,11 +51,15 @@ void xbt_ex_setup_backtrace(xbt_ex_t * e)
   /* To search for the right executable path when not trivial */
   struct stat stat_buf;
   char *binary_name = NULL;
   /* To search for the right executable path when not trivial */
   struct stat stat_buf;
   char *binary_name = NULL;

-   
-  e->used = backtrace((void **) e->bt, XBT_BACKTRACE_SIZE);
+
+  xbt_assert0(e && e->used,"Backtrace not setup yet, cannot set it up for display");
+

   backtrace_syms = backtrace_symbols(e->bt, e->used);
   backtrace_syms = backtrace_symbols(e->bt, e->used);

+  /* ignore first one, which is this xbt_backtrace_current() */
+  e->used--;
+  memmove(backtrace_syms,backtrace_syms+1,sizeof(char*)*e->used);

   addrs = xbt_new(char *, e->used);
   addrs = xbt_new(char *, e->used);

-   
+

   e->bt_strings = NULL;
 
   /* Some arches only have stubs of backtrace, no implementation (hppa comes to mind) */
   e->bt_strings = NULL;
 
   /* Some arches only have stubs of backtrace, no implementation (hppa comes to mind) */


@@ -124,10 +140,15 @@ void xbt_ex_setup_backtrace(xbt_ex_t * e)
   }
 
   for (i = 0; i < e->used; i++) {
   }
 
   for (i = 0; i < e->used; i++) {

+    char *fgets_res;

     DEBUG2("Looking for symbol %d, addr = '%s'", i, addrs[i]);
     DEBUG2("Looking for symbol %d, addr = '%s'", i, addrs[i]);

-    fgets(line_func, 1024, pipe);
+    fgets_res = fgets(line_func, 1024, pipe);
+    if (fgets_res == NULL)
+      THROW2(system_error,0, "Cannot run fgets to look for symbol %d, addr %s",i, addrs[i]);

     line_func[strlen(line_func) - 1] = '\0';
     line_func[strlen(line_func) - 1] = '\0';

-    fgets(line_pos, 1024, pipe);
+    fgets_res = fgets(line_pos, 1024, pipe);
+    if (fgets_res == NULL)
+      THROW2(system_error,0, "Cannot run fgets to look for symbol %d, addr %s",i, addrs[i]);

     line_pos[strlen(line_pos) - 1] = '\0';
 
     if (strcmp("??", line_func)) {
     line_pos[strlen(line_pos) - 1] = '\0';
 
     if (strcmp("??", line_func)) {


@@ -178,7 +199,7 @@ void xbt_ex_setup_backtrace(xbt_ex_t * e)
         if (found) {
           DEBUG3("%#lx in [%#lx-%#lx]", addr, first, last);
           DEBUG0
         if (found) {
           DEBUG3("%#lx in [%#lx-%#lx]", addr, first, last);
           DEBUG0

-            ("Symbol found, map lines not further displayed (even if looking for next ones)");
+          ("Symbol found, map lines not further displayed (even if looking for next ones)");

         }
       }
       fclose(maps);
         }
       }
       fclose(maps);


@@ -186,13 +207,13 @@ void xbt_ex_setup_backtrace(xbt_ex_t * e)
 
       if (!found) {
         VERB0
 
       if (!found) {
         VERB0

-          ("Problem while reading the maps file. Following backtrace will be mangled.");
+        ("Problem while reading the maps file. Following backtrace will be mangled.");

         DEBUG1("No dynamic. Static symbol: %s", backtrace_syms[i]);
         e->bt_strings[i] = bprintf("**   In ?? (%s)", backtrace_syms[i]);
         continue;
       }
 
         DEBUG1("No dynamic. Static symbol: %s", backtrace_syms[i]);
         e->bt_strings[i] = bprintf("**   In ?? (%s)", backtrace_syms[i]);
         continue;
       }
 

-      /* Ok, Found the offset of the maps line containing the searched symbol. 
+      /* Ok, Found the offset of the maps line containing the searched symbol.

          We now need to substract this from the address we got from backtrace.
        */
 
          We now need to substract this from the address we got from backtrace.
        */
 


@@ -200,7 +221,7 @@ void xbt_ex_setup_backtrace(xbt_ex_t * e)
       addrs[i] = bprintf("0x%0*lx", addr_len - 2, addr - offset);
       DEBUG2("offset=%#lx new addr=%s", offset, addrs[i]);
 
       addrs[i] = bprintf("0x%0*lx", addr_len - 2, addr - offset);
       DEBUG2("offset=%#lx new addr=%s", offset, addrs[i]);
 

-      /* Got it. We have our new address. Let's get the library path and we 
+      /* Got it. We have our new address. Let's get the library path and we

          are set */
       p = xbt_strdup(backtrace_syms[i]);
       if (p[0] == '[') {
          are set */
       p = xbt_strdup(backtrace_syms[i]);
       if (p[0] == '[') {


@@ -224,9 +245,13 @@ void xbt_ex_setup_backtrace(xbt_ex_t * e)
           CRITICAL0("Cannot fork addr2line to display the backtrace");
           abort();
         }
           CRITICAL0("Cannot fork addr2line to display the backtrace");
           abort();
         }

-        fgets(line_func, 1024, subpipe);
+        fgets_res=fgets(line_func, 1024, subpipe);
+        if (fgets_res == NULL)
+          THROW1(system_error,0, "Cannot read result of subcommand %s",subcmd);

         line_func[strlen(line_func) - 1] = '\0';
         line_func[strlen(line_func) - 1] = '\0';

-        fgets(line_pos, 1024, subpipe);
+        fgets_res = fgets(line_pos, 1024, subpipe);
+        if (fgets_res == NULL)
+          THROW1(system_error,0, "Cannot read result of subcommand %s",subcmd);

         line_pos[strlen(line_pos) - 1] = '\0';
         pclose(subpipe);
         free(subcmd);
         line_pos[strlen(line_pos) - 1] = '\0';
         pclose(subpipe);
         free(subcmd);