Skip to content

Commit cd140fa

Browse files
authored
Merge pull request #203 from duijf/fix-accept-resource-leak
socket: Make Accept() return after Close() is called
2 parents 3ce7832 + 07884b1 commit cd140fa

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

socket.go

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ package vz
77
*/
88
import "C"
99
import (
10+
"errors"
1011
"fmt"
1112
"net"
1213
"os"
@@ -198,6 +199,11 @@ func (v *VirtioSocketListener) Close() error {
198199
C.uint32_t(v.port),
199200
)
200201
v.handle.Delete()
202+
v.acceptch <- connResults{
203+
conn: nil,
204+
err: errors.New("accept failed: listener has been closed"),
205+
}
206+
close(v.acceptch)
201207
})
202208
return nil
203209
}

0 commit comments

Comments
 (0)