diff options
| author | Eric Dumazet <eric.dumazet@gmail.com> | 2011-10-17 13:01:47 -0400 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2011-10-19 02:39:43 -0400 |
| commit | 8bae8bd6cb24eecad9fda3e125d36ab9c67d3fd7 (patch) | |
| tree | 2cfbe13ebde2a8d771688a27e702554f1b2ea912 /drivers | |
| parent | 4d97480b1806e883eb1c7889d4e7a87e936e06d9 (diff) | |
pptp: fix skb leak in pptp_xmit()
In case we cant transmit skb, we must free it
Signed-off-by: Eric Dumazet <eric.dumazet@gmail.com>
CC: Dmitry Kozlov <xeb@mail.ru>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/net/pptp.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/net/pptp.c b/drivers/net/pptp.c index eae542a7e987..9c0403d0107c 100644 --- a/drivers/net/pptp.c +++ b/drivers/net/pptp.c | |||
| @@ -285,8 +285,10 @@ static int pptp_xmit(struct ppp_channel *chan, struct sk_buff *skb) | |||
| 285 | ip_send_check(iph); | 285 | ip_send_check(iph); |
| 286 | 286 | ||
| 287 | ip_local_out(skb); | 287 | ip_local_out(skb); |
| 288 | return 1; | ||
| 288 | 289 | ||
| 289 | tx_error: | 290 | tx_error: |
| 291 | kfree_skb(skb); | ||
| 290 | return 1; | 292 | return 1; |
| 291 | } | 293 | } |
| 292 | 294 | ||
