Commit 8d8a51e2 authored by David S. Miller's avatar David S. Miller

l2tp: Fix build warning with ipv6 disabled.

net/l2tp/l2tp_core.c: In function ‘l2tp_verify_udp_checksum’:
net/l2tp/l2tp_core.c:499:22: warning: unused variable ‘tunnel’ [-Wunused-variable]

Create a helper "l2tp_tunnel()" to facilitate this, and as a side
effect get rid of a bunch of unnecessary void pointer casts.
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 5c0c52c9
...@@ -115,6 +115,11 @@ struct l2tp_net { ...@@ -115,6 +115,11 @@ struct l2tp_net {
static void l2tp_session_set_header_len(struct l2tp_session *session, int version); static void l2tp_session_set_header_len(struct l2tp_session *session, int version);
static void l2tp_tunnel_free(struct l2tp_tunnel *tunnel); static void l2tp_tunnel_free(struct l2tp_tunnel *tunnel);
static inline struct l2tp_tunnel *l2tp_tunnel(struct sock *sk)
{
return sk->sk_user_data;
}
static inline struct l2tp_net *l2tp_pernet(struct net *net) static inline struct l2tp_net *l2tp_pernet(struct net *net)
{ {
BUG_ON(!net); BUG_ON(!net);
...@@ -496,7 +501,6 @@ static void l2tp_recv_dequeue(struct l2tp_session *session) ...@@ -496,7 +501,6 @@ static void l2tp_recv_dequeue(struct l2tp_session *session)
static inline int l2tp_verify_udp_checksum(struct sock *sk, static inline int l2tp_verify_udp_checksum(struct sock *sk,
struct sk_buff *skb) struct sk_buff *skb)
{ {
struct l2tp_tunnel *tunnel = (struct l2tp_tunnel *)sk->sk_user_data;
struct udphdr *uh = udp_hdr(skb); struct udphdr *uh = udp_hdr(skb);
u16 ulen = ntohs(uh->len); u16 ulen = ntohs(uh->len);
__wsum psum; __wsum psum;
...@@ -505,7 +509,7 @@ static inline int l2tp_verify_udp_checksum(struct sock *sk, ...@@ -505,7 +509,7 @@ static inline int l2tp_verify_udp_checksum(struct sock *sk,
return 0; return 0;
#if IS_ENABLED(CONFIG_IPV6) #if IS_ENABLED(CONFIG_IPV6)
if (sk->sk_family == PF_INET6 && !tunnel->v4mapped) { if (sk->sk_family == PF_INET6 && !l2tp_tunnel(sk)->v4mapped) {
if (!uh->check) { if (!uh->check) {
LIMIT_NETDEBUG(KERN_INFO "L2TP: IPv6: checksum is 0\n"); LIMIT_NETDEBUG(KERN_INFO "L2TP: IPv6: checksum is 0\n");
return 1; return 1;
...@@ -1305,10 +1309,9 @@ EXPORT_SYMBOL_GPL(l2tp_xmit_skb); ...@@ -1305,10 +1309,9 @@ EXPORT_SYMBOL_GPL(l2tp_xmit_skb);
*/ */
static void l2tp_tunnel_destruct(struct sock *sk) static void l2tp_tunnel_destruct(struct sock *sk)
{ {
struct l2tp_tunnel *tunnel; struct l2tp_tunnel *tunnel = l2tp_tunnel(sk);
struct l2tp_net *pn; struct l2tp_net *pn;
tunnel = sk->sk_user_data;
if (tunnel == NULL) if (tunnel == NULL)
goto end; goto end;
...@@ -1676,7 +1679,7 @@ int l2tp_tunnel_create(struct net *net, int fd, int version, u32 tunnel_id, u32 ...@@ -1676,7 +1679,7 @@ int l2tp_tunnel_create(struct net *net, int fd, int version, u32 tunnel_id, u32
} }
/* Check if this socket has already been prepped */ /* Check if this socket has already been prepped */
tunnel = (struct l2tp_tunnel *)sk->sk_user_data; tunnel = l2tp_tunnel(sk);
if (tunnel != NULL) { if (tunnel != NULL) {
/* This socket has already been prepped */ /* This socket has already been prepped */
err = -EBUSY; err = -EBUSY;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment