Skip to content

Commit 10280b6

Browse files
authored
Merge pull request #2922 from tamird/avoid-type-assertion
Consistently connect to Unix sockets
2 parents ca778e3 + 777ab85 commit 10280b6

File tree

3 files changed

+12
-7
lines changed

3 files changed

+12
-7
lines changed

pkg/networks/usernet/client.go

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -139,8 +139,9 @@ func NewClient(endpointSock string, subnet net.IP) *Client {
139139
func create(sock string, subnet net.IP, base string) *Client {
140140
client := &http.Client{
141141
Transport: &http.Transport{
142-
DialContext: func(context.Context, string, string) (net.Conn, error) {
143-
return net.Dial("unix", sock)
142+
DialContext: func(ctx context.Context, _, _ string) (net.Conn, error) {
143+
var d net.Dialer
144+
return d.DialContext(ctx, "unix", sock)
144145
},
145146
},
146147
}

pkg/qemu/qemu_driver.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -417,9 +417,9 @@ func (a *qArgTemplateApplier) applyTemplate(qArg string) (string, error) {
417417
if !ok {
418418
return "", fmt.Errorf("non-string argument %+v", v)
419419
}
420-
addr := &net.UnixAddr{
421-
Net: "unix",
422-
Name: s,
420+
addr, err := net.ResolveUnixAddr("unix", s)
421+
if err != nil {
422+
return "", err
423423
}
424424
conn, err := net.DialUnix("unix", nil, addr)
425425
if err != nil {

pkg/vz/network_darwin.go

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,11 @@ import (
1818
)
1919

2020
func PassFDToUnix(unixSock string) (*os.File, error) {
21-
unixConn, err := net.Dial("unix", unixSock)
21+
unixAddr, err := net.ResolveUnixAddr("unix", unixSock)
22+
if err != nil {
23+
return nil, err
24+
}
25+
unixConn, err := net.DialUnix("unix", nil, unixAddr)
2226
if err != nil {
2327
return nil, err
2428
}
@@ -27,7 +31,7 @@ func PassFDToUnix(unixSock string) (*os.File, error) {
2731
if err != nil {
2832
return nil, err
2933
}
30-
err = fd.Put(unixConn.(*net.UnixConn), server)
34+
err = fd.Put(unixConn, server)
3135
if err != nil {
3236
return nil, err
3337
}

0 commit comments

Comments
 (0)