Commit 95075150 authored by Tom Parkin's avatar Tom Parkin Committed by David S. Miller

l2tp: avoid multiple assignments

checkpatch warns about multiple assignments.

Update l2tp accordingly.
Signed-off-by: default avatarTom Parkin <tparkin@katalix.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent fb92f57b
...@@ -621,8 +621,8 @@ void l2tp_recv_common(struct l2tp_session *session, struct sk_buff *skb, ...@@ -621,8 +621,8 @@ void l2tp_recv_common(struct l2tp_session *session, struct sk_buff *skb,
int length) int length)
{ {
struct l2tp_tunnel *tunnel = session->tunnel; struct l2tp_tunnel *tunnel = session->tunnel;
u32 ns = 0, nr = 0;
int offset; int offset;
u32 ns, nr;
/* Parse and check optional cookie */ /* Parse and check optional cookie */
if (session->peer_cookie_len > 0) { if (session->peer_cookie_len > 0) {
...@@ -644,7 +644,6 @@ void l2tp_recv_common(struct l2tp_session *session, struct sk_buff *skb, ...@@ -644,7 +644,6 @@ void l2tp_recv_common(struct l2tp_session *session, struct sk_buff *skb,
* the control of the LNS. If no sequence numbers present but * the control of the LNS. If no sequence numbers present but
* we were expecting them, discard frame. * we were expecting them, discard frame.
*/ */
ns = nr = 0;
L2TP_SKB_CB(skb)->has_seq = 0; L2TP_SKB_CB(skb)->has_seq = 0;
if (tunnel->version == L2TP_HDR_VER_2) { if (tunnel->version == L2TP_HDR_VER_2) {
if (hdrflags & L2TP_HDRFLAG_S) { if (hdrflags & L2TP_HDRFLAG_S) {
...@@ -826,7 +825,8 @@ static int l2tp_udp_recv_core(struct l2tp_tunnel *tunnel, struct sk_buff *skb) ...@@ -826,7 +825,8 @@ static int l2tp_udp_recv_core(struct l2tp_tunnel *tunnel, struct sk_buff *skb)
} }
/* Point to L2TP header */ /* Point to L2TP header */
optr = ptr = skb->data; optr = skb->data;
ptr = skb->data;
/* Get L2TP header flags */ /* Get L2TP header flags */
hdrflags = ntohs(*(__be16 *)ptr); hdrflags = ntohs(*(__be16 *)ptr);
......
...@@ -124,7 +124,8 @@ static int l2tp_ip_recv(struct sk_buff *skb) ...@@ -124,7 +124,8 @@ static int l2tp_ip_recv(struct sk_buff *skb)
goto discard; goto discard;
/* Point to L2TP header */ /* Point to L2TP header */
optr = ptr = skb->data; optr = skb->data;
ptr = skb->data;
session_id = ntohl(*((__be32 *)ptr)); session_id = ntohl(*((__be32 *)ptr));
ptr += 4; ptr += 4;
...@@ -153,7 +154,8 @@ static int l2tp_ip_recv(struct sk_buff *skb) ...@@ -153,7 +154,8 @@ static int l2tp_ip_recv(struct sk_buff *skb)
goto discard_sess; goto discard_sess;
/* Point to L2TP header */ /* Point to L2TP header */
optr = ptr = skb->data; optr = skb->data;
ptr = skb->data;
ptr += 4; ptr += 4;
pr_debug("%s: ip recv\n", tunnel->name); pr_debug("%s: ip recv\n", tunnel->name);
print_hex_dump_bytes("", DUMP_PREFIX_OFFSET, ptr, length); print_hex_dump_bytes("", DUMP_PREFIX_OFFSET, ptr, length);
...@@ -284,8 +286,10 @@ static int l2tp_ip_bind(struct sock *sk, struct sockaddr *uaddr, int addr_len) ...@@ -284,8 +286,10 @@ static int l2tp_ip_bind(struct sock *sk, struct sockaddr *uaddr, int addr_len)
chk_addr_ret != RTN_MULTICAST && chk_addr_ret != RTN_BROADCAST) chk_addr_ret != RTN_MULTICAST && chk_addr_ret != RTN_BROADCAST)
goto out; goto out;
if (addr->l2tp_addr.s_addr) if (addr->l2tp_addr.s_addr) {
inet->inet_rcv_saddr = inet->inet_saddr = addr->l2tp_addr.s_addr; inet->inet_rcv_saddr = addr->l2tp_addr.s_addr;
inet->inet_saddr = addr->l2tp_addr.s_addr;
}
if (chk_addr_ret == RTN_MULTICAST || chk_addr_ret == RTN_BROADCAST) if (chk_addr_ret == RTN_MULTICAST || chk_addr_ret == RTN_BROADCAST)
inet->inet_saddr = 0; /* Use device */ inet->inet_saddr = 0; /* Use device */
......
...@@ -137,7 +137,8 @@ static int l2tp_ip6_recv(struct sk_buff *skb) ...@@ -137,7 +137,8 @@ static int l2tp_ip6_recv(struct sk_buff *skb)
goto discard; goto discard;
/* Point to L2TP header */ /* Point to L2TP header */
optr = ptr = skb->data; optr = skb->data;
ptr = skb->data;
session_id = ntohl(*((__be32 *)ptr)); session_id = ntohl(*((__be32 *)ptr));
ptr += 4; ptr += 4;
...@@ -166,7 +167,8 @@ static int l2tp_ip6_recv(struct sk_buff *skb) ...@@ -166,7 +167,8 @@ static int l2tp_ip6_recv(struct sk_buff *skb)
goto discard_sess; goto discard_sess;
/* Point to L2TP header */ /* Point to L2TP header */
optr = ptr = skb->data; optr = skb->data;
ptr = skb->data;
ptr += 4; ptr += 4;
pr_debug("%s: ip recv\n", tunnel->name); pr_debug("%s: ip recv\n", tunnel->name);
print_hex_dump_bytes("", DUMP_PREFIX_OFFSET, ptr, length); print_hex_dump_bytes("", DUMP_PREFIX_OFFSET, ptr, length);
......
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