]> git.baikalelectronics.ru Git - kernel.git/commit
rtnetlink: check DO_SETLINK_NOTIFY correctly in do_setlink
authorXin Long <lucien.xin@gmail.com>
Sun, 15 Oct 2017 10:13:45 +0000 (18:13 +0800)
committerDavid S. Miller <davem@davemloft.net>
Mon, 16 Oct 2017 19:48:45 +0000 (20:48 +0100)
commit470730c932493529eeb622c9c37a063134c215dd
treee54399965691ebc190697fbfb461b07426bef3c2
parentb0aa14feb4b128fecefafe2ef5288f49c630e953
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: 8d53569e8eee ("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>
net/core/rtnetlink.c