ProcessExecutor: do not use mutable TimerResults object from parent in forked process

Author: firewave

cli/processexecutor.cpp

@@ -386,12 +386,13 @@ unsigned int ProcessExecutor::check()
 #endif
                 close(pipes[0]);
 
-                // reset so we do not have the data which has already been transferred back
+                // create a separate result object so we do not get the results which have already been transferred back
+                std::unique_ptr<TimerResults> timerResults;
                 if (mTimerResults)
-                    mTimerResults->reset();
+                    timerResults.reset(new TimerResults);
 
                 PipeWriter pipewriter(pipes[1], mSettings.debugipc);
-                CppCheck fileChecker(mSettings, supprs, pipewriter, mTimerResults, false, mExecuteCommand);
+                CppCheck fileChecker(mSettings, supprs, pipewriter, timerResults.get(), false, mExecuteCommand);
                 unsigned int resultOfCheck = 0;
 
                 if (iFileSettings != mFileSettings.end()) {
@@ -403,7 +404,7 @@ unsigned int ProcessExecutor::check()
 
                 pipewriter.writeSuppr(supprs.nomsg);
 
-                pipewriter.writeTimer(mTimerResults);
+                pipewriter.writeTimer(timerResults.get());
 
                 pipewriter.writeEnd(std::to_string(resultOfCheck));
                 std::exit(EXIT_SUCCESS);