Re: [PATCH v6 8/9] iio: accel: mma8452: Fix use-after-free bug in error error path

[Esben Haabendal <esben@geanix.com>]

<sashiko-bot@kernel.org> writes:

> Thank you for your contribution! Sashiko AI review found 4 potential issue(s) to consider:
>
> Pre-existing issues:
> - [High] The device is left in ACTIVE mode on probe failure paths, potentially draining power and asserting interrupts indefinitely.
> - [High] The driver leaks the PM runtime usage count on multiple error paths when I2C reads/writes fail after acquiring a runtime PM reference.
> - [High] Time-of-check to time-of-use (TOCTOU) race condition during read-modify-write of I2C event configuration registers.

I will address these 3 in a new patch series after this one is merged.

> - [High] In `mma8452_runtime_suspend()`, if disabling the second regulator
> fails, the first regulator remains disabled but the PM state is left as
> RPM_ACTIVE.

This is fixed by patch from Sanjay Chitroda [1], which is currently in
iio testing branch.

> Sashiko AI review · https://sashiko.dev/#/patchset/20260825-mma8452-open-drain-v6-0-9b252804ee80@geanix.com?part=8

[1] https://lore.kernel.org/all/20260808-15-apr-pm-iio-mma8452-v4-temp-v5-1-d177e93ce3f8@gmail.com/

/Esben