diff options
| author | Felipe Balbi <balbi@ti.com> | 2012-05-29 14:38:32 +0300 |
|---|---|---|
| committer | Felipe Balbi <balbi@ti.com> | 2012-06-03 23:11:34 +0300 |
| commit | 23673d7d26d81bd17cde3ed74c57b3f39585325f (patch) | |
| tree | e269bf56ca6bdd6a1e40ddba6ee42f70798e93b5 /drivers/usb/gadget | |
| parent | 70617db7ad7395498e6bc54c634199bf895426c6 (diff) | |
| download | op-kernel-dev-23673d7d26d81bd17cde3ed74c57b3f39585325f.zip op-kernel-dev-23673d7d26d81bd17cde3ed74c57b3f39585325f.tar.gz | |
usb: gadget: omap_udc: kfree(NULL) is safe
we don't need to check for _req because
kfree(NULL) is safe. Also, if someone
actually passes a NULL pointer to be freed
by usb_ep_free_request(), he deserves any
issue he faces.
Signed-off-by: Felipe Balbi <balbi@ti.com>
Diffstat (limited to 'drivers/usb/gadget')
| -rw-r--r-- | drivers/usb/gadget/omap_udc.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/usb/gadget/omap_udc.c b/drivers/usb/gadget/omap_udc.c index f13bcdc..50e8490 100644 --- a/drivers/usb/gadget/omap_udc.c +++ b/drivers/usb/gadget/omap_udc.c @@ -282,8 +282,7 @@ omap_free_request(struct usb_ep *ep, struct usb_request *_req) { struct omap_req *req = container_of(_req, struct omap_req, req); - if (_req) - kfree(req); + kfree(req); } /*-------------------------------------------------------------------------*/ |
