diff options
| author | Xin Long <lucien.xin@gmail.com> | 2017-10-15 12:13:45 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2017-10-16 21:48:45 +0200 |
| commit | 64ff90cc2e6f42596d7a0c37e41dc95292bb63b1 (patch) | |
| tree | e54399965691ebc190697fbfb461b07426bef3c2 /net/batman-adv | |
| parent | rtnetlink: bring NETDEV_CHANGEUPPER event process back in rtnetlink_event (diff) | |
| download | linux-64ff90cc2e6f42596d7a0c37e41dc95292bb63b1.tar.xz linux-64ff90cc2e6f42596d7a0c37e41dc95292bb63b1.zip | |
rtnetlink: check DO_SETLINK_NOTIFY correctly in do_setlink
The check 'status & DO_SETLINK_NOTIFY' in do_setlink doesn't really
work after status & DO_SETLINK_MODIFIED, as:
DO_SETLINK_MODIFIED 0x1
DO_SETLINK_NOTIFY 0x3
Considering that notifications are suppposed to be sent only when
status have the flag DO_SETLINK_NOTIFY, the right check would be:
(status & DO_SETLINK_NOTIFY) == DO_SETLINK_NOTIFY
This would avoid lots of duplicated notifications when setting some
properties of a link.
Fixes: ba9989069f4e ("rtnl/do_setlink(): notify when a netdev is modified")
Signed-off-by: Xin Long <lucien.xin@gmail.com>
Acked-by: David Ahern <dsahern@gmail.com>
Acked-by: Nicolas Dichtel <nicolas.dichtel@6wind.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/batman-adv')
0 files changed, 0 insertions, 0 deletions