Commit ff248202 authored by Vladislav Vaintroub's avatar Vladislav Vaintroub

Fix process handle leak in buildbot. GenerateConsoleCtrlEvent sent to...

Fix process  handle leak in buildbot. GenerateConsoleCtrlEvent sent to non-existing process will add a process handle to this non-existing process to console host process conhost.exe
parent 61d3621e
...@@ -357,14 +357,14 @@ int main(int argc, const char** argv ) ...@@ -357,14 +357,14 @@ int main(int argc, const char** argv )
CloseHandle(job_handle); CloseHandle(job_handle);
message("Job terminated and closed"); message("Job terminated and closed");
if (!jobobject_assigned)
{
GenerateConsoleCtrlEvent(CTRL_BREAK_EVENT, process_info.dwProcessId);
TerminateProcess(process_info.hProcess, 202);
}
if (wait_res != WAIT_OBJECT_0 + CHILD) if (wait_res != WAIT_OBJECT_0 + CHILD)
{ {
if (!jobobject_assigned)
{
TerminateProcess(process_info.hProcess, 202);
}
/* The child has not yet returned, wait for it */ /* The child has not yet returned, wait for it */
message("waiting for child to exit"); message("waiting for child to exit");
if ((wait_res= WaitForSingleObject(wait_handles[CHILD], INFINITE)) if ((wait_res= WaitForSingleObject(wait_handles[CHILD], INFINITE))
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment