[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Date: Sat, 11 Nov 2006 17:06:43 +0100
From: Adrian Bunk <bunk@...sta.de>
To: greg@...ah.com
Cc: linux-usb-devel@...ts.sourceforge.net, linux-kernel@...r.kernel.org
Subject: drivers/usb/gadget/ether.c: NULL dereference
The Coverity checker spotted the following NULL dereference of "skb" in
drivers/usb/gadget/ether.c:
<-- snip -->
...
static int
rx_submit (struct eth_dev *dev, struct usb_request *req, gfp_t gfp_flags)
{
struct sk_buff *skb;
int retval = -ENOMEM;
...
if ((skb = alloc_skb (size + NET_IP_ALIGN, gfp_flags)) == 0) {
DEBUG (dev, "no rx skb\n");
goto enomem;
}
...
enomem:
defer_kevent (dev, WORK_RX_MEMORY);
if (retval) {
DEBUG (dev, "rx submit --> %d\n", retval);
dev_kfree_skb_any (skb);
...
<-- snip -->
cu
Adrian
--
"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists