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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Mon, 9 Dec 2013 14:27:55 +0800
From:	Fan Du <fan.du@...driver.com>
To:	David Miller <davem@...emloft.net>
CC:	<netdev@...r.kernel.org>, Eric Dumazet <eric.dumazet@...il.com>
Subject: [PATCHv2 net] vxlan: release rt when found circular route



On 2013年12月07日 04:14, David Miller wrote:
> From: Fan Du<fan.du@...driver.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@...driver.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@...driver.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@...driver.com>
Acked-by: Eric Dumazet <edumazet@...gle.com>
---
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
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html