diff options
author | jacobkeeler <jacob.keeler@livioradio.com> | 2018-09-28 09:44:01 -0400 |
---|---|---|
committer | jacobkeeler <jacob.keeler@livioradio.com> | 2018-09-28 09:44:01 -0400 |
commit | 0c32601453970fa16798d3d917b360c0a8ac3c02 (patch) | |
tree | 3636c29c2b8e29821878f6ad6878b1a969b30909 /src/components/application_manager/rpc_plugins | |
parent | 35defc42777c57412cbf8b793489ab0dac502f4e (diff) | |
download | sdl_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')
3 files changed, 6 insertions, 4 deletions
diff --git a/src/components/application_manager/rpc_plugins/rc_rpc_plugin/src/commands/rc_command_request.cc b/src/components/application_manager/rpc_plugins/rc_rpc_plugin/src/commands/rc_command_request.cc index 40249387a7..c24529d720 100644 --- a/src/components/application_manager/rpc_plugins/rc_rpc_plugin/src/commands/rc_command_request.cc +++ b/src/components/application_manager/rpc_plugins/rc_rpc_plugin/src/commands/rc_command_request.cc @@ -52,6 +52,7 @@ RCCommandRequest::RCCommandRequest( params.hmi_capabilities_, params.policy_handler_) , is_subscribed(false) + , auto_allowed_(false) , resource_allocation_manager_(params.resource_allocation_manager_) , interior_data_cache_(params.interior_data_cache_) , interior_data_manager_(params.interior_data_manager_) {} diff --git a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/include/sdl_rpc_plugin/commands/mobile/create_interaction_choice_set_request.h b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/include/sdl_rpc_plugin/commands/mobile/create_interaction_choice_set_request.h index 4564858845..1ad1c07b2a 100644 --- a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/include/sdl_rpc_plugin/commands/mobile/create_interaction_choice_set_request.h +++ b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/include/sdl_rpc_plugin/commands/mobile/create_interaction_choice_set_request.h @@ -128,7 +128,7 @@ class CreateInteractionChoiceSetRequest int32_t choice_set_id_; size_t expected_chs_count_; size_t received_chs_count_; - bool should_send_warnings; + bool should_send_warnings_; /** * @brief Flag for stop sending VR commands to HMI, in case one of responses 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); |