summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAlexander Aring <aahringo@redhat.com>2021-04-04 20:30:53 -0400
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2021-04-16 11:59:11 +0200
commit8f1696d0f979131a314d0b6b8822c4a466ea122d (patch)
tree1ade900c79b5b9c4197dfcaf5156a9646f6d2e24
parentc86de71fddc294cea39ec9dc7d1dc02ebb0b10e9 (diff)
downloadlinux-stable-8f1696d0f979131a314d0b6b8822c4a466ea122d.tar.gz
linux-stable-8f1696d0f979131a314d0b6b8822c4a466ea122d.tar.bz2
linux-stable-8f1696d0f979131a314d0b6b8822c4a466ea122d.zip
net: ieee802154: forbid monitor for del llsec seclevel
commit 9dde130937e95b72adfae64ab21d6e7e707e2dac upstream. This patch forbids to del llsec seclevel for monitor interfaces which we don't support yet. Otherwise we will access llsec mib which isn't initialized for monitors. Reported-by: syzbot+fbf4fc11a819824e027b@syzkaller.appspotmail.com Signed-off-by: Alexander Aring <aahringo@redhat.com> Link: https://lore.kernel.org/r/20210405003054.256017-15-aahringo@redhat.com Signed-off-by: Stefan Schmidt <stefan@datenfreihafen.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r--net/ieee802154/nl802154.c3
1 files changed, 3 insertions, 0 deletions
diff --git a/net/ieee802154/nl802154.c b/net/ieee802154/nl802154.c
index c1412cd87fe7..0a88bb254a93 100644
--- a/net/ieee802154/nl802154.c
+++ b/net/ieee802154/nl802154.c
@@ -2137,6 +2137,9 @@ static int nl802154_del_llsec_seclevel(struct sk_buff *skb,
struct wpan_dev *wpan_dev = dev->ieee802154_ptr;
struct ieee802154_llsec_seclevel sl;
+ if (wpan_dev->iftype == NL802154_IFTYPE_MONITOR)
+ return -EOPNOTSUPP;
+
if (!info->attrs[NL802154_ATTR_SEC_LEVEL] ||
llsec_parse_seclevel(info->attrs[NL802154_ATTR_SEC_LEVEL],
&sl) < 0)