Thread (2 messages) 2 messages, 2 authors, 2016-09-15

[PATCH -next] [media] Input: fix missing unlock on error in rmi_f54_request_report()

From: Wei Yongjun <hidden>
Date: 2016-09-13 14:24:57
Subsystem: input (keyboard, mouse, joystick, touchscreen) drivers, the rest · Maintainers: Dmitry Torokhov, Linus Torvalds

From: Wei Yongjun <redacted>

Add the missing unlock before return from function
rmi_f54_request_report() in the error handling case.

Signed-off-by: Wei Yongjun <redacted>
---
 drivers/input/rmi4/rmi_f54.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/input/rmi4/rmi_f54.c b/drivers/input/rmi4/rmi_f54.c
index bd86d3d..99a8836 100644
--- a/drivers/input/rmi4/rmi_f54.c
+++ b/drivers/input/rmi4/rmi_f54.c
@@ -198,8 +198,10 @@ static int rmi_f54_request_report(struct rmi_function *fn, u8 report_type)
 	mutex_lock(&f54->data_mutex);
 
 	error = rmi_write(rmi_dev, fn->fd.command_base_addr, F54_GET_REPORT);
-	if (error < 0)
+	if (error < 0) {
+		mutex_unlock(&f54->data_mutex);
 		return error;
+	}
 
 	init_completion(&f54->cmd_done);
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help