xfrm interface: don't take extra reference to netdev
authorNicolas Dichtel <nicolas.dichtel@6wind.com>
Thu, 23 Apr 2020 09:39:20 +0000 (11:39 +0200)
committerSteffen Klassert <steffen.klassert@secunet.com>
Mon, 27 Apr 2020 06:50:49 +0000 (08:50 +0200)
I don't see any reason to do this. Maybe needed before
commit 56c5ee1a5823 ("xfrm interface: fix memory leak on creation").

Signed-off-by: Nicolas Dichtel <nicolas.dichtel@6wind.com>
Signed-off-by: Steffen Klassert <steffen.klassert@secunet.com>
net/xfrm/xfrm_interface.c

index 3361e3a..eb9928c 100644 (file)
@@ -145,7 +145,6 @@ static int xfrmi_create(struct net_device *dev)
        if (err < 0)
                goto out;
 
-       dev_hold(dev);
        xfrmi_link(xfrmn, xi);
 
        return 0;
@@ -175,7 +174,6 @@ static void xfrmi_dev_uninit(struct net_device *dev)
        struct xfrmi_net *xfrmn = net_generic(xi->net, xfrmi_net_id);
 
        xfrmi_unlink(xfrmn, xi);
-       dev_put(dev);
 }
 
 static void xfrmi_scrub_packet(struct sk_buff *skb, bool xnet)