diff options
| author | Denys Vlasenko <dvlasenk@redhat.com> | 2010-11-15 16:58:16 +0100 |
|---|---|---|
| committer | Denys Vlasenko <dvlasenk@redhat.com> | 2010-11-15 16:58:16 +0100 |
| commit | 46d60e990d57758c1d72ed76efdeee19e55996f1 (patch) | |
| tree | 6563abd994c198d3fe699602d5a694fedbdd6e0f /src/daemon/MiddleWare.cpp | |
| parent | 2cf0d770b66b6c6ce50af2767f575db552cd784c (diff) | |
| download | abrt-46d60e990d57758c1d72ed76efdeee19e55996f1.tar.gz abrt-46d60e990d57758c1d72ed76efdeee19e55996f1.tar.xz abrt-46d60e990d57758c1d72ed76efdeee19e55996f1.zip | |
Make "abrt-handle-crashdump -e BOGUS_EVENT" fail (also fail if asked over dbus)
Signed-off-by: Denys Vlasenko <dvlasenk@redhat.com>
Diffstat (limited to 'src/daemon/MiddleWare.cpp')
| -rw-r--r-- | src/daemon/MiddleWare.cpp | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/src/daemon/MiddleWare.cpp b/src/daemon/MiddleWare.cpp index a29f2f3d..329c04ab 100644 --- a/src/daemon/MiddleWare.cpp +++ b/src/daemon/MiddleWare.cpp @@ -319,6 +319,10 @@ report_status_t Report(const map_crash_data_t& client_report, l_state.last_line = NULL; int r = run_event(run_state, dump_dir_name, plugin_name.c_str()); + if (r == -1) + { + l_state.last_line = xasprintf("Error: no processing is specified for event '%s'", plugin_name.c_str()); + } if (r == 0) { at_least_one_reporter_succeeded = true; @@ -347,9 +351,9 @@ report_status_t Report(const map_crash_data_t& client_report, { char *s = (char*)li->data; /* Need to make a copy: just cutting s at '=' and unsetenv'ing - * the result would be a bug! s _itself_ is in environment now, - * we must not modify it there! - */ + * the result would be a bug! s _itself_ is in environment now, + * we must not modify it there! + */ char *name = xstrndup(s, strchrnul(s, '=') - s); VERB3 log("Unexporting '%s'", name); unsetenv(name); |
