usb: gadget: f_hid: actually limit the number of instances
authorAndrzej Pietrasiewicz <andrzej.p@samsung.com>
Fri, 24 Jul 2015 07:48:40 +0000 (09:48 +0200)
committerFelipe Balbi <balbi@ti.com>
Mon, 27 Jul 2015 15:19:50 +0000 (10:19 -0500)
There is a predefined maximum number of hid instances, currently 4.
A chrdev region is allocated accordingly, but with configfs the user
can create as many hid function directories as they like. To make
the number of hid instances consistent with the number of allocated minors,
the limit is enforced at directory creation time.

Signed-off-by: Andrzej Pietrasiewicz <andrzej.p@samsung.com>
Signed-off-by: Felipe Balbi <balbi@ti.com>
drivers/usb/gadget/function/f_hid.c

index f7f35a3..6df9715 100644 (file)
@@ -699,6 +699,10 @@ static inline int hidg_get_minor(void)
        int ret;
 
        ret = ida_simple_get(&hidg_ida, 0, 0, GFP_KERNEL);
+       if (ret >= HIDG_MINORS) {
+               ida_simple_remove(&hidg_ida, ret);
+               ret = -ENODEV;
+       }
 
        return ret;
 }