From a90661654d14e77b6442c4a34ced900e2cd9953e Mon Sep 17 00:00:00 2001 From: Bhasker Hariharan Date: Tue, 26 Jan 2021 12:07:16 -0800 Subject: Fix couple of potential route leaks. connect() can be invoked multiple times on UDP/RAW sockets and in such a case we should release the cached route from the previous connect. Fixes #5359 PiperOrigin-RevId: 353919891 --- pkg/tcpip/transport/udp/endpoint.go | 5 +++++ 1 file changed, 5 insertions(+) (limited to 'pkg/tcpip/transport/udp/endpoint.go') diff --git a/pkg/tcpip/transport/udp/endpoint.go b/pkg/tcpip/transport/udp/endpoint.go index ea8a7017e..a3e513ff7 100644 --- a/pkg/tcpip/transport/udp/endpoint.go +++ b/pkg/tcpip/transport/udp/endpoint.go @@ -1009,6 +1009,11 @@ func (e *endpoint) Connect(addr tcpip.FullAddress) *tcpip.Error { e.ID = id e.boundBindToDevice = btd + if e.route != nil { + // If the endpoint was already connected then make sure we release the + // previous route. + e.route.Release() + } e.route = r e.dstPort = addr.Port e.RegisterNICID = nicID -- cgit v1.2.3