Messages in this thread Patch in this message | | | Date | Thu, 13 Jun 2013 23:12:44 +0100 | From | Dave Kilroy <> | Subject | Re: [PATCH] orinoco_usb: fix memory leak in ezusb_access_ltv() when device disconnected |
| |
On 13/06/2013 21:26, Alexey Khoroshilov wrote: > If "device is disconnected" check occurs to be true in ezusb_access_ltv(), > it just return -ENODEV. But that means request_context is leaked since > there are no any references to it anymore. > The patch adds a call to ezusb_request_context_put() before return. > > Found by Linux Driver Verification project (linuxtesting.org). > > Signed-off-by: Alexey Khoroshilov <khoroshilov@ispras.ru> > --- > drivers/net/wireless/orinoco/orinoco_usb.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/net/wireless/orinoco/orinoco_usb.c b/drivers/net/wireless/orinoco/orinoco_usb.c > index 1f9cb55..bdfe637 100644 > --- a/drivers/net/wireless/orinoco/orinoco_usb.c > +++ b/drivers/net/wireless/orinoco/orinoco_usb.c > @@ -881,7 +881,8 @@ static int ezusb_access_ltv(struct ezusb_priv *upriv, > > if (!upriv->udev) { > dbg("Device disconnected"); > - return -ENODEV; > + retval = -ENODEV; > + goto exit; > } > > if (upriv->read_urb->status != -EINPROGRESS) It looks like there is also loss of a request_context in ezusb_xmit after orinoco_process_xmit_skb fails. Something like the following should resolve it.
The remaining request_context allocations look ok to me.
Dave. --- Apologies, this is cut+paste from an old tree and mangled to be diff-like. Untested. Uncompiled.
diff a/drivers/net/wireless/orinoco/orinoco_usb.c b/drivers/net/wireless/orinoco/orinoco_usb.c --- a/drivers/net/wireless/orinoco/orinoco_usb.c +++ b/drivers/net/wireless/orinoco/orinoco_usb.c @@ -1194,8 +1194,10 @@ static netdev_tx_t ezusb_xmit(struct sk_buff *skb, struct net_device *dev)
err = orinoco_process_xmit_skb(skb, dev, priv, &tx_control, &mic[0]); - if (err) + if (err) { + ezusb_request_context_put(ctx); goto drop; + }
{ __le16 *tx_cntl = (__le16 *)buf;
| |