Re: [PATCH 1/2] regulator: mt6370: Fix potential UAF issue
From: ChiYuan Huang
Date: Thu Dec 01 2022 - 22:38:53 EST
Mark Brown <broonie@xxxxxxxxxx> 於 2022年12月1日 週四 晚上7:43寫道:
>
> On Wed, Nov 30, 2022 at 04:37:42PM +0800, cy_huang wrote:
>
> > The original code uses i2c dev as the parent in order to reuse
> > the 'regulator_of_get_init_data'. But this will cause regulation
> > constraint devres attached to i2c dev, not the mfd cell platform
> > device.
>
> This is a general issue which will affect a lot of MFDs, we would be
> better to fix this by changing the API to provide a device to be used
> for the devres allocations separately to the one used for looking up the
> DT.
Not to affect too much, the better way may change the 'regulator_register' API.
Append it as regulator_register(dev, .....
This could separate device object with devres allocation and DT lookup.