diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2014-09-09 12:06:09 (GMT) |
---|---|---|
committer | Felipe Balbi <balbi@ti.com> | 2014-09-09 14:51:21 (GMT) |
commit | 85b06f5e53d17c15844ef3cd45d0c7107f0ae45c (patch) | |
tree | 622b0f357dba7e623ed2b3cb205041e42223786c /drivers/usb/gadget | |
parent | 13443799b559cde593826091a7de135483b245e5 (diff) | |
download | linux-85b06f5e53d17c15844ef3cd45d0c7107f0ae45c.tar.xz |
usb: gadget: f_fs: signedness bug in __ffs_func_bind_do_descs()
We need "idx" to be signed for the error handling to work.
Fixes: 6d5c1c77bbf9 ('usb: gadget: f_fs: fix the redundant ep files problem')
Acked-by: Michal Nazarewicz <mina86@mina86.com>
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Felipe Balbi <balbi@ti.com>
Diffstat (limited to 'drivers/usb/gadget')
-rw-r--r-- | drivers/usb/gadget/function/f_fs.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/usb/gadget/function/f_fs.c b/drivers/usb/gadget/function/f_fs.c index dc30adf..1aad353 100644 --- a/drivers/usb/gadget/function/f_fs.c +++ b/drivers/usb/gadget/function/f_fs.c @@ -2352,7 +2352,8 @@ static int __ffs_func_bind_do_descs(enum ffs_entity_type type, u8 *valuep, struct usb_endpoint_descriptor *ds = (void *)desc; struct ffs_function *func = priv; struct ffs_ep *ffs_ep; - unsigned ep_desc_id, idx; + unsigned ep_desc_id; + int idx; static const char *speed_names[] = { "full", "high", "super" }; if (type != FFS_DESCRIPTOR) |