From 94b1d40a72f55b787fa78a9398f47eb8514e0d21 Mon Sep 17 00:00:00 2001 From: Denys Vlasenko Date: Mon, 8 Feb 2010 13:38:50 +0100 Subject: make "reason" field less verbose; bz reporter: include it in "summary" Signed-off-by: Denys Vlasenko --- inc/abrtlib.h | 3 +++ lib/Plugins/Bugzilla.cpp | 6 ++++++ src/Hooks/abrt-hook-ccpp.cpp | 25 +++++++++++++++---------- 3 files changed, 24 insertions(+), 10 deletions(-) diff --git a/inc/abrtlib.h b/inc/abrtlib.h index 443c3725..97aa28b8 100644 --- a/inc/abrtlib.h +++ b/inc/abrtlib.h @@ -52,6 +52,9 @@ int vdprintf(int d, const char *format, va_list ap); #define NORETURN __attribute__ ((noreturn)) +#define ARRAY_SIZE(x) ((unsigned)(sizeof(x) / sizeof((x)[0]))) + + /* Logging */ enum { LOGMODE_NONE = 0, diff --git a/lib/Plugins/Bugzilla.cpp b/lib/Plugins/Bugzilla.cpp index b809e34c..9e889c4b 100644 --- a/lib/Plugins/Bugzilla.cpp +++ b/lib/Plugins/Bugzilla.cpp @@ -315,8 +315,14 @@ uint32_t ctx::new_bug(const map_crash_data_t& pCrashData) const std::string& release = get_crash_data_item_content(pCrashData, FILENAME_RELEASE); const std::string& arch = get_crash_data_item_content(pCrashData, FILENAME_ARCHITECTURE); const std::string& uuid = get_crash_data_item_content(pCrashData, CD_DUPHASH); + const char *reason = get_crash_data_item_content_or_NULL(pCrashData, FILENAME_REASON); std::string summary = "[abrt] crash in " + package; + if (reason != NULL) + { + summary += ": "; + summary += reason; + } std::string status_whiteboard = "abrt_hash:" + uuid; std::string description = "abrt "VERSION" detected a crash.\n\n"; diff --git a/src/Hooks/abrt-hook-ccpp.cpp b/src/Hooks/abrt-hook-ccpp.cpp index d3f6b25d..54a4c7a5 100644 --- a/src/Hooks/abrt-hook-ccpp.cpp +++ b/src/Hooks/abrt-hook-ccpp.cpp @@ -88,17 +88,23 @@ int main(int argc, char** argv) { error_msg_and_die("pid '%s' or limit '%s' is bogus", argv[2], argv[5]); } - if (signal_no != SIGQUIT - && signal_no != SIGILL - && signal_no != SIGABRT - && signal_no != SIGFPE - && signal_no != SIGSEGV - ) { - /* not an error, exit silently */ + + const char *signame = NULL; + /* Tried to use array for this but C++ does not support v[] = { [IDX] = "str" } */ + switch (signal_no) + { + case SIGQUIT: signame = "QUIT"; break; + case SIGILL : signame = "ILL" ; break; + case SIGABRT: signame = "ABRT"; break; + case SIGFPE : signame = "FPE" ; break; + case SIGSEGV: signame = "SEGV"; break; + } + if (signame == NULL) + { + /* not a signal we care about, exit silently */ return 0; } - char *user_pwd = get_cwd(pid); /* may be NULL on error */ int core_fd = STDIN_FILENO; @@ -192,8 +198,7 @@ int main(int argc, char** argv) } char* cmdline = get_cmdline(pid); /* never NULL */ - const char *signame = strsignal(signal_no); - char *reason = xasprintf("Process was terminated by signal %s (%s)", signal_str, signame ? signame : signal_str); + char *reason = xasprintf("Process %s was killed by signal %s (SIG%s)", executable, signal_str, signame ? signame : signal_str); unsigned path_len = snprintf(path, sizeof(path), "%s/ccpp-%ld-%lu.new", dddir, (long)time(NULL), (long)pid); -- cgit