diff options
| author | David Crawshaw <crawshaw@tailscale.com> | 2020-05-29 15:38:26 +1000 |
|---|---|---|
| committer | David Crawshaw <crawshaw@tailscale.com> | 2020-05-30 05:24:55 +1000 |
| commit | 057ff989fe129441ed067563a7d2b8bb3548b63e (patch) | |
| tree | 16d154fcbb4df3787182f7a527397f5af6575d9a /ipn | |
| parent | 004780b3124dd65700b7ab53802628ecc3c2517e (diff) | |
| download | tailscale-crawshaw/pinger2.tar.xz tailscale-crawshaw/pinger2.zip | |
wgengine: ensure pingers are gone before returning from Closecrawshaw/pinger2
We canceled the pingers in Close, but didn't wait around for their
goroutines to be cleaned up. This caused the ipn/e2e_test to catch
pingers in its resource leak check.
This commit introduces an object, but also simplifies the semantics
around the pinger's cancel functions. They no longer need to be called
while holding the mutex.
Signed-off-by: David Crawshaw <crawshaw@tailscale.com>
Diffstat (limited to 'ipn')
0 files changed, 0 insertions, 0 deletions