mirror of
https://github.com/nxp-imx/linux-imx.git
synced 2025-09-02 18:06:13 +02:00
79afd0939b
The blamed commit tried to simplify how the deallocations are done but,
in the process, introduced a double-free on the mc_dev variable.
In case the MC device is a DPRC, a new mc_bus is allocated and the
mc_dev variable is just a reference to one of its fields. In this
circumstance, on the error path only the mc_bus should be freed.
This commit introduces back the following checkpatch warning which is a
false-positive.
WARNING: kfree(NULL) is safe and this check is probably not required
+ if (mc_bus)
+ kfree(mc_bus);
Fixes:
|
||
|---|---|---|
| .. | ||
| dpbp.c | ||
| dpcon.c | ||
| dpmcp.c | ||
| dprc-driver.c | ||
| dprc.c | ||
| fsl-mc-allocator.c | ||
| fsl-mc-bus.c | ||
| fsl-mc-msi.c | ||
| fsl-mc-private.h | ||
| fsl-mc-uapi.c | ||
| Kconfig | ||
| Makefile | ||
| mc-io.c | ||
| mc-sys.c | ||
| obj-api.c | ||