Skip to content

Commit 9446a7b

Browse files
authored
Coverity issues in files chromelloger.h and ze_metrices.h fixed. (#457)
* Coverity issues in files chromelloger.h and ze_metrices.h fixed. * defined tmp_string inside loop in chromelogger.h
1 parent c72c519 commit 9446a7b

File tree

2 files changed

+6
-4
lines changed

2 files changed

+6
-4
lines changed

tools/unitrace/src/chromelogger.h

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1206,13 +1206,14 @@ class ChromeLogger {
12061206
std::string dir = utils::GetEnv("UNITRACE_TraceOutputDir");
12071207
chrome_trace_file_name_ = (dir + '/' + chrome_trace_file_name_);
12081208
}
1209-
std::string tmpString;
1209+
12101210
if (this->CheckOption(TRACE_KERNEL_NAME_FILTER)) {
12111211
if (this->CheckOption(TRACE_K_NAME_FILTER_IN)) {
12121212
filter_in_ = true;
12131213
}
1214+
std::string tmpString;
12141215
tmpString = utils::GetEnv("UNITRACE_TraceKernelString");
1215-
filter_strings_set_.insert(tmpString);
1216+
filter_strings_set_.insert(std::move(tmpString));
12161217
} else if (this->CheckOption(TRACE_K_NAME_FILTER_FILE)) {
12171218
if (this->CheckOption(TRACE_K_NAME_FILTER_IN)) {
12181219
filter_in_ = true;
@@ -1221,8 +1222,9 @@ class ChromeLogger {
12211222
std::ifstream kfile(kernel_file, std::ios::in);
12221223
PTI_ASSERT(kfile.fail() != 1 && kfile.eof() != 1);
12231224
while (!kfile.eof()) {
1225+
std::string tmpString;
12241226
kfile >> tmpString;
1225-
filter_strings_set_.insert(tmpString);
1227+
filter_strings_set_.insert(std::move(tmpString));
12261228
}
12271229
} else {
12281230
filtering_on_ = false;

tools/unitrace/src/levelzero/ze_metrics.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1022,7 +1022,7 @@ class ZeMetricProfiler {
10221022
if (!units.empty()) {
10231023
name += "[" + units + "]";
10241024
}
1025-
name_list.push_back(name);
1025+
name_list.push_back(std::move(name));
10261026
}
10271027

10281028
return name_list;

0 commit comments

Comments
 (0)