From: zhang kai <zhangkai...@126.com>

In the end of function __ip_vs_get_out_rt/__ip_vs_get_out_rt_v6,the
'local' variable is always zero.

Signed-off-by: zhang kai <zhangkai...@126.com>
Acked-by: Julian Anastasov <j...@ssi.bg>
Signed-off-by: Simon Horman <ho...@verge.net.au>
---
 net/netfilter/ipvs/ip_vs_xmit.c | 18 ++++++------------
 1 file changed, 6 insertions(+), 12 deletions(-)

diff --git a/net/netfilter/ipvs/ip_vs_xmit.c b/net/netfilter/ipvs/ip_vs_xmit.c
index 888d3068a492..b1e300f8881b 100644
--- a/net/netfilter/ipvs/ip_vs_xmit.c
+++ b/net/netfilter/ipvs/ip_vs_xmit.c
@@ -407,12 +407,9 @@ __ip_vs_get_out_rt(struct netns_ipvs *ipvs, int skb_af, 
struct sk_buff *skb,
                goto err_put;
 
        skb_dst_drop(skb);
-       if (noref) {
-               if (!local)
-                       skb_dst_set_noref(skb, &rt->dst);
-               else
-                       skb_dst_set(skb, dst_clone(&rt->dst));
-       } else
+       if (noref)
+               skb_dst_set_noref(skb, &rt->dst);
+       else
                skb_dst_set(skb, &rt->dst);
 
        return local;
@@ -574,12 +571,9 @@ __ip_vs_get_out_rt_v6(struct netns_ipvs *ipvs, int skb_af, 
struct sk_buff *skb,
                goto err_put;
 
        skb_dst_drop(skb);
-       if (noref) {
-               if (!local)
-                       skb_dst_set_noref(skb, &rt->dst);
-               else
-                       skb_dst_set(skb, dst_clone(&rt->dst));
-       } else
+       if (noref)
+               skb_dst_set_noref(skb, &rt->dst);
+       else
                skb_dst_set(skb, &rt->dst);
 
        return local;
-- 
2.11.0

Reply via email to