diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2012-08-09 08:57:30 +0200 |
|---|---|---|
| committer | John W. Linville <linville@tuxdriver.com> | 2012-08-10 20:50:26 +0200 |
| commit | 5d774b74ef7d0428a3e45c9bb5b3a0915d2240cb (patch) | |
| tree | 4cd95db54931154ab55ac10f14924eab29386f0c /drivers/net/wireless/at76c50x-usb.c | |
| parent | rndis_wlan: Fix potential memory leak in update_pmkid() (diff) | |
| download | linux-5d774b74ef7d0428a3e45c9bb5b3a0915d2240cb.tar.xz linux-5d774b74ef7d0428a3e45c9bb5b3a0915d2240cb.zip | |
wireless: at76c50x: signedness bug in at76_dfu_get_state()
This return holds the number of bytes transfered (1 byte) or a negative
error code. The type should be int instead of u8.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Pavel Roskin <proski@gnu.org>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'drivers/net/wireless/at76c50x-usb.c')
| -rw-r--r-- | drivers/net/wireless/at76c50x-usb.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/wireless/at76c50x-usb.c b/drivers/net/wireless/at76c50x-usb.c index efc162e0b511..88b8d64c90f1 100644 --- a/drivers/net/wireless/at76c50x-usb.c +++ b/drivers/net/wireless/at76c50x-usb.c @@ -342,7 +342,7 @@ static int at76_dfu_get_status(struct usb_device *udev, return ret; } -static u8 at76_dfu_get_state(struct usb_device *udev, u8 *state) +static int at76_dfu_get_state(struct usb_device *udev, u8 *state) { int ret; |