Commit c7fbe9e7 authored by Dan Carpenter's avatar Dan Carpenter Committed by Luis Henriques

iio: inkern: fix a NULL dereference on error

commit d81dac3c upstream.

In twl4030_bci_probe() there are some failure paths where we call
iio_channel_release() with a NULL pointer.  (Apparently, that driver can
opperate without a valid channel pointer).  Let's fix it by adding a
NULL check in iio_channel_release().

Fixes: 2202e1fc ('drivers: power: twl4030_charger: fix link problems when building as module')
Signed-off-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: default avatarJonathan Cameron <jic23@kernel.org>
Signed-off-by: default avatarLuis Henriques <luis.henriques@canonical.com>
parent f5892ce3
...@@ -325,6 +325,8 @@ EXPORT_SYMBOL_GPL(iio_channel_get); ...@@ -325,6 +325,8 @@ EXPORT_SYMBOL_GPL(iio_channel_get);
void iio_channel_release(struct iio_channel *channel) void iio_channel_release(struct iio_channel *channel)
{ {
if (!channel)
return;
iio_device_put(channel->indio_dev); iio_device_put(channel->indio_dev);
kfree(channel); kfree(channel);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment