[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <CANn89iL+YJsz-PCe-suCCnKVkuxUVMy9R0mJFGW9o=3Vi8A1gQ@mail.gmail.com>
Date: Wed, 11 Jun 2025 05:00:55 -0700
From: Eric Dumazet <edumazet@...gle.com>
To: Hyunwoo Kim <imv4bel@...il.com>
Cc: Vladimir Oltean <vladimir.oltean@....com>, vinicius.gomes@...el.com, jhs@...atatu.com,
xiyou.wangcong@...il.com, jiri@...nulli.us, davem@...emloft.net,
kuba@...nel.org, pabeni@...hat.com, horms@...nel.org, netdev@...r.kernel.org,
v4bel@...ori.io
Subject: Re: [PATCH] net/sched: fix use-after-free in taprio_dev_notifier
On Wed, Jun 11, 2025 at 4:38 AM Hyunwoo Kim <imv4bel@...il.com> wrote:
>
> On Wed, Jun 11, 2025 at 04:28:44AM -0700, Eric Dumazet wrote:
> > On Wed, Jun 11, 2025 at 4:24 AM Hyunwoo Kim <imv4bel@...il.com> wrote:
> > >
> > > On Wed, Jun 11, 2025 at 04:01:50AM -0700, Eric Dumazet wrote:
> > > > On Wed, Jun 11, 2025 at 3:03 AM Hyunwoo Kim <imv4bel@...il.com> wrote:
> > > > >
> > > > > Since taprio’s taprio_dev_notifier() isn’t protected by an
> > > > > RCU read-side critical section, a race with advance_sched()
> > > > > can lead to a use-after-free.
> > > > >
> > > > > Adding rcu_read_lock() inside taprio_dev_notifier() prevents this.
> > > > >
> > > > > Signed-off-by: Hyunwoo Kim <imv4bel@...il.com>
> > > >
> > > > Looks good to me, but we need a Fixes: tag and/or a CC: stable@ o make
> > > > sure this patch reaches appropriate stable trees.
> > >
> > > Understood. I will submit the v2 patch after adding the tags.
> >
> > Thanks, please wait ~24 hours (as described in
> > Documentation/process/maintainer-netdev.rst )
>
> Okay, I will submit the v2 patch tomorrow. Thank you for reviewing it!
>
> >
> > >
> > > >
> > > > Also please CC the author of the patch.
> > >
> > > Does “CC” here refer to a patch tag, or to the email’s cc? And by
> > > “patch author” you mean the author of the patch
> > > fed87cc6718ad5f80aa739fee3c5979a8b09d3a6, right?
> >
> > Exactly. Blamed patch author.
>
> To avoid confusion: when you say “CC the patch author,” do you mean
> adding the author to the CC tag in the v2 patch, or simply including
> them in the email’s CC field?
Whatever works for you. We only prefer to let them review the fix,
maybe they know
something we do not know ;)
Powered by blists - more mailing lists