linux-kernel - RE: [char-misc-next] mei: bus: fix device leak

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CY5PR11MB63668002881229DE234FE345ED78A@CY5PR11MB6366.namprd11.prod.outlook.com>
Date: Tue, 24 Jun 2025 09:39:52 +0000
From: "Usyskin, Alexander" <alexander.usyskin@...el.com>
To: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
CC: "Abliyev, Reuven" <reuven.abliyev@...el.com>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [char-misc-next] mei: bus: fix device leak

> Subject: Re: [char-misc-next] mei: bus: fix device leak
> 
> On Tue, Jun 24, 2025 at 10:15:21AM +0300, Alexander Usyskin wrote:
> > The bus rescan function creates bus devices for all clients.
> > The fixup routine is executed on all devices, unneeded
> > devices are removed and fully initialized once set
> > is_added flag to 1.
> >
> > If link to firmware is reset right after all devices are
> > initialized, but before fixup is executed, the rescan tries
> > to remove devices.
> > The is_added flag is not set and the mei_cl_bus_dev_destroy
> > returns prematurely.
> > Allow to clean up device when is_added flag is unset to
> > account for above scenario.
> >
> > Signed-off-by: Alexander Usyskin <alexander.usyskin@...el.com>
> 
> What commit id does this fix?
> 
> thanks,
> 
> greg k-h

This is the one that introduces this function:
commit 6009595a66e4 ("mei: bus: link client devices instead of host clients")

Will send v2 with Fixes tag.

- - 
Thanks,
Sasha