summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorjacobkeeler <jacob.keeler@livioradio.com>2021-01-26 13:14:19 -0500
committerjacobkeeler <jacob.keeler@livioradio.com>2021-01-26 13:14:19 -0500
commitc6f73b8a3f87cd3f295c2e912e39372b9e56897f (patch)
tree7b4e66f4ec12175237c7ca6ebb99b9435915576d
parent3d28b125b14c988839db9981e432adb3a68389b7 (diff)
downloadsdl_core-c6f73b8a3f87cd3f295c2e912e39372b9e56897f.tar.gz
Fix invalid iterators in resumption data processor
-rw-r--r--src/components/application_manager/src/resumption/resumption_data_processor_impl.cc14
1 files changed, 7 insertions, 7 deletions
diff --git a/src/components/application_manager/src/resumption/resumption_data_processor_impl.cc b/src/components/application_manager/src/resumption/resumption_data_processor_impl.cc
index 17980ae5ad..3b091c74a8 100644
--- a/src/components/application_manager/src/resumption/resumption_data_processor_impl.cc
+++ b/src/components/application_manager/src/resumption/resumption_data_processor_impl.cc
@@ -534,9 +534,9 @@ void ResumptionDataProcessorImpl::DeleteSubmenus(
application->app_id(), resumption_status_, resumption_status_lock_);
auto accessor = application->sub_menu_map();
- const auto& sub_menu_map = accessor.GetData();
+ const auto sub_menu_map = accessor.GetData();
- for (const auto& smenu : sub_menu_map) {
+ for (const auto smenu : sub_menu_map) {
auto failed_submenu_request =
FindResumptionSubmenuRequest(smenu.first, failed_requests);
if (!failed_submenu_request) {
@@ -627,9 +627,9 @@ void ResumptionDataProcessorImpl::DeleteCommands(
};
auto accessor = application->commands_map();
- const auto& commands_map = accessor.GetData();
+ const auto commands_map = accessor.GetData();
- for (const auto& cmd : commands_map) {
+ for (const auto cmd : commands_map) {
const auto cmd_id = extract_cmd_id(cmd.second);
if (0 == cmd_id) {
SDL_LOG_ERROR("Can't extract cmd_id for command with internal number: "
@@ -719,8 +719,8 @@ void ResumptionDataProcessorImpl::DeleteChoicesets(
application->app_id(), resumption_status_, resumption_status_lock_);
auto accessor = application->choice_set_map();
- const auto& choices = accessor.GetData();
- for (const auto& choice : choices) {
+ const auto choices = accessor.GetData();
+ for (const auto choice : choices) {
auto failed_choice_set =
FindResumptionChoiceSetRequest(choice.first, failed_requests);
if (!failed_choice_set) {
@@ -881,7 +881,7 @@ void ResumptionDataProcessorImpl::DeleteButtonsSubscriptions(
SDL_LOG_AUTO_TRACE();
const ButtonSubscriptions button_subscriptions =
application->SubscribedButtons().GetData();
- for (auto& btn : button_subscriptions) {
+ for (auto btn : button_subscriptions) {
const auto hmi_btn = static_cast<hmi_apis::Common_ButtonName::eType>(btn);
if (hmi_apis::Common_ButtonName::CUSTOM_BUTTON == hmi_btn) {
continue;