diff options
author | Herbert Xu <herbert@gondor.apana.org.au> | 2007-10-14 03:38:32 -0400 |
---|---|---|
committer | David S. Miller <davem@sunset.davemloft.net> | 2007-10-15 15:26:25 -0400 |
commit | 776c729e8d91b2740583a2169678f2d3f383458b (patch) | |
tree | 16ad06616e8d04c127266bf323fb35fbed03b96d /net/ipv4/ip_fragment.c | |
parent | 1706d58763c36133d7fce6cc78b1444fd40db28c (diff) |
[IPV4]: Change ip_defrag to return an integer
Now that ip_frag always returns the packet given to it on input, we can
change it to return an integer indicating error instead. This patch does
that and updates all its callers accordingly.
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4/ip_fragment.c')
-rw-r--r-- | net/ipv4/ip_fragment.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/net/ipv4/ip_fragment.c b/net/ipv4/ip_fragment.c index d7fa2bf3a0c1..32108cf2a784 100644 --- a/net/ipv4/ip_fragment.c +++ b/net/ipv4/ip_fragment.c | |||
@@ -739,7 +739,7 @@ out_fail: | |||
739 | } | 739 | } |
740 | 740 | ||
741 | /* Process an incoming IP datagram fragment. */ | 741 | /* Process an incoming IP datagram fragment. */ |
742 | struct sk_buff *ip_defrag(struct sk_buff *skb, u32 user) | 742 | int ip_defrag(struct sk_buff *skb, u32 user) |
743 | { | 743 | { |
744 | struct ipq *qp; | 744 | struct ipq *qp; |
745 | 745 | ||
@@ -759,12 +759,12 @@ struct sk_buff *ip_defrag(struct sk_buff *skb, u32 user) | |||
759 | 759 | ||
760 | spin_unlock(&qp->lock); | 760 | spin_unlock(&qp->lock); |
761 | ipq_put(qp, NULL); | 761 | ipq_put(qp, NULL); |
762 | return ret ? NULL : skb; | 762 | return ret; |
763 | } | 763 | } |
764 | 764 | ||
765 | IP_INC_STATS_BH(IPSTATS_MIB_REASMFAILS); | 765 | IP_INC_STATS_BH(IPSTATS_MIB_REASMFAILS); |
766 | kfree_skb(skb); | 766 | kfree_skb(skb); |
767 | return NULL; | 767 | return -ENOMEM; |
768 | } | 768 | } |
769 | 769 | ||
770 | void __init ipfrag_init(void) | 770 | void __init ipfrag_init(void) |