On Thu, 14 Jun 2018, Jaejoong Kim wrote: > fsg_common_set_num_buffers() may fail due to ENOMEM. So add > error handling for fail case. > > Signed-off-by: Jaejoong Kim <climbbb.kim@xxxxxxxxx> > --- > drivers/usb/gadget/function/f_mass_storage.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/drivers/usb/gadget/function/f_mass_storage.c b/drivers/usb/gadget/function/f_mass_storage.c > index 83eeafe..b6e2930 100644 > --- a/drivers/usb/gadget/function/f_mass_storage.c > +++ b/drivers/usb/gadget/function/f_mass_storage.c > @@ -3308,7 +3308,9 @@ static ssize_t fsg_opts_num_buffers_store(struct config_item *item, > if (ret) > goto end; > > - fsg_common_set_num_buffers(opts->common, num); > + ret = fsg_common_set_num_buffers(opts->common, num); > + if (ret) > + goto end; > ret = len; > > end: Acked-by: Alan Stern <stern@xxxxxxxxxxxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
