* [PATCH] net:mac802154: add init of .sec and .sec_mtx for sdata in ieee802154_setup_sdata
@ 2021-04-04 4:57 Du Cheng
2021-04-05 0:33 ` Alexander Aring
0 siblings, 1 reply; 3+ messages in thread
From: Du Cheng @ 2021-04-04 4:57 UTC (permalink / raw)
To: Alexander Aring, Stefan Schmidt
Cc: linux-wpan, gregkh, skhan, Du Cheng, syzbot+cde43a581a8e5f317bc2
add mutex_init(&sdata->sec_mtx) and mac802154_llsec_init(&sdata->sec) for
NL802154_IFTYPE_MONITOR inside ieee802154_setup_sdata. As the unintiated
mutex and sec data structure were used in ieee802154_get_llsec_params
which would cause a kernel crash. BUG reported by syzkaller.
Reported-by: syzbot+cde43a581a8e5f317bc2@syzkaller.appspotmail.com
Signed-off-by: Du Cheng <ducheng2@gmail.com>
---
link to syzkaller bug:
https://syzkaller.appspot.com/bug?id=a9cc0c65e7bb15be7143107d4215ebc8ef047528
This patch has passed syzbot testing.
net/mac802154/iface.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/net/mac802154/iface.c b/net/mac802154/iface.c
index 1cf5ac09edcb..bec903e98db0 100644
--- a/net/mac802154/iface.c
+++ b/net/mac802154/iface.c
@@ -602,6 +602,9 @@ ieee802154_setup_sdata(struct ieee802154_sub_if_data *sdata,
sdata->dev->needs_free_netdev = true;
sdata->dev->netdev_ops = &mac802154_monitor_ops;
wpan_dev->promiscuous_mode = true;
+
+ mutex_init(&sdata->sec_mtx);
+ mac802154_llsec_init(&sdata->sec);
break;
default:
BUG();
--
2.30.2
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] net:mac802154: add init of .sec and .sec_mtx for sdata in ieee802154_setup_sdata
2021-04-04 4:57 [PATCH] net:mac802154: add init of .sec and .sec_mtx for sdata in ieee802154_setup_sdata Du Cheng
@ 2021-04-05 0:33 ` Alexander Aring
2021-04-05 3:50 ` Du Cheng
0 siblings, 1 reply; 3+ messages in thread
From: Alexander Aring @ 2021-04-05 0:33 UTC (permalink / raw)
To: Du Cheng
Cc: Stefan Schmidt, linux-wpan - ML, Greg Kroah-Hartman, skhan,
syzbot+cde43a581a8e5f317bc2
Hi,
On Sun, 4 Apr 2021 at 00:57, Du Cheng <ducheng2@gmail.com> wrote:
>
> add mutex_init(&sdata->sec_mtx) and mac802154_llsec_init(&sdata->sec) for
> NL802154_IFTYPE_MONITOR inside ieee802154_setup_sdata. As the unintiated
> mutex and sec data structure were used in ieee802154_get_llsec_params
> which would cause a kernel crash. BUG reported by syzkaller.
this patch will fix it in a way that the user is allowed to set llsec
parameters for monitors but they will never use them because it's not
supported. I resend my patch series which should forbid monitors to
set any llsec parameters for now and the user will be aware that this
feature isn't supported right now.
Thanks.
- Alex
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] net:mac802154: add init of .sec and .sec_mtx for sdata in ieee802154_setup_sdata
2021-04-05 0:33 ` Alexander Aring
@ 2021-04-05 3:50 ` Du Cheng
0 siblings, 0 replies; 3+ messages in thread
From: Du Cheng @ 2021-04-05 3:50 UTC (permalink / raw)
To: Alexander Aring
Cc: Stefan Schmidt, linux-wpan - ML, Greg Kroah-Hartman, skhan,
syzbot+cde43a581a8e5f317bc2
Le Sun, Apr 04, 2021 at 08:33:48PM -0400, Alexander Aring a écrit :
> Hi,
>
> On Sun, 4 Apr 2021 at 00:57, Du Cheng <ducheng2@gmail.com> wrote:
> >
> > add mutex_init(&sdata->sec_mtx) and mac802154_llsec_init(&sdata->sec) for
> > NL802154_IFTYPE_MONITOR inside ieee802154_setup_sdata. As the unintiated
> > mutex and sec data structure were used in ieee802154_get_llsec_params
> > which would cause a kernel crash. BUG reported by syzkaller.
>
> this patch will fix it in a way that the user is allowed to set llsec
> parameters for monitors but they will never use them because it's not
> supported. I resend my patch series which should forbid monitors to
> set any llsec parameters for now and the user will be aware that this
> feature isn't supported right now.
>
> Thanks.
>
> - Alex
Hi Alex,
Noted. Please CC me when you send your patch series.
Thanks!
Regards,
Du Cheng
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-04-05 3:50 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-04 4:57 [PATCH] net:mac802154: add init of .sec and .sec_mtx for sdata in ieee802154_setup_sdata Du Cheng
2021-04-05 0:33 ` Alexander Aring
2021-04-05 3:50 ` Du Cheng
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).