Merge pull request #402 from aiburegit/triage

guelfey · web-flow · commit 450a4fbbb1d0 · 2025-11-14T11:31:42.000+01:00
fix: added socket closing to prevent resource leak
diff --git a/transport_nonce_tcp.go b/transport_nonce_tcp.go
@@ -30,10 +30,12 @@ func newNonceTcpTransport(keys string) (transport, error) {
 	}
 	b, err := os.ReadFile(noncefile)
 	if err != nil {
+		socket.Close()
 		return nil, err
 	}
 	_, err = socket.Write(b)
 	if err != nil {
+		socket.Close()
 		return nil, err
 	}
 	return NewConn(socket)

Original file line number	Diff line number	Diff line change
`@@ -30,10 +30,12 @@ func newNonceTcpTransport(keys string) (transport, error) {`
`30`	`30`	`}`
`31`	`31`	`b, err := os.ReadFile(noncefile)`
`32`	`32`	`if err != nil {`
	`33`	`+ socket.Close()`
`33`	`34`	`return nil, err`
`34`	`35`	`}`
`35`	`36`	`_, err = socket.Write(b)`
`36`	`37`	`if err != nil {`
	`38`	`+ socket.Close()`
`37`	`39`	`return nil, err`
`38`	`40`	`}`
`39`	`41`	`return NewConn(socket)`