From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx2.suse.de ([195.135.220.15]:42402 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751112AbeCGKgg (ORCPT ); Wed, 7 Mar 2018 05:36:36 -0500 Message-ID: <1520418651.5229.4.camel@suse.com> Subject: Re: [PATCH net-next 2/2] net: cdc_eem: clean up bind error path From: Oliver Neukum To: Johan Hovold , "David S. Miller" Cc: linux-usb@vger.kernel.org, netdev@vger.kernel.org Date: Wed, 07 Mar 2018 11:30:51 +0100 In-Reply-To: <20180307094658.9008-2-johan@kernel.org> References: <20180307094658.9008-1-johan@kernel.org> <20180307094658.9008-2-johan@kernel.org> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: netdev-owner@vger.kernel.org List-ID: Am Mittwoch, den 07.03.2018, 10:46 +0100 schrieb Johan Hovold: > Drop bogus call to usb_driver_release_interface() from an error path in > the usbnet bind() callback, which is called during interface probe. At > this point the interface is not bound and usb_driver_release_interface() > returns early. > > Also remove the bogus call to clear the interface data, which is owned > by the usbnet driver and would not even have been set by the time bind() > is called. > > Signed-off-by: Johan Hovold Acked-by: Oliver Neukum > --- > drivers/net/usb/cdc_eem.c | 5 +---- > 1 file changed, 1 insertion(+), 4 deletions(-) > > diff --git a/drivers/net/usb/cdc_eem.c b/drivers/net/usb/cdc_eem.c > index f7180f8db39e..61ea4eaace5d 100644 > --- a/drivers/net/usb/cdc_eem.c > +++ b/drivers/net/usb/cdc_eem.c > @@ -83,11 +83,8 @@ static int eem_bind(struct usbnet *dev, struct usb_interface *intf) > int status = 0; > > status = usbnet_get_endpoints(dev, intf); > - if (status < 0) { > - usb_set_intfdata(intf, NULL); > - usb_driver_release_interface(driver_of(intf), intf); > + if (status < 0) > return status; > - } > > /* no jumbogram (16K) support for now */ >