Commit 0d7eacbe authored by Jason Wang's avatar Jason Wang Committed by David S. Miller

macvtap: correctly free skb during socket destruction

We should use kfree_skb() instead of kfree() to free an skb.

Fixes: 362899b8 ("macvtap: switch to use skb array")
Reported-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: default avatarJason Wang <jasowang@redhat.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 2186f6ee
...@@ -536,7 +536,7 @@ static void macvtap_sock_destruct(struct sock *sk) ...@@ -536,7 +536,7 @@ static void macvtap_sock_destruct(struct sock *sk)
struct sk_buff *skb; struct sk_buff *skb;
while ((skb = skb_array_consume(&q->skb_array)) != NULL) while ((skb = skb_array_consume(&q->skb_array)) != NULL)
kfree(skb); kfree_skb(skb);
} }
static int macvtap_open(struct inode *inode, struct file *file) static int macvtap_open(struct inode *inode, struct file *file)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment