diff options
author | Jacob Keeler <jacob.keeler@livioradio.com> | 2020-09-21 14:42:56 -0400 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-09-21 14:42:56 -0400 |
commit | 053e1850c52dc940f8b01a49b7697c244fb3db6d (patch) | |
tree | 82f5afd55f6e2b4daa4485aa9b0479d98afad485 /src | |
parent | 93202c39ac25d6bd6ccd1c9df948106dab38acfa (diff) | |
parent | 68aa6ad1b2aa0419a73538de8717a23e41c0e1a7 (diff) | |
download | sdl_core-053e1850c52dc940f8b01a49b7697c244fb3db6d.tar.gz |
Merge pull request #3507 from smartdevicelink/fix/coverity_7.0
Add fixes for new Coverity issues
Diffstat (limited to 'src')
3 files changed, 8 insertions, 2 deletions
diff --git a/src/components/application_manager/include/application_manager/resumption/resumption_data_processor.h b/src/components/application_manager/include/application_manager/resumption/resumption_data_processor.h index 60e2e5d1a4..3f55b02ed3 100644 --- a/src/components/application_manager/include/application_manager/resumption/resumption_data_processor.h +++ b/src/components/application_manager/include/application_manager/resumption/resumption_data_processor.h @@ -74,6 +74,11 @@ struct ResumptionRequest { class ResumptionDataProcessor { public: /** + * @brief ResumptionDataProcessor class destructor + */ + virtual ~ResumptionDataProcessor() {} + + /** * @brief Running resumption data process from saved_app to application. * @param application application which will be resumed * @param saved_app application specific section from backup file 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 c067167041..a490429197 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 @@ -1015,7 +1015,7 @@ void ResumptionDataProcessorImpl::DeletePluginsSubscriptions( if (!status.successful_module_subscriptions_.empty()) { extension_modules_subscriptions[message_params::kModuleData] = - new smart_objects::SmartObject(smart_objects::SmartType_Array); + smart_objects::SmartObject(smart_objects::SmartType_Array); auto& module_data_so = extension_modules_subscriptions[message_params::kModuleData]; @@ -1023,7 +1023,7 @@ void ResumptionDataProcessorImpl::DeletePluginsSubscriptions( uint32_t index = 0; for (const auto& module : status.successful_module_subscriptions_) { module_data_so[index] = - new smart_objects::SmartObject(smart_objects::SmartType_Map); + smart_objects::SmartObject(smart_objects::SmartType_Map); module_data_so[index][message_params::kModuleType] = module.first; module_data_so[index][message_params::kModuleId] = module.second; index++; diff --git a/src/components/include/utils/ilogger.h b/src/components/include/utils/ilogger.h index ba2f93df9b..3cf4777391 100644 --- a/src/components/include/utils/ilogger.h +++ b/src/components/include/utils/ilogger.h @@ -76,6 +76,7 @@ class Logger { class ThirdPartyLoggerInterface { public: + virtual ~ThirdPartyLoggerInterface() {} virtual void Init() = 0; virtual void DeInit() = 0; virtual bool IsEnabledFor(const std::string& component, |