Skip to content

Conversation

@solomatov
Copy link
Contributor

@solomatov solomatov commented Apr 27, 2021

This fixes a potential ext. host leak

The scenario is the following:

  • ext host starts
  • some listener, say timer, is started during the startup process
  • in 60s we don't receive IPC socket, and reject
  • rejection goes up to the main function
  • we have a timer, and it keeps the process alive
  • as a result we have a process leak

@solomatov
Copy link
Contributor Author

cc @alexdima

@alexdima
Copy link
Member

Thank you!

@alexdima alexdima added this to the April 2021 milestone Apr 28, 2021
@alexdima alexdima merged commit 5728f84 into microsoft:main Apr 28, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Jun 12, 2021
@solomatov solomatov deleted the ext-host-hang branch July 23, 2021 20:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants