diff options
author | Jesse Gross <jesse@nicira.com> | 2012-01-31 14:22:25 -0800 |
---|---|---|
committer | Jesse Gross <jesse@nicira.com> | 2012-01-31 16:08:30 -0800 |
commit | c240693fc4d23786ab74bebfde45b44efc8ed39b (patch) | |
tree | 6d7b7cf12a0c9659b28d2e881c2b34b530b5a874 /datapath | |
parent | 5442edb9e0afb6fdc4cd81e5d1ebd3e0d4e6accf (diff) |
datapath: Don't dereference potentially NULL vport in patch ports.
When changing patch ports we need to update any peers pointing to us.
When destroying the port, this means pointing them to a NULL port.
However, the new namespaces code attempted to dereference this port
in order to get the namespace. The net is already passed in, so we
can just use that instead.
Bug #9477
Reported-by: Michael Hu <mhu@nicira.com>
Signed-off-by: Jesse Gross <jesse@nicira.com>
Acked-by: Ben Pfaff <blp@nicira.com>
Diffstat (limited to 'datapath')
-rw-r--r-- | datapath/vport-patch.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/datapath/vport-patch.c b/datapath/vport-patch.c index 0056792b..ea46a8dc 100644 --- a/datapath/vport-patch.c +++ b/datapath/vport-patch.c @@ -233,7 +233,7 @@ error: static void update_peers(struct net *net, const char *name, struct vport *vport) { - struct hlist_head *bucket = hash_bucket(ovs_dp_get_net(vport->dp), name); + struct hlist_head *bucket = hash_bucket(net, name); struct patch_vport *peer_vport; struct hlist_node *node; |