diff options
| author | Stefan Schmidt <[email protected]> | 2018-04-12 14:28:49 +0200 |
|---|---|---|
| committer | Stefan Schmidt <[email protected]> | 2018-04-23 20:56:23 +0200 |
| commit | 7e0ffee1b2e7d26cc2147be0d9d5186823e923f3 (patch) | |
| tree | 67b80ad93acea2f525353b6e7fa0945a5e98950f | |
| parent | f9e628a6645ed4f42a15dfa3c5af45a916a228e4 (diff) | |
net: ieee802154: mcr20a: do not leak resources on error path
We already allocated the device and platform data at this
point. Instead of simply return from the probe function we
need to cleanup the resources first.
Signed-off-by: Stefan Schmidt <[email protected]>
Acked-by: Xue Liu <[email protected]>
| -rw-r--r-- | drivers/net/ieee802154/mcr20a.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/drivers/net/ieee802154/mcr20a.c b/drivers/net/ieee802154/mcr20a.c index 944470d69ba9..de0d7f28a181 100644 --- a/drivers/net/ieee802154/mcr20a.c +++ b/drivers/net/ieee802154/mcr20a.c @@ -1309,8 +1309,10 @@ mcr20a_probe(struct spi_device *spi) /* init buf */ lp->buf = devm_kzalloc(&spi->dev, SPI_COMMAND_BUFFER, GFP_KERNEL); - if (!lp->buf) - return -ENOMEM; + if (!lp->buf) { + ret = -ENOMEM; + goto free_dev; + } mcr20a_setup_tx_spi_messages(lp); mcr20a_setup_rx_spi_messages(lp); |