diff options
Diffstat (limited to 'net/bridge/br_netlink.c')
| -rw-r--r-- | net/bridge/br_netlink.c | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/net/bridge/br_netlink.c b/net/bridge/br_netlink.c index fe0a79018ab2..4a6a378c84e3 100644 --- a/net/bridge/br_netlink.c +++ b/net/bridge/br_netlink.c | |||
| @@ -120,10 +120,11 @@ static int br_dump_ifinfo(struct sk_buff *skb, struct netlink_callback *cb) | |||
| 120 | idx = 0; | 120 | idx = 0; |
| 121 | for_each_netdev(net, dev) { | 121 | for_each_netdev(net, dev) { |
| 122 | /* not a bridge port */ | 122 | /* not a bridge port */ |
| 123 | if (dev->br_port == NULL || idx < cb->args[0]) | 123 | if (!br_port_exists(dev) || idx < cb->args[0]) |
| 124 | goto skip; | 124 | goto skip; |
| 125 | 125 | ||
| 126 | if (br_fill_ifinfo(skb, dev->br_port, NETLINK_CB(cb->skb).pid, | 126 | if (br_fill_ifinfo(skb, br_port_get(dev), |
| 127 | NETLINK_CB(cb->skb).pid, | ||
| 127 | cb->nlh->nlmsg_seq, RTM_NEWLINK, | 128 | cb->nlh->nlmsg_seq, RTM_NEWLINK, |
| 128 | NLM_F_MULTI) < 0) | 129 | NLM_F_MULTI) < 0) |
| 129 | break; | 130 | break; |
| @@ -168,9 +169,9 @@ static int br_rtm_setlink(struct sk_buff *skb, struct nlmsghdr *nlh, void *arg) | |||
| 168 | if (!dev) | 169 | if (!dev) |
| 169 | return -ENODEV; | 170 | return -ENODEV; |
| 170 | 171 | ||
| 171 | p = dev->br_port; | 172 | if (!br_port_exists(dev)) |
| 172 | if (!p) | ||
| 173 | return -EINVAL; | 173 | return -EINVAL; |
| 174 | p = br_port_get(dev); | ||
| 174 | 175 | ||
| 175 | /* if kernel STP is running, don't allow changes */ | 176 | /* if kernel STP is running, don't allow changes */ |
| 176 | if (p->br->stp_enabled == BR_KERNEL_STP) | 177 | if (p->br->stp_enabled == BR_KERNEL_STP) |
