diff options
author | Souptick Joarder <jrdr.linux@gmail.com> | 2016-10-28 08:00:16 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2016-11-10 08:50:46 -0500 |
commit | 0a940b0d252bb8616bb36a254f6a3273f007c3f3 (patch) | |
tree | 10a1e7ab159e99d9a2d259e2c5b52208aeb86064 /drivers/tty | |
parent | 7f8b4ebeb2a1e7bf7af80926db012c9356f7a662 (diff) |
serial: ifx6x60: Free memory when probe fails
When spi_setup() fails it doesn't free ifx_dev and we have a memory
leak. Fix this by freeing ifx_dev before the return.
Signed-off-by: Souptick joarder <jrdr.linux@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/tty')
-rw-r--r-- | drivers/tty/serial/ifx6x60.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/tty/serial/ifx6x60.c b/drivers/tty/serial/ifx6x60.c index d386346248de..157883653256 100644 --- a/drivers/tty/serial/ifx6x60.c +++ b/drivers/tty/serial/ifx6x60.c | |||
@@ -1042,6 +1042,7 @@ static int ifx_spi_spi_probe(struct spi_device *spi) | |||
1042 | ret = spi_setup(spi); | 1042 | ret = spi_setup(spi); |
1043 | if (ret) { | 1043 | if (ret) { |
1044 | dev_err(&spi->dev, "SPI setup wasn't successful %d", ret); | 1044 | dev_err(&spi->dev, "SPI setup wasn't successful %d", ret); |
1045 | kfree(ifx_dev); | ||
1045 | return -ENODEV; | 1046 | return -ENODEV; |
1046 | } | 1047 | } |
1047 | 1048 | ||