Skip to content

Commit c4854ec

Browse files
ramirosendavem330
authored andcommitted
ipmr: change the prototype of ip_mr_forward().
This patch changes the prototpye of the ip_mr_forward() method to return void instead of int. The ip_mr_forward() method always returns 0; moreover, the return value of this method is not checked anywhere. Signed-off-by: Rami Rosen <ramirose@gmail.com> Acked-by: Nicolas Dichtel <nicolas.dichtel@6wind.com> Signed-off-by: David S. Miller <davem@davemloft.net>
1 parent 45c9149 commit c4854ec

File tree

1 file changed

+7
-8
lines changed

1 file changed

+7
-8
lines changed

net/ipv4/ipmr.c

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -127,9 +127,9 @@ static struct kmem_cache *mrt_cachep __read_mostly;
127127
static struct mr_table *ipmr_new_table(struct net *net, u32 id);
128128
static void ipmr_free_table(struct mr_table *mrt);
129129

130-
static int ip_mr_forward(struct net *net, struct mr_table *mrt,
131-
struct sk_buff *skb, struct mfc_cache *cache,
132-
int local);
130+
static void ip_mr_forward(struct net *net, struct mr_table *mrt,
131+
struct sk_buff *skb, struct mfc_cache *cache,
132+
int local);
133133
static int ipmr_cache_report(struct mr_table *mrt,
134134
struct sk_buff *pkt, vifi_t vifi, int assert);
135135
static int __ipmr_fill_mroute(struct mr_table *mrt, struct sk_buff *skb,
@@ -1795,9 +1795,9 @@ static int ipmr_find_vif(struct mr_table *mrt, struct net_device *dev)
17951795

17961796
/* "local" means that we should preserve one skb (for local delivery) */
17971797

1798-
static int ip_mr_forward(struct net *net, struct mr_table *mrt,
1799-
struct sk_buff *skb, struct mfc_cache *cache,
1800-
int local)
1798+
static void ip_mr_forward(struct net *net, struct mr_table *mrt,
1799+
struct sk_buff *skb, struct mfc_cache *cache,
1800+
int local)
18011801
{
18021802
int psend = -1;
18031803
int vif, ct;
@@ -1903,14 +1903,13 @@ static int ip_mr_forward(struct net *net, struct mr_table *mrt,
19031903
ipmr_queue_xmit(net, mrt, skb2, cache, psend);
19041904
} else {
19051905
ipmr_queue_xmit(net, mrt, skb, cache, psend);
1906-
return 0;
1906+
return;
19071907
}
19081908
}
19091909

19101910
dont_forward:
19111911
if (!local)
19121912
kfree_skb(skb);
1913-
return 0;
19141913
}
19151914

19161915
static struct mr_table *ipmr_rt_fib_lookup(struct net *net, struct sk_buff *skb)

0 commit comments

Comments
 (0)