The rtnl lock is taken just the line above, no need to take the rcu also.

Fixes: 1788b8569f5d ("gtp: fix use-after-free in gtp_encap_destroy()")
Signed-off-by: Nicolas Dichtel <nicolas.dich...@6wind.com>
---
 drivers/net/gtp.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/drivers/net/gtp.c b/drivers/net/gtp.c
index c84a10569388..6f871ec31393 100644
--- a/drivers/net/gtp.c
+++ b/drivers/net/gtp.c
@@ -1071,7 +1071,6 @@ static int gtp_genl_new_pdp(struct sk_buff *skb, struct 
genl_info *info)
        }
 
        rtnl_lock();
-       rcu_read_lock();
 
        gtp = gtp_find_dev(sock_net(skb->sk), info->attrs);
        if (!gtp) {
@@ -1100,7 +1099,6 @@ static int gtp_genl_new_pdp(struct sk_buff *skb, struct 
genl_info *info)
        }
 
 out_unlock:
-       rcu_read_unlock();
        rtnl_unlock();
        return err;
 }
-- 
2.26.2

Reply via email to