]> asedeno.scripts.mit.edu Git - linux.git/commit
iio: adc: ina2xx: avoid kthread_stop() with stale task_struct
authorAkinobu Mita <akinobu.mita@gmail.com>
Sun, 24 Jun 2018 15:05:21 +0000 (00:05 +0900)
committerJonathan Cameron <Jonathan.Cameron@huawei.com>
Sat, 7 Jul 2018 16:06:39 +0000 (17:06 +0100)
commit7d6cd21d82bacab2d1786fe5e989e4815b75d9a3
tree30353131c00095713ea411e1bbbb7b4b9c5f2b3e
parent5df362a6cf49ca8d2e69a2f68e605310e3c576cb
iio: adc: ina2xx: avoid kthread_stop() with stale task_struct

When the buffer is enabled for ina2xx driver, a dedicated kthread is
invoked to capture mesurement data.  When the buffer is disabled, the
kthread is stopped.

However if the kthread gets register access errors, it immediately exits
and when the malfunctional buffer is disabled, the stale task_struct
pointer is accessed as there is no kthread to be stopped.

A similar issue in the usbip driver is prevented by kthread_get_run and
kthread_stop_put helpers by increasing usage count of the task_struct.
This change applies the same solution.

Cc: Stefan BrĂ¼ns <stefan.bruens@rwth-aachen.de>
Cc: Jonathan Cameron <jic23@kernel.org>
Signed-off-by: Akinobu Mita <akinobu.mita@gmail.com>
Fixes: c43a102e67db ("iio: ina2xx: add support for TI INA2xx Power Monitors")
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
drivers/iio/adc/ina2xx-adc.c