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

On 2013年12月07日 04:14, David Miller wrote:
> From: Fan Du<fan.du@windriver.com>
> 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<fan.du@windriver.com>
> ...
>> @@ -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 <fan.du@windriver.com>
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 <fan.du@windriver.com>
Acked-by: Eric Dumazet <edumazet@google.com>
---
v2: use rt_tx_error label to release rt.
---
  drivers/net/vxlan.c |    2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

From: Fan Du <fan.du@windriver.com>
Date: Mon, 9 Dec 2013 14:27:55 +0800

> From: Fan Du <fan.du@windriver.com>
> 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 <fan.du@windriver.com>
> Acked-by: Eric Dumazet <edumazet@google.com>
> ---
> v2: use rt_tx_error label to release rt.

Applied and queued up for -stable, thanks.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html