From f964da092f6191bd02754ee0dc786107d85d5a3e Mon Sep 17 00:00:00 2001 From: Denys Vlasenko Date: Wed, 27 Jan 2010 13:28:46 +0100 Subject: abrtd: set "Reported" status only if at least one reporter succeeded Signed-off-by: Denys Vlasenko --- src/Daemon/MiddleWare.cpp | 17 +++++++++++------ 1 file changed, 11 insertions(+), 6 deletions(-) (limited to 'src/Daemon') diff --git a/src/Daemon/MiddleWare.cpp b/src/Daemon/MiddleWare.cpp index c56b7b1c..01c3850b 100644 --- a/src/Daemon/MiddleWare.cpp +++ b/src/Daemon/MiddleWare.cpp @@ -437,6 +437,7 @@ report_status_t Report(const map_crash_data_t& client_report, key = analyzer; } + bool at_least_one_reporter_succeeded = false; report_status_t ret; std::string message; if (keyPtr != end) @@ -485,6 +486,7 @@ report_status_t Report(const map_crash_data_t& client_report, if (message != "") message += "; "; message += res; + at_least_one_reporter_succeeded = true; } } catch (CABRTException& e) @@ -493,13 +495,16 @@ report_status_t Report(const map_crash_data_t& client_report, ret[plugin_name].push_back(e.what()); // REPORT_STATUS_IDX_MSG update_client("Reporting via '%s' was not successful: %s", plugin_name, e.what()); } - } - } + } // for + } // if - CDatabase* database = g_pPluginManager->GetDatabase(g_settings_sDatabase.c_str()); - database->Connect(); - database->SetReported(UUID, UID, message.c_str()); - database->DisConnect(); + if (at_least_one_reporter_succeeded) + { + CDatabase* database = g_pPluginManager->GetDatabase(g_settings_sDatabase.c_str()); + database->Connect(); + database->SetReported(UUID, UID, message.c_str()); + database->DisConnect(); + } return ret; #undef client_report -- cgit