Subject: [PATCHv2 net] vxlan: release rt when found circular route

From: Fan Du <[email-masked]>
Date: Mon, 9 Dec 2013 14:27:55 +0800

On 2013年12月07日 04:14, David Miller wrote:
> From: Fan Du<[email-masked]>
> Date: Thu, 5 Dec 2013 16:03:51 +0800
>
>> Otherwise causing dst memory leakage.
>> Have Checked all other type tunnel device transmit implementation,
>> no such things happens anymore.
>>
>> Signed-off-by: Fan Du<[email-masked]>
> ...
>> @@ -1665,6 +1665,7 @@ static void vxlan_xmit_one(struct sk_buff *skb, struct net_device *dev,
>>   		}
>>
>>   		if (rt->dst.dev == dev) {
>> +			ip_rt_put(rt);
>>   			netdev_dbg(dev, "circular route to %pI4\n",
>>   				&dst->sin.sin_addr.s_addr);
>>   			dev->stats.collisions++;
>
> There is already a "rt_tx_error" label that will handle releasing the
> 'rt', so at the end of this basic block, goto rt_tx_error instead of
> just plain tx_error.

My first thought is using rt_tx_error indeed :), but second thought is
this might criticized by being ugly. Anyway please review.

 From 8867cb9783511a1c945a087209484f74de912689 Mon Sep 17 00:00:00 2001
From: Fan Du <[email-masked]>
Date: Mon, 9 Dec 2013 10:33:53 +0800
Subject: [PATCHv2 net] vxlan: release rt when found circular route

Otherwise causing dst memory leakage.
Have Checked all other type tunnel device transmit implementation,
no such things happens anymore.

Signed-off-by: Fan Du <[email-masked]>
Acked-by: Eric Dumazet <[email-masked]>
---
v2: use rt_tx_error label to release rt.
---
  drivers/net/vxlan.c |    2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/vxlan.c b/drivers/net/vxlan.c
index 0358c07..249e01c 100644
--- a/drivers/net/vxlan.c
+++ b/drivers/net/vxlan.c
@@ -1668,7 +1668,7 @@ static void vxlan_xmit_one(struct sk_buff *skb, struct net_device *dev,
  			netdev_dbg(dev, "circular route to %pI4\n",
  				   &dst->sin.sin_addr.s_addr);
  			dev->stats.collisions++;
-			goto tx_error;
+			goto rt_tx_error;
  		}

  		/* Bypass encapsulation if the destination is local */
-- 
1.7.9.5

-- 
浮沉随浪只记今朝笑

--fan