Merge pull request #62 from libp2p/fix/close-canceled-dial

don't leak connections when canceling dials

Merge pull request #62 from libp2p/fix/close-canceled-dial
don't leak connections when canceling dials
c0bf6756 · Steven Allen · GitHub · 09cb6bb5 · fef42503 · c0bf6756
Unverified Commit c0bf6756 authored Mar 09, 2018 by Steven Allen Committed by GitHub Mar 09, 2018
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 0 deletions

limiter.go limiter.go +3 -0

No files found.
--- a/limiter.go
+++ b/limiter.go
@@ -166,5 +166,8 @@ func (dl *dialLimiter) executeDial(j *dialJob) {
 	select {
 	case j.resp <- dialResult{Conn: con, Addr: j.addr, Err: err}:
 	case <-j.ctx.Done():
+		if err == nil {
+			con.Close()
+		}
 	}
 }