diff options
| author | Dave Jones <davej@redhat.com> | 2006-03-11 18:49:13 -0800 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2006-03-11 18:49:13 -0800 |
| commit | 8f22757ee84da64d4f662f1dd03dae95358a1efe (patch) | |
| tree | ebbf96d3c733c5cad9ab002740f1d4b829e07a4e /drivers | |
| parent | 535744878e34d01a53f946f26dfbca37186f2cf8 (diff) | |
| download | op-kernel-dev-8f22757ee84da64d4f662f1dd03dae95358a1efe.zip op-kernel-dev-8f22757ee84da64d4f662f1dd03dae95358a1efe.tar.gz | |
[TUN]: Fix leak in tun_get_user()
We're leaking an skb in a failure path in this function.
Coverity #632
Signed-off-by: Dave Jones <davej@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/net/tun.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/net/tun.c b/drivers/net/tun.c index 50b8c67..a1ed2d9 100644 --- a/drivers/net/tun.c +++ b/drivers/net/tun.c @@ -249,8 +249,11 @@ static __inline__ ssize_t tun_get_user(struct tun_struct *tun, struct iovec *iv, if (align) skb_reserve(skb, align); - if (memcpy_fromiovec(skb_put(skb, len), iv, len)) + if (memcpy_fromiovec(skb_put(skb, len), iv, len)) { + tun->stats.rx_dropped++; + kfree_skb(skb); return -EFAULT; + } skb->dev = tun->dev; switch (tun->flags & TUN_TYPE_MASK) { |
