Message ID | 20130806150114.GA29184@redhat.com |
---|---|
State | Accepted, archived |
Delegated to: | David Miller |
Headers | show |
From: Dave Jones <davej@redhat.com> Date: Tue, 6 Aug 2013 11:01:14 -0400 > 8139cp: Fix skb leak in rx_status_loop failure path. > Introduced in cf3c4c03060b688cbc389ebc5065ebcce5653e96 > > Signed-off-by: Dave Jones <davej@redhat.com> Applied, but Dave please take care of all of the usual formalities that I took care for you here next time. Reference commits by SHA1 ID _and_ in parenthesis and double quotes, show the commit message header line as well. Post patches as self contained patch postings, so that I don't have to edit the commit message and attend to the other details like that. Thanks. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/net/ethernet/realtek/8139cp.c b/drivers/net/ethernet/realtek/8139cp.c index 6f35f84..d2e5919 100644 --- a/drivers/net/ethernet/realtek/8139cp.c +++ b/drivers/net/ethernet/realtek/8139cp.c @@ -524,6 +524,7 @@ rx_status_loop: PCI_DMA_FROMDEVICE); if (dma_mapping_error(&cp->pdev->dev, new_mapping)) { dev->stats.rx_dropped++; + kfree_skb(new_skb); goto rx_next; }