HID: rmi: Check that the RMI_STARTED bit is set before unregistering the RMI transpor...
authorAndrew Duggan <aduggan@synaptics.com>
Wed, 23 Oct 2019 01:24:03 +0000 (01:24 +0000)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 4 Jan 2020 18:13:00 +0000 (19:13 +0100)
commitf573f40410761c66c43cc73989fd27b884228c8b
tree20bf5567869a9b393a4ccf5183b30f237295cb9a
parent51ade3d12e7451feda2f43c063a98e3e841cee03
HID: rmi: Check that the RMI_STARTED bit is set before unregistering the RMI transport device

[ Upstream commit 8725aa4fa7ded30211ebd28bb1c9bae806eb3841 ]

In the event that the RMI device is unreachable, the calls to rmi_set_mode() or
rmi_set_page() will fail before registering the RMI transport device. When the
device is removed, rmi_remove() will call rmi_unregister_transport_device()
which will attempt to access the rmi_dev pointer which was not set.
This patch adds a check of the RMI_STARTED bit before calling
rmi_unregister_transport_device().  The RMI_STARTED bit is only set
after rmi_register_transport_device() completes successfully.

The kernel oops was reported in this message:
https://www.spinics.net/lists/linux-input/msg58433.html

[jkosina@suse.cz: reworded changelog as agreed with Andrew]
Signed-off-by: Andrew Duggan <aduggan@synaptics.com>
Reported-by: Federico Cerutti <federico@ceres-c.it>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/hid/hid-rmi.c