diff options
author | Simon Que <sque@chromium.org> | 2013-01-17 16:02:52 -0500 |
---|---|---|
committer | Jiri Kosina <jkosina@suse.cz> | 2013-01-18 04:59:24 -0500 |
commit | b77a989acca3fb4d09c2d8c3749499c663f5ab9f (patch) | |
tree | 17f7716e21657fd59523cbb528dfdcf4d42630a6 /drivers/hid | |
parent | aaca9cc0165f06a037a1e07770f6b379ff496288 (diff) |
HID: Fix uninitialized variable "size" in hid-wiimote-debug
This variable is initialized conditionally, based on whether a wiimote
call succeeds. However, the logic is not obvious to the compiler so it
throws a warning. Eliminate the warning by initializing "size" to 0.
The warning is:
files/drivers/hid/hid-wiimote-debug.c:69:18: warning: 'size' may be used
uninitialized in this function [-Wmaybe-uninitialized]
Signed-off-by: Simon Que <sque@chromium.org>
Signed-off-by: David Herrmann <dh.herrmann@googlemail.com>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
Diffstat (limited to 'drivers/hid')
-rw-r--r-- | drivers/hid/hid-wiimote-debug.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/hid/hid-wiimote-debug.c b/drivers/hid/hid-wiimote-debug.c index eec329197c16..90124ffaa2a5 100644 --- a/drivers/hid/hid-wiimote-debug.c +++ b/drivers/hid/hid-wiimote-debug.c | |||
@@ -31,7 +31,7 @@ static ssize_t wiidebug_eeprom_read(struct file *f, char __user *u, size_t s, | |||
31 | unsigned long flags; | 31 | unsigned long flags; |
32 | ssize_t ret; | 32 | ssize_t ret; |
33 | char buf[16]; | 33 | char buf[16]; |
34 | __u16 size; | 34 | __u16 size = 0; |
35 | 35 | ||
36 | if (s == 0) | 36 | if (s == 0) |
37 | return -EINVAL; | 37 | return -EINVAL; |