summaryrefslogtreecommitdiffstats
path: root/drivers/input
diff options
context:
space:
mode:
authorDan Carpenter <dan.carpenter@oracle.com>2014-12-02 21:55:53 -0800
committerDmitry Torokhov <dmitry.torokhov@gmail.com>2014-12-03 15:28:26 -0800
commit3b5005e9206af245033e9209eee193f3d7b2f636 (patch)
tree7abcd9b5828d8f9c4a9c52696bcc93910e2b5187 /drivers/input
parent939ffb1712890ee22146d2dfc24adbc7da6afa84 (diff)
downloadop-kernel-dev-3b5005e9206af245033e9209eee193f3d7b2f636.zip
op-kernel-dev-3b5005e9206af245033e9209eee193f3d7b2f636.tar.gz
Input: lm8323 - missing error check in lm8323_set_disable()
The missing error handling here is not especially harmful but static checkers complain that "i" can be used uninitialized. Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Diffstat (limited to 'drivers/input')
-rw-r--r--drivers/input/keyboard/lm8323.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/input/keyboard/lm8323.c b/drivers/input/keyboard/lm8323.c
index cb32e2b..21bea52 100644
--- a/drivers/input/keyboard/lm8323.c
+++ b/drivers/input/keyboard/lm8323.c
@@ -616,6 +616,8 @@ static ssize_t lm8323_set_disable(struct device *dev,
unsigned int i;
ret = kstrtouint(buf, 10, &i);
+ if (ret)
+ return ret;
mutex_lock(&lm->lock);
lm->kp_enabled = !i;
OpenPOWER on IntegriCloud