diff options
| author | Benjamin Poirier <bpoirier@suse.de> | 2015-01-14 08:52:35 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2015-01-14 22:34:41 +0100 |
| commit | 4ccce02eb31b847dd6bf8486f037ba1db39403c5 (patch) | |
| tree | 7e5ea9ced71bf9e2153c37380b595bdc7f520538 /include | |
| parent | bridge: only provide proxy ARP when CONFIG_INET is enabled (diff) | |
| download | linux-4ccce02eb31b847dd6bf8486f037ba1db39403c5.tar.xz linux-4ccce02eb31b847dd6bf8486f037ba1db39403c5.zip | |
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>
Diffstat (limited to 'include')
| -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) |