Hi Simon On Thu, Jan 17, 2013 at 10:02 PM, Simon Que <[email protected]> wrote: > 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]
Thanks! Signed-off-by: David Herrmann <[email protected]> > Signed-off-by: Simon Que <[email protected]> > --- > 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 eec3291..90124ff 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, > unsigned long flags; > ssize_t ret; > char buf[16]; > - __u16 size; > + __u16 size = 0; > > if (s == 0) > return -EINVAL; > -- > 1.7.8.6 > -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
