Messages in this thread | | | Date | Wed, 4 Jan 2012 09:55:19 +0100 (CET) | From | Jiri Kosina <> | Subject | Re: [PATCH] input: Synaptics USB device driver |
| |
On Tue, 3 Jan 2012, Jan Steinhoff wrote:
[ ... snip ... ] > +static int synusb_setup_iurb(struct synusb *synusb, > + struct usb_endpoint_descriptor *endpoint) > +{ > + char *buf; > + > + if (endpoint->wMaxPacketSize < 8) > + return 0; > + if (synusb->iurb) { > + synusb_warn(synusb, "Found more than one possible " > + "int in endpoint"); > + return 0; > + } > + synusb->iurb = usb_alloc_urb(0, GFP_KERNEL); > + if (synusb->iurb == NULL) > + return -ENOMEM; > + buf = usb_alloc_coherent(synusb->udev, 8, GFP_ATOMIC, > + &synusb->iurb->transfer_dma); > + if (buf == NULL) > + return -ENOMEM;
You seem to leak synusb->iurb here.
-- Jiri Kosina SUSE Labs
| |