[PATCH] fix core backtrace generation when debuginfo is present

[Richard Marko]

In case when package debuginfo is present, gdb will output only
function names but no offsets preventing core backtrace generation.

Setting non-existent debuginfo directory resolves this.
---
 src/lib/hooklib.c                                 | 29 +++++++++++++++--------
 src/plugins/abrt-action-generate-core-backtrace.c |  2 +-
 2 files changed, 20 insertions(+), 11 deletions(-)

diff --git a/src/lib/hooklib.c b/src/lib/hooklib.c
index 1b30a5b..ca799e0 100644
--- a/src/lib/hooklib.c
+++ b/src/lib/hooklib.c
@@ -249,17 +249,26 @@ char *get_backtrace(const char *dump_dir_name, unsigned timeout_sec, const char
     // (https://bugzilla.redhat.com/show_bug.cgi?id=528668):
     args[2] = (char*)"-ex";
     struct strbuf *set_debug_file_directory = strbuf_new();
-    strbuf_append_str(set_debug_file_directory, "set debug-file-directory /usr/lib/debug");
-    const char *p = debuginfo_dirs;
-    while (1)
+    if(debuginfo_dirs == NULL)
     {
-        while (*p == ':')
-            p++;
-        if (*p == '\0')
-            break;
-        const char *colon_or_nul = strchrnul(p, ':');
-        strbuf_append_strf(set_debug_file_directory, ":%.*s/usr/lib/debug", (int)(colon_or_nul - p), p);
-        p = colon_or_nul;
+        // set non-existent debug file directory to prevent resolving
+        // function names - we need offsets for core backtrace.
+        strbuf_append_str(set_debug_file_directory, "set debug-file-directory /");
+    }
+    else
+    {
+        strbuf_append_str(set_debug_file_directory, "set debug-file-directory /usr/lib/debug");
+        const char *p = debuginfo_dirs;
+        while (1)
+        {
+            while (*p == ':')
+                p++;
+            if (*p == '\0')
+                break;
+            const char *colon_or_nul = strchrnul(p, ':');
+            strbuf_append_strf(set_debug_file_directory, ":%.*s/usr/lib/debug", (int)(colon_or_nul - p), p);
+            p = colon_or_nul;
+        }
     }
     args[3] = strbuf_free_nobuf(set_debug_file_directory);

diff --git a/src/plugins/abrt-action-generate-core-backtrace.c b/src/plugins/abrt-action-generate-core-backtrace.c
index 5fccb89..9bd09a4 100644
--- a/src/plugins/abrt-action-generate-core-backtrace.c
+++ b/src/plugins/abrt-action-generate-core-backtrace.c
@@ -106,7 +106,7 @@ int main(int argc, char **argv)
     if (strcmp(analyzer, "CCpp") == 0)
     {
         VERB1 log("Querying gdb for backtrace");
-        char *gdb_out = get_backtrace(dump_dir_name, exec_timeout_sec, "");
+        char *gdb_out = get_backtrace(dump_dir_name, exec_timeout_sec, NULL);
         if (gdb_out == NULL)
             xfunc_die();

[Jakub Filak]

It works! Push it

On Wednesday 29 of August 2012 19:53:08 Richard Marko wrote:
> In case when package debuginfo is present, gdb will output only
> function names but no offsets preventing core backtrace generation.
> Setting non-existent debuginfo directory resolves this.
>
> src/lib/hooklib.c                                 | 29
> +++++++++++++++-------- src/plugins/abrt-action-generate-core-backtrace.c |
>  2 +-
>  2 files changed, 20 insertions(+), 11 deletions(-)
>
> diff --git a/src/lib/hooklib.c b/src/lib/hooklib.c
> index 1b30a5b..ca799e0 100644
> --- a/src/lib/hooklib.c
> +++ b/src/lib/hooklib.c
> @@ -249,17 +249,26 @@ char *get_backtrace(const char *dump_dir_name,
> unsigned timeout_sec, const char //
> (https://bugzilla.redhat.com/show_bug.cgi?id=528668):
>      args[2] = (char*)"-ex";
>      struct strbuf *set_debug_file_directory = strbuf_new();
> strbuf_append_str(set_debug_file_directory, "set debug-file-directory
>
> /usr/lib/debug"); -    const char *p = debuginfo_dirs;
> while (1)
> if(debuginfo_dirs == NULL)
> {
>    while (*p == ':')
>        p++;
>    if (*p == '\0')
>        break;
>    const char *colon_or_nul = strchrnul(p, ':');
>    strbuf_append_strf(set_debug_file_directory, ":%.*s/usr/lib/debug",
>
> (int)(colon_or_nul - p), p); -        p = colon_or_nul;
>    // set non-existent debug file directory to prevent resolving
>    // function names - we need offsets for core backtrace.
>    strbuf_append_str(set_debug_file_directory, "set
>
> debug-file-directory /"); +    }
> else
> {
>    strbuf_append_str(set_debug_file_directory, "set
>
> debug-file-directory /usr/lib/debug"); +        const char *p =
> debuginfo_dirs;
>    while (1)
>    {
>        while (*p == ':')
>            p++;
>        if (*p == '\0')
>            break;
>        const char *colon_or_nul = strchrnul(p, ':');
>        strbuf_append_strf(set_debug_file_directory,
>
> ":%.*s/usr/lib/debug", (int)(colon_or_nul - p), p); +            p =
> colon_or_nul;
>    }
> }
>    args[3] = strbuf_free_nobuf(set_debug_file_directory);
>
> diff --git a/src/plugins/abrt-action-generate-core-backtrace.c
> b/src/plugins/abrt-action-generate-core-backtrace.c index 5fccb89..9bd09a4
> 100644
> --- a/src/plugins/abrt-action-generate-core-backtrace.c
> +++ b/src/plugins/abrt-action-generate-core-backtrace.c
> @@ -106,7 +106,7 @@ int main(int argc, char **argv)
>      if (strcmp(analyzer, "CCpp") == 0)
>      {
>          VERB1 log("Querying gdb for backtrace");
>    char *gdb_out = get_backtrace(dump_dir_name, exec_timeout_sec, "");
>    char *gdb_out = get_backtrace(dump_dir_name, exec_timeout_sec,
>
> NULL); if (gdb_out == NULL)
>              xfunc_die();