diff options
| author | Vaishnav Achath <vaishnav.a@ti.com> | 2022-06-01 12:46:11 +0530 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2022-07-14 13:26:35 +0100 |
| commit | 73d5fe046270281a46344e06bf986c607632f7ea (patch) | |
| tree | f98437f861cedbd9347e0f16b356dcba9394bc92 /include | |
| parent | a77c46f2b4d48a81f36442ee0c2160baebf6c1a0 (diff) | |
| download | linux-stable-73d5fe046270281a46344e06bf986c607632f7ea.tar.gz linux-stable-73d5fe046270281a46344e06bf986c607632f7ea.tar.bz2 linux-stable-73d5fe046270281a46344e06bf986c607632f7ea.zip | |
spi: cadence-quadspi: Remove spi_master_put() in probe failure path
Currently the spi_master is allocated by devm_spi_alloc_master()
and devres core manages the deallocation, but in probe failure
path spi_master_put() is being handled manually which causes
"refcount underflow use-after-free" warning when probe failure happens
after allocating spi_master.
Trimmed backtrace during failure:
refcount_t: underflow; use-after-free.
pc : refcount_warn_saturate+0xf4/0x144
Call trace:
refcount_warn_saturate
kobject_put
put_device
devm_spi_release_controller
devres_release_all
This commit makes relevant changes to remove spi_master_put() from probe
failure path.
Fixes: 606e5d408184 ("spi: cadence-quadspi: Handle spi_unregister_master() in remove()")
Signed-off-by: Vaishnav Achath <vaishnav.a@ti.com>
Link: https://lore.kernel.org/r/20220601071611.11853-1-vaishnav.a@ti.com
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'include')
0 files changed, 0 insertions, 0 deletions