diff options
author | Benjamin Poirier <bpoirier@suse.de> | 2015-01-14 10:52:35 +0300 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2015-01-15 00:34:41 +0300 |
commit | 4ccce02eb31b847dd6bf8486f037ba1db39403c5 (patch) | |
tree | 7e5ea9ced71bf9e2153c37380b595bdc7f520538 | |
parent | d92cfdbbeaefb467de4fac98679fa6c33461d250 (diff) | |
download | linux-4ccce02eb31b847dd6bf8486f037ba1db39403c5.tar.xz |
netdevice: Add missing parentheses in macro
For example, one could conceivably call
for_each_netdev_in_bond_rcu(condition ? bond1 : bond2, slave)
and get an unexpected result.
Signed-off-by: Benjamin Poirier <bpoirier@suse.de>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | include/linux/netdevice.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/linux/netdevice.h b/include/linux/netdevice.h index 7f794db8e486..52fd8e8694cf 100644 --- a/include/linux/netdevice.h +++ b/include/linux/netdevice.h @@ -2085,7 +2085,7 @@ extern rwlock_t dev_base_lock; /* Device list lock */ list_for_each_entry_continue_rcu(d, &(net)->dev_base_head, dev_list) #define for_each_netdev_in_bond_rcu(bond, slave) \ for_each_netdev_rcu(&init_net, slave) \ - if (netdev_master_upper_dev_get_rcu(slave) == bond) + if (netdev_master_upper_dev_get_rcu(slave) == (bond)) #define net_device_entry(lh) list_entry(lh, struct net_device, dev_list) static inline struct net_device *next_net_device(struct net_device *dev) |