diff options
author | Eric Dumazet <edumazet@google.com> | 2012-04-24 06:17:59 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2012-04-25 20:39:46 -0400 |
commit | 808db80a7eeff8314ac51654bfefc864582eea13 (patch) | |
tree | 9c42d1486a5e6052d0c285a5a6a6d38fdbe0b890 /net/ipv6/ip6_output.c | |
parent | 081579840b4b2421e37bc67e3b089b7ca64ef040 (diff) |
ipv6: call consume_skb() in frag/reassembly
Some kfree_skb() calls should be replaced by consume_skb() to avoid
drop_monitor/dropwatch false positives.
Signed-off-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv6/ip6_output.c')
-rw-r--r-- | net/ipv6/ip6_output.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/ipv6/ip6_output.c b/net/ipv6/ip6_output.c index b7ca46161cb9..b347062aa809 100644 --- a/net/ipv6/ip6_output.c +++ b/net/ipv6/ip6_output.c | |||
@@ -210,7 +210,7 @@ int ip6_xmit(struct sock *sk, struct sk_buff *skb, struct flowi6 *fl6, | |||
210 | kfree_skb(skb); | 210 | kfree_skb(skb); |
211 | return -ENOBUFS; | 211 | return -ENOBUFS; |
212 | } | 212 | } |
213 | kfree_skb(skb); | 213 | consume_skb(skb); |
214 | skb = skb2; | 214 | skb = skb2; |
215 | skb_set_owner_w(skb, sk); | 215 | skb_set_owner_w(skb, sk); |
216 | } | 216 | } |
@@ -889,7 +889,7 @@ slow_path: | |||
889 | } | 889 | } |
890 | IP6_INC_STATS(net, ip6_dst_idev(skb_dst(skb)), | 890 | IP6_INC_STATS(net, ip6_dst_idev(skb_dst(skb)), |
891 | IPSTATS_MIB_FRAGOKS); | 891 | IPSTATS_MIB_FRAGOKS); |
892 | kfree_skb(skb); | 892 | consume_skb(skb); |
893 | return err; | 893 | return err; |
894 | 894 | ||
895 | fail: | 895 | fail: |