2006-05-31 04:07:31

by Florin Malita

[permalink] [raw]
Subject: [PATCH] pppoe: missing result check in __pppoe_xmit()

skb_clone() may fail, we should check the result.

Coverity CID: 1215.

Signed-off-by: Florin Malita <[email protected]>
---

diff --git a/drivers/net/pppoe.c b/drivers/net/pppoe.c
index 475dc93..0d101a1 100644
--- a/drivers/net/pppoe.c
+++ b/drivers/net/pppoe.c
@@ -861,6 +861,9 @@ static int __pppoe_xmit(struct sock *sk,
* give dev_queue_xmit something it can free.
*/
skb2 = skb_clone(skb, GFP_ATOMIC);
+
+ if (skb2 == NULL)
+ goto abort;
}

ph = (struct pppoe_hdr *) skb_push(skb2, sizeof(struct pppoe_hdr));