summaryrefslogtreecommitdiff
path: root/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/create_interaction_choice_set_request.cc
diff options
context:
space:
mode:
authorjacobkeeler <jacob.keeler@livioradio.com>2018-09-28 09:44:01 -0400
committerjacobkeeler <jacob.keeler@livioradio.com>2018-09-28 09:44:01 -0400
commit0c32601453970fa16798d3d917b360c0a8ac3c02 (patch)
tree3636c29c2b8e29821878f6ad6878b1a969b30909 /src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/create_interaction_choice_set_request.cc
parent35defc42777c57412cbf8b793489ab0dac502f4e (diff)
downloadsdl_core-0c32601453970fa16798d3d917b360c0a8ac3c02.tar.gz
Fix uninitialized membersfix/coverity_fixes_5.0.0
CIDs 188281, 188276, 188250, 188245, 188239
Diffstat (limited to 'src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/create_interaction_choice_set_request.cc')
-rw-r--r--src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/create_interaction_choice_set_request.cc7
1 files changed, 4 insertions, 3 deletions
diff --git a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/create_interaction_choice_set_request.cc b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/create_interaction_choice_set_request.cc
index cd1f95b93c..416f4f2085 100644
--- a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/create_interaction_choice_set_request.cc
+++ b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/create_interaction_choice_set_request.cc
@@ -63,6 +63,7 @@ CreateInteractionChoiceSetRequest::CreateInteractionChoiceSetRequest(
, choice_set_id_(0)
, expected_chs_count_(0)
, received_chs_count_(0)
+ , should_send_warnings_(false)
, error_from_hmi_(false)
, is_timed_out_(false) {}
@@ -80,7 +81,7 @@ void CreateInteractionChoiceSetRequest::Run() {
SendResponse(false, mobile_apis::Result::APPLICATION_NOT_REGISTERED);
return;
}
- should_send_warnings = false;
+
for (uint32_t i = 0;
i < (*message_)[strings::msg_params][strings::choice_set].length();
++i) {
@@ -109,7 +110,7 @@ void CreateInteractionChoiceSetRequest::Run() {
return;
} else if (verification_result_image == Result::WARNINGS ||
verification_result_secondary_image == Result::WARNINGS) {
- should_send_warnings = true;
+ should_send_warnings_ = true;
break;
}
}
@@ -465,7 +466,7 @@ void CreateInteractionChoiceSetRequest::DeleteChoices() {
void CreateInteractionChoiceSetRequest::OnAllHMIResponsesReceived() {
LOG4CXX_AUTO_TRACE(logger_);
- if (!error_from_hmi_ && should_send_warnings) {
+ if (!error_from_hmi_ && should_send_warnings_) {
SendResponse(true, mobile_apis::Result::WARNINGS, kInvalidImageWarningInfo);
} else if (!error_from_hmi_) {
SendResponse(true, mobile_apis::Result::SUCCESS);