[ovs-dev] [PATCH] datapath: Fix a kernel crash when adding a 2nd VXLAN port with existing UDP port

Kyle Mestery kmestery at cisco.com
Wed Jan 9 19:47:41 UTC 2013


When creating a second VXLAN tunnel port with the same UDP
port as an existing VXLAN tunnel port, make sure to set
*vxport before returning. Fixes a kernel crash.

Reported-by: Todd Ruch <Todd.Ruch at wwtas.com>
Signed-off-by: Kyle Mestery <kmestery at cisco.com>
---
 datapath/vport-vxlan.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/datapath/vport-vxlan.c b/datapath/vport-vxlan.c
index a2cbeb9..f72b95f 100644
--- a/datapath/vport-vxlan.c
+++ b/datapath/vport-vxlan.c
@@ -267,6 +267,7 @@ static int vxlan_tunnel_setup(struct net *net, struct nlattr *options,
 	if (vxlan_port) {
 		vxlan_port->count++;
 		err = 0;
+		*vxport = vxlan_port;
 		goto out;
 	}
 
-- 
1.7.11.7




More information about the dev mailing list