diff options
| author |   Dan Carpenter <dan.carpenter@oracle.com> | 2014-12-02 21:55:53 -0800 |
|---|---|---|
| committer |   Dmitry Torokhov <dmitry.torokhov@gmail.com> | 2014-12-03 15:28:26 -0800 |
| commit | 3b5005e9206af245033e9209eee193f3d7b2f636 (patch) | |
| tree | 7abcd9b5828d8f9c4a9c52696bcc93910e2b5187 | |
| parent | Input: initialize device counter variables with -1 (diff) | |
| download | wireguard-linux-3b5005e9206af245033e9209eee193f3d7b2f636.tar.xz wireguard-linux-3b5005e9206af245033e9209eee193f3d7b2f636.zip | |
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>
| -rw-r--r-- | drivers/input/keyboard/lm8323.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/input/keyboard/lm8323.c b/drivers/input/keyboard/lm8323.c index cb32e2b506b7..21bea52d4365 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; |