Re: unregister_netdevice 2.6.9

[YOSHIFUJI Hideaki / 吉藤英明 <yoshfuji@xxxxxxxxxxxxxx>]

In article <Pine.LNX.4.58LT.0410191738420.2725@xxxxxxxxxxxxxxxx> (at Tue, 19 
Oct 2004 17:58:58 +0200 (CEST)), Lukasz Trabinski <lukasz@xxxxxxxxxxxxx> says:

> After shutdown/reboot :
> 
> unregister_netdevice: waiting for xxxx to become free. Usage count = 1
> 
> and still wait.

Does this fix your problem?

--- Begin Message ---

To:	kaber@xxxxxxxxx, davem@xxxxxxxxxxxxx
Subject:	Re: [PATCH 2.6.9]: Fix netdevice/inet6_dev reference leaks in ip6_route_add error paths
From:	YOSHIFUJI Hideaki / 吉藤英明 <yoshfuji@xxxxxxxxxxxxxx>
Date:	Wed, 20 Oct 2004 15:35:25 +0900 (JST)
Cc:	netdev@xxxxxxxxxxx
Delivered-to:	yoshfuji@xxxxxxxxxxxxxxxxx
Delivered-to:	yoshfuji@xxxxxxxxxxxxxxxxxxxxxx
In-reply-to:	<417591A2.4090201@xxxxxxxxx>
Organization:	USAGI Project
References:	<417591A2.4090201@xxxxxxxxx>
Sender:	netdev-bounce@xxxxxxxxxxx

In article <417591A2.4090201@xxxxxxxxx> (at Wed, 20 Oct 2004 00:13:54 +0200), 
Patrick McHardy <kaber@xxxxxxxxx> says:

> The recent changes to ip6_route_add added multiple leaks in
> error paths. This patch should fix them.

Agreed. David, please apply.

Signed-off-by: Hideaki YOSHIFUJI <yoshfuji@xxxxxxxxxxxxxx>

--yoshfuji

--- Begin Message ---

To:	"David S. Miller" <davem@xxxxxxxxxx>
Subject:	[PATCH 2.6.9]: Fix netdevice/inet6_dev reference leaks in ip6_route_add error paths
From:	Patrick McHardy <kaber@xxxxxxxxx>
Date:	Wed, 20 Oct 2004 00:13:54 +0200
Cc:	netdev@xxxxxxxxxxx, YOSHIFUJI Hideaki <yoshfuji@xxxxxxxxxxxxxx>
Delivered-to:	yoshfuji@xxxxxxxxxxxxxxxxx
Delivered-to:	yoshfuji@xxxxxxxxxxxxxxxxxxxxxx
User-agent:	Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.6) Gecko/20040413 Debian/1.6-5

The recent changes to ip6_route_add added multiple leaks in
error paths. This patch should fix them.

Regards
Patrick

# This is a BitKeeper generated diff -Nru style patch.
#
# ChangeSet
#   2004/10/19 23:46:56+02:00 kaber@xxxxxxxxxxxx 
#   [IPV6]: Fix netdevice/inet6_dev reference leaks in ip6_route_add error paths
#   
#   Signed-off-by: Patrick McHardy <kaber@xxxxxxxxx>
# 
# net/ipv6/route.c
#   2004/10/19 23:46:17+02:00 kaber@xxxxxxxxxxxx +8 -3
#   [IPV6]: Fix netdevice/inet6_dev reference leaks in ip6_route_add error paths
#   
#   Signed-off-by: Patrick McHardy <kaber@xxxxxxxxx>
# 
diff -Nru a/net/ipv6/route.c b/net/ipv6/route.c
--- a/net/ipv6/route.c  2004-10-19 23:52:23 +02:00
+++ b/net/ipv6/route.c  2004-10-19 23:52:23 +02:00
@@ -798,8 +798,10 @@
 
        rt = ip6_dst_alloc();
 
-       if (rt == NULL)
-               return -ENOMEM;
+       if (rt == NULL) {
+               err = -ENOMEM;
+               goto out;
+       }
 
        rt->u.dst.obsolete = -1;
        rt->rt6i_expires = clock_t_to_jiffies(rtmsg->rtmsg_info);
@@ -962,7 +964,10 @@
 out:
        if (dev)
                dev_put(dev);
-       dst_free((struct dst_entry *) rt);
+       if (idev)
+               in6_dev_put(idev);
+       if (rt)
+               dst_free((struct dst_entry *) rt);
        return err;
 }
 

--- End Message ---

--- End Message ---