Merge pull request #705 from devbv/master

portmap: fix bug that new udp connection deletes all existing conntrack entries
containernetworking · Feb 23, 2022 · 26745d3 · 26745d3
2 parents 9f67faf + dca23ad
commit 26745d3
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/pkg/utils/conntrack.go b/pkg/utils/conntrack.go
@@ -62,8 +62,8 @@ func DeleteConntrackEntriesForDstIP(dstIP string, protocol uint8) error {
 // by the given destination port, protocol and IP family
 func DeleteConntrackEntriesForDstPort(port uint16, protocol uint8, family netlink.InetFamily) error {
 	filter := &netlink.ConntrackFilter{}
-	filter.AddPort(netlink.ConntrackOrigDstPort, port)
 	filter.AddProtocol(protocol)
+	filter.AddPort(netlink.ConntrackOrigDstPort, port)
 
 	_, err := netlink.ConntrackDeleteFilter(netlink.ConntrackTable, family, filter)
 	if err != nil {