From: Pavitrakumar Managutte Date: Wed, 22 Oct 2014 14:03:22 +0000 (+0530) Subject: usb: gadget: function: Fixed the return value on error path X-Git-Tag: v3.18-rc3~9^2~2^2 X-Git-Url: https://openfabrics.org/gitweb/?a=commitdiff_plain;h=9b1763553a89b2a84881119eeabfccdb803bb468;p=~emulex%2Finfiniband.git usb: gadget: function: Fixed the return value on error path Fixed the return value on failure. status variable is set to 0 at usb_assign_descriptors call and the same is returned on error which is incorrect. Acked-by: Sebastian Andrzej Siewior Signed-off-by: Pavitrakumar Managutte Signed-off-by: Felipe Balbi --- diff --git a/drivers/usb/gadget/function/f_rndis.c b/drivers/usb/gadget/function/f_rndis.c index 2f0517f5bae..f13fc6a5856 100644 --- a/drivers/usb/gadget/function/f_rndis.c +++ b/drivers/usb/gadget/function/f_rndis.c @@ -802,8 +802,10 @@ rndis_bind(struct usb_configuration *c, struct usb_function *f) if (rndis->manufacturer && rndis->vendorID && rndis_set_param_vendor(rndis->config, rndis->vendorID, - rndis->manufacturer)) + rndis->manufacturer)) { + status = -EINVAL; goto fail_free_descs; + } /* NOTE: all that is done without knowing or caring about * the network link ... which is unavailable to this code