diff options
author | Ghanan Gowripalan <ghanan@google.com> | 2021-05-14 16:32:28 -0700 |
---|---|---|
committer | gVisor bot <gvisor-bot@google.com> | 2021-05-14 16:37:13 -0700 |
commit | 820c77d5e78787cb7ffb197892d912c9f4c25a22 (patch) | |
tree | 6b19fe38deace2b303055b87f3287952fb489dba /pkg/sentry | |
parent | df2352796d1cbe5eea563d54380be60be18455bc (diff) |
Validate DAD configs when initializing DAD state
Make sure that the initial configurations used by the DAD state is
valid.
Before this change, an invalid DAD configuration (with a zero-valued
retransmit timer) was used so the DAD state would attempt to resolve
DAD immediately.
This lead to a deadlock in TestDADResolve as when DAD resolves, the
stack notifies the NDP dispatcher which would attempt to write to an
unbuffered channel while holding a lock. The test goroutine also
attempts to obtain a stack.Route (before receiving from the channel)
which ends up attempting to take the same lock.
Test: stack_test.TestDADResolve
PiperOrigin-RevId: 373888540
Diffstat (limited to 'pkg/sentry')
0 files changed, 0 insertions, 0 deletions