diff options
author | Johannes Berg <johannes.berg@intel.com> | 2017-10-16 17:09:53 +0200 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2017-10-18 12:22:28 +0100 |
commit | 48044eb490be71c203e14dd89e8bae87209eab52 (patch) | |
tree | d3f0d375ba4de0aa909875f66eff5bbde5d00984 /COPYING | |
parent | 2de09681e4ce8b1caa79d2e4482b72d8ef41c550 (diff) | |
download | linux-48044eb490be71c203e14dd89e8bae87209eab52.tar.gz linux-48044eb490be71c203e14dd89e8bae87209eab52.tar.bz2 linux-48044eb490be71c203e14dd89e8bae87209eab52.zip |
netlink: fix netlink_ack() extack race
It seems that it's possible to toggle NETLINK_F_EXT_ACK
through setsockopt() while another thread/CPU is building
a message inside netlink_ack(), which could then trigger
the WARN_ON()s I added since if it goes from being turned
off to being turned on between allocating and filling the
message, the skb could end up being too small.
Avoid this whole situation by storing the value of this
flag in a separate variable and using that throughout the
function instead.
Fixes: 2d4bc93368f5 ("netlink: extended ACK reporting")
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'COPYING')
0 files changed, 0 insertions, 0 deletions