]> git.proxmox.com Git - mirror_ubuntu-eoan-kernel.git/commitdiff
Input: atmel_mxt_ts - fix invalid return from mxt_get_bootloader_version
authorNick Dyer <nick.dyer@itdev.co.uk>
Sun, 8 Jun 2014 06:17:26 +0000 (23:17 -0700)
committerDmitry Torokhov <dmitry.torokhov@gmail.com>
Sun, 8 Jun 2014 06:20:24 +0000 (23:20 -0700)
The patch e57a66aa8534: "Input: atmel_mxt_ts - read and report
bootloader version" from May 18, 2014, leads to the following static
checker warning:

drivers/input/touchscreen/atmel_mxt_ts.c:437 mxt_get_bootloader_version()
warn: signedness bug returning '(-5)'

drivers/input/touchscreen/atmel_mxt_ts.c
   429  static u8 mxt_get_bootloader_version(struct mxt_data *data, u8 val)
   430  {
   431          struct device *dev = &data->client->dev;
   432          u8 buf[3];
   433
   434          if (val & MXT_BOOT_EXTENDED_ID) {
   435                  if (mxt_bootloader_read(data, &buf[0], 3) != 0) {
   436                          dev_err(dev, "%s: i2c failure\n", __func__);
   437                          return -EIO;
                                       ^^^^
This gets truncated into a number from 0-255 and anyway the caller
doesn't check for errors.

(reported by Dan Carpenter)

Signed-off-by: Nick Dyer <nick.dyer@itdev.co.uk>
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
drivers/input/touchscreen/atmel_mxt_ts.c

index 97495c7edae8e952a5cc6a20229e9f35f392e38a..6e0b4a2120d3ece91d59c9fc23e50089dc28fa82 100644 (file)
@@ -434,7 +434,7 @@ static u8 mxt_get_bootloader_version(struct mxt_data *data, u8 val)
        if (val & MXT_BOOT_EXTENDED_ID) {
                if (mxt_bootloader_read(data, &buf[0], 3) != 0) {
                        dev_err(dev, "%s: i2c failure\n", __func__);
-                       return -EIO;
+                       return val;
                }
 
                dev_dbg(dev, "Bootloader ID:%d Version:%d\n", buf[1], buf[2]);