Skip to content

Commit c2cd0c5

Browse files
authored
[NPU] Fix coverity (#34756)
### Details: - *Fix coverity to protect the logger as well* ### Tickets: - *E#208646* ### AI Assistance: - *AI assistance used: no / yes* - *If yes, summarize how AI was used and what human validation was performed (build/tests/manual checks).* Signed-off-by: Bogdan Pereanu <bogdan.pereanu@intel.com>
1 parent cda2053 commit c2cd0c5

File tree

1 file changed

+15
-14
lines changed

1 file changed

+15
-14
lines changed

src/plugins/intel_npu/src/plugin/src/properties.cpp

Lines changed: 15 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -1093,13 +1093,15 @@ bool Properties::isPropertyRegistered(const std::string& propertyName) const {
10931093

10941094
FilteredConfig Properties::getConfigForSpecificCompiler(const ov::AnyMap& properties,
10951095
const ICompilerAdapter* compiler) {
1096-
auto [updatedConfig, compilerConfigsFilteredByCompiler, currentlyUsedCompiler, currentlyUsedPlatform] = [&]() {
1097-
std::lock_guard<std::mutex> lock(_mutex);
1098-
return std::make_tuple(_config,
1099-
_compilerConfigsFilteredByCompiler,
1100-
_currentlyUsedCompiler,
1101-
_currentlyUsedPlatform);
1102-
}();
1096+
auto [updatedConfig, compilerConfigsFilteredByCompiler, currentlyUsedCompiler, currentlyUsedPlatform, logger] =
1097+
[&]() {
1098+
std::lock_guard<std::mutex> lock(_mutex);
1099+
return std::make_tuple(_config,
1100+
_compilerConfigsFilteredByCompiler,
1101+
_currentlyUsedCompiler,
1102+
_currentlyUsedPlatform,
1103+
_logger);
1104+
}();
11031105

11041106
std::optional<ov::intel_npu::CompilerType> propertiesCompilerType = std::nullopt;
11051107
std::optional<std::string> propertiesPlatform = std::nullopt;
@@ -1120,7 +1122,7 @@ FilteredConfig Properties::getConfigForSpecificCompiler(const ov::AnyMap& proper
11201122
propertiesPlatform.value_or(currentlyUsedPlatform) == currentlyUsedPlatform)) {
11211123
// In case the compiler properties are not initialized or the compiler/platform was changed since last call -
11221124
// filter out options again
1123-
filterPropertiesByCompilerSupport(updatedConfig, compiler, _backend, _logger);
1125+
filterPropertiesByCompilerSupport(updatedConfig, compiler, _backend, logger);
11241126
}
11251127

11261128
const std::map<std::string, std::string> rawConfig = any_copy(properties);
@@ -1144,9 +1146,9 @@ FilteredConfig Properties::getConfigForSpecificCompiler(const ov::AnyMap& proper
11441146
}
11451147

11461148
FilteredConfig Properties::getConfigWithCompilerPropertiesDisabled(const ov::AnyMap& properties) {
1147-
auto [updatedConfig, compilerConfigsFilteredByCompiler] = [&]() {
1149+
auto [updatedConfig, compilerConfigsFilteredByCompiler, logger] = [&]() {
11481150
std::lock_guard<std::mutex> lock(_mutex);
1149-
return std::make_tuple(_config, _compilerConfigsFilteredByCompiler);
1151+
return std::make_tuple(_config, _compilerConfigsFilteredByCompiler, _logger);
11501152
}();
11511153

11521154
if (compilerConfigsFilteredByCompiler) {
@@ -1164,16 +1166,15 @@ FilteredConfig Properties::getConfigWithCompilerPropertiesDisabled(const ov::Any
11641166
const auto optionMode = updatedConfig.getOpt(key).mode();
11651167

11661168
if (optionMode == OptionMode::CompileTime) {
1167-
_logger.info(
1169+
logger.info(
11681170
"Config key '%s' is recognized as a compiler option, will not be used for current configuration.",
11691171
key.c_str());
11701172
continue;
11711173
}
11721174

11731175
if (optionMode == OptionMode::Both && !updatedConfig.isAvailable(key)) {
1174-
_logger.info(
1175-
"Config key '%s' is not enabled by the plugin, will not be used for current configuration.",
1176-
key.c_str());
1176+
logger.info("Config key '%s' is not enabled by the plugin, will not be used for current configuration.",
1177+
key.c_str());
11771178
continue;
11781179
}
11791180
}

0 commit comments

Comments
 (0)