Re: [PATCH 6.1.y] Fix memory leak in io_wq_create() on success path

xk c <[email protected]>
Newsgroups org.kernel.vger.stable,org.kernel.vger.linux-kernel
Message-ID <CAGeKw7JFxZ6QBg_p6ma2NjPMBe5utoZ715z3763d0qH=uLehYA@mail.gmail.com>
Greg KH <[email protected]> 于2026年8月6日周四 22:33写道:
>
> On Thu, Aug 06, 2026 at 10:23:14PM +0800, Chen Xiaokun wrote:
> > The commit 657ca82526d0 introduced a temporary allowed_mask cpumask in
> > io_wq_create(), which was freed only on the error path.  This caused a
> > memory leak of cpumask_size() bytes on every successful io_wq_create().
> >
> > Fixes: 657ca82526d0 ("io_uring/io-wq: inherit cpuset of cgroup in io worker")
> > Signed-off-by: Chen Xiaokun <[email protected]>
> > ---
> >  io_uring/io-wq.c | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/io_uring/io-wq.c b/io_uring/io-wq.c
> > index 66fdd69fd6cb..34a1a9067cc2 100644
> > --- a/io_uring/io-wq.c
> > +++ b/io_uring/io-wq.c
> > @@ -1218,6 +1218,7 @@ struct io_wq *io_wq_create(unsigned bounded, struct io_wq_data *data)
> >       wq->task = get_task_struct(data->task);
> >       atomic_set(&wq->worker_refs, 1);
> >       init_completion(&wq->worker_done);
> > +     free_cpumask_var(allowed_mask);
> >       return wq;
> >  err:
> >       io_wq_put_hash(data->hash);
> > --
> > 2.55.0
> >
> >
>
> Why is this a stable-only patch?  Was the backport wrong?  Or is this
> also in Linus's tree?
>

Linus's tree has not this issue.

> thanks,
>
> greg k-h

thanks,
Chen
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.