diff options
author | Karel Klic <kklic@redhat.com> | 2011-03-28 18:08:55 +0200 |
---|---|---|
committer | Karel Klic <kklic@redhat.com> | 2011-03-28 18:08:55 +0200 |
commit | 4def4f1609679232ea2c457ccc6bc4621184201d (patch) | |
tree | e9e441843e24856ab04f68335d2f3e21f76bcea9 | |
parent | 466f32513e67bf937d42321d1b979c5a477905b6 (diff) | |
download | abrt-4def4f1609679232ea2c457ccc6bc4621184201d.tar.gz abrt-4def4f1609679232ea2c457ccc6bc4621184201d.tar.xz abrt-4def4f1609679232ea2c457ccc6bc4621184201d.zip |
abrt-action-analyze-backtrace: log backtrace parser failure every time; remove dead code
-rw-r--r-- | src/plugins/abrt-action-analyze-backtrace.c | 10 |
1 files changed, 2 insertions, 8 deletions
diff --git a/src/plugins/abrt-action-analyze-backtrace.c b/src/plugins/abrt-action-analyze-backtrace.c index e65b1781..b690fc9b 100644 --- a/src/plugins/abrt-action-analyze-backtrace.c +++ b/src/plugins/abrt-action-analyze-backtrace.c @@ -88,13 +88,7 @@ int main(int argc, char **argv) char *executable = dd_load_text(dd, FILENAME_EXECUTABLE); /* Read backtrace */ - /* NB: get_backtrace() closes dd */ char *backtrace_str = dd_load_text(dd, FILENAME_BACKTRACE); - if (!backtrace_str) - { - backtrace_str = xstrdup(""); - log("Backtrace file is missing"); - } /* Compute backtrace hash */ struct btp_location location; @@ -110,8 +104,8 @@ int main(int argc, char **argv) * The parser failed. Compute the UUID from the executable * and package only. This is not supposed to happen often. */ - VERB1 log(_("Backtrace parsing failed for %s"), dump_dir_name); - VERB1 log("%d:%d: %s", location.line, location.column, location.message); + log(_("Backtrace parsing failed for %s"), dump_dir_name); + log("%d:%d: %s", location.line, location.column, location.message); struct strbuf *emptybt = strbuf_new(); strbuf_prepend_str(emptybt, executable); strbuf_prepend_str(emptybt, package); |