When an error occurs adding a udc platform device there is a risk of an infinite loop. If more than one platform device was added i will remain >= than 0. The intention seems to clean up all the different already added platform devices before the failure occurs, so fixed the code to actually do so. We need to decrement first because the adding at the current index of i is the one that failed.
At the same time the code is harmonized for hcd platform adding. Found with coverity : CID 751073 Signed-off-by: Philippe De Swert <[email protected]> --- drivers/usb/gadget/dummy_hcd.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/drivers/usb/gadget/dummy_hcd.c b/drivers/usb/gadget/dummy_hcd.c index c588e8e..03a2300 100644 --- a/drivers/usb/gadget/dummy_hcd.c +++ b/drivers/usb/gadget/dummy_hcd.c @@ -2684,9 +2684,8 @@ static int __init init(void) for (i = 0; i < mod_data.num; i++) { retval = platform_device_add(the_hcd_pdev[i]); if (retval < 0) { - i--; - while (i >= 0) - platform_device_del(the_hcd_pdev[i--]); + while (--i >= 0) + platform_device_del(the_hcd_pdev[i]); goto err_add_hcd; } } @@ -2705,8 +2704,7 @@ static int __init init(void) for (i = 0; i < mod_data.num; i++) { retval = platform_device_add(the_udc_pdev[i]); if (retval < 0) { - i--; - while (i >= 0) + while (--i >= 0) platform_device_del(the_udc_pdev[i]); goto err_add_udc; } -- 1.8.1.2 -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
