From: "Nicolas Escande" <nico.escande@gmail.com>
To: "Deepanshu Kartikey" <kartikey406@gmail.com>,
<johannes@sipsolutions.net>, <nico.escande@gmail.com>
Cc: <linux-wireless@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
<syzbot+f5752cd6b94fe38be666@syzkaller.appspotmail.com>
Subject: Re: [PATCH v2] wifi: mac80211: mesh: free pending CSA settings on interface stop
Date: Tue, 25 Aug 2026 22:53:26 +0200 [thread overview]
Message-ID: <DKYBN6NP5C9O.2G7X3PSMOKQ7F@gmail.com> (raw)
In-Reply-To: <20260825135717.36369-1-kartikey406@gmail.com>
On Tue Aug 25, 2026 at 3:57 PM CEST, Deepanshu Kartikey wrote:
> ifmsh->csa is allocated in ieee80211_mesh_csa_beacon() and normally
> freed in ieee80211_mesh_finish_csa() once the channel switch
> completes. If the mesh interface is stopped while a channel switch
> is still in progress, ifmsh->csa is never freed, leaking the
> mesh_csa_settings object.
>
> Free it in ieee80211_stop_mesh(), the same way ifmsh->beacon is
> already handled there.
>
> Reported-by: syzbot+f5752cd6b94fe38be666@syzkaller.appspotmail.com
> Closes: https://syzkaller.appspot.com/bug?extid=f5752cd6b94fe38be666
> Link: https://lore.kernel.org/all/20260824054611.29728-1-kartikey406@gmail.com/T/ [v1]
> Signed-off-by: Deepanshu Kartikey <kartikey406@gmail.com>
> ---
> v2: drop unnecessary NULL check around csa, kfree_rcu() already
> handles a NULL pointer (Nicolas Escande)
> ---
> net/mac80211/mesh.c | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/net/mac80211/mesh.c b/net/mac80211/mesh.c
> index d4507e4e6ec1..48015dcb7add 100644
> --- a/net/mac80211/mesh.c
> +++ b/net/mac80211/mesh.c
> @@ -1201,6 +1201,7 @@ void ieee80211_stop_mesh(struct ieee80211_sub_if_data *sdata)
> struct ieee80211_local *local = sdata->local;
> struct ieee80211_if_mesh *ifmsh = &sdata->u.mesh;
> struct beacon_data *bcn;
> + struct mesh_csa_settings *csa;
>
> netif_carrier_off(sdata->dev);
>
> @@ -1222,6 +1223,11 @@ void ieee80211_stop_mesh(struct ieee80211_sub_if_data *sdata)
> RCU_INIT_POINTER(ifmsh->beacon, NULL);
> kfree_rcu(bcn, rcu_head);
>
> + /* free any pending, unfinished channel switch */
> + csa = sdata_dereference(ifmsh->csa, sdata);
> + RCU_INIT_POINTER(ifmsh->csa, NULL);
> + kfree_rcu(csa, rcu_head);
> +
> /* free all potentially still buffered group-addressed frames */
> local->total_ps_buffered -= skb_queue_len(&ifmsh->ps.bc_buf);
> skb_queue_purge(&ifmsh->ps.bc_buf);
Reviewed-by: Nicolas Escande <nico.escande@gmail.com>
prev parent reply other threads:[~2026-08-25 20:53 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-08-25 13:57 Deepanshu Kartikey
2026-08-25 20:53 ` Nicolas Escande [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=DKYBN6NP5C9O.2G7X3PSMOKQ7F@gmail.com \
--to=nico.escande@gmail.com \
--cc=johannes@sipsolutions.net \
--cc=kartikey406@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-wireless@vger.kernel.org \
--cc=syzbot+f5752cd6b94fe38be666@syzkaller.appspotmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®