net/dns: close ctx when close dns directManager

Signed-off-by: Asutorufa <16442314+Asutorufa@users.noreply.github.com>
This commit is contained in:
Asutorufa 2024-03-29 17:57:20 +08:00 committed by Brad Fitzpatrick
parent 1d2af801fa
commit e20ce7bf0c
2 changed files with 7 additions and 1 deletions

View File

@ -403,6 +403,8 @@ func (m *directManager) GetBaseConfig() (OSConfig, error) {
}
func (m *directManager) Close() error {
m.ctxClose()
// We used to keep a file for the tailscale config and symlinked
// to it, but then we stopped because /etc/resolv.conf being a
// symlink to surprising places breaks snaps and other sandboxing

View File

@ -4,6 +4,7 @@
package dns
import (
"context"
"errors"
"fmt"
"io/fs"
@ -78,7 +79,10 @@ func testDirect(t *testing.T, fs wholeFileFS) {
}
}
m := directManager{logf: t.Logf, fs: fs}
ctx, cancel := context.WithCancel(context.Background())
defer cancel()
m := directManager{logf: t.Logf, fs: fs, ctx: ctx, ctxClose: cancel}
if err := m.SetDNS(OSConfig{
Nameservers: []netip.Addr{netip.MustParseAddr("8.8.8.8"), netip.MustParseAddr("8.8.4.4")},
SearchDomains: []dnsname.FQDN{"ts.net.", "ts-dns.test."},