diff options
| author | Ian Gudger <igudger@google.com> | 2020-02-27 14:14:34 -0800 |
|---|---|---|
| committer | gVisor bot <gvisor-bot@google.com> | 2020-02-27 14:15:44 -0800 |
| commit | c6bdc6b05b4abfcb3c677013496c9a6b1d1365dd (patch) | |
| tree | 5ec78044281d27c9f47b8e04916ae0204d834a56 /pkg/p9/messages.go | |
| parent | d9ee81183fee2288681822a179ef230226fb8930 (diff) | |
Fix a race in TCP endpoint teardown and teardown the stack in tcp_test.
Call stack.Close on stacks when we are done with them in tcp_test. This avoids
leaking resources and reduces the test's flakiness when race/gotsan is enabled.
It also provides test coverage for the race also fixed in this change, which
can be reliably triggered with the stack.Close change (and without the other
changes) when race/gotsan is enabled.
The race was possible when calling Abort (via stack.Close) on an endpoint
processing a SYN segment as part of a passive connect.
Updates #1564
PiperOrigin-RevId: 297685432
Diffstat (limited to 'pkg/p9/messages.go')
0 files changed, 0 insertions, 0 deletions