openvswitch: remove BUG_ON from get_dpdev

lrq-max · davem330 · commit 04087d9a89be · 2018-11-08T17:14:59.000-08:00
if local is NULL pointer, and the following access of local's
dev will trigger panic, which is same as BUG_ON

Signed-off-by: Li RongQing &lt;lirongqing@baidu.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
diff --git a/net/openvswitch/vport-netdev.c b/net/openvswitch/vport-netdev.c
@@ -84,7 +84,6 @@ static struct net_device *get_dpdev(const struct datapath *dp)
 	struct vport *local;
 
 	local = ovs_vport_ovsl(dp, OVSP_LOCAL);
-	BUG_ON(!local);
 	return local->dev;
 }
 

Original file line number	Diff line number	Diff line change
`@@ -84,7 +84,6 @@ static struct net_device get_dpdev(const struct datapath dp)`
`84`	`84`	`struct vport *local;`
`85`	`85`
`86`	`86`	`local = ovs_vport_ovsl(dp, OVSP_LOCAL);`
`87`		`- BUG_ON(!local);`
`88`	`87`	`return local->dev;`
`89`	`88`	`}`
`90`	`89`