Message ID | 1518456819-22244-16-git-send-email-jchapman@katalix.com |
---|---|
State | Changes Requested, archived |
Delegated to: | David Miller |
Headers | show |
Series | l2tp: fix API races discovered by syzbot | expand |
diff --git a/net/l2tp/l2tp_core.c b/net/l2tp/l2tp_core.c index a91cd384e397..28940cf9cc76 100644 --- a/net/l2tp/l2tp_core.c +++ b/net/l2tp/l2tp_core.c @@ -1449,8 +1449,6 @@ int l2tp_tunnel_create(struct net *net, int fd, int version, u32 tunnel_id, u32 } else { sock = sockfd_lookup(fd, &err); if (!sock) { - pr_err("tunl %u: sockfd_lookup(fd=%d) returned %d\n", - tunnel_id, fd, err); err = -EBADF; goto err; }
If when creating a new tunnel, the indicated fd is closed by another thread, we emit an error message about it. e.g. l2tp_core: tunl 4: sockfd_lookup(fd=3) returned -9 It's not useful so remove it. Signed-off-by: James Chapman <jchapman@katalix.com> --- net/l2tp/l2tp_core.c | 2 -- 1 file changed, 2 deletions(-)