diff options
author | Florian Westphal <fw@strlen.de> | 2010-06-02 20:43:44 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-06-05 05:23:14 -0400 |
commit | af9b4738574b46025de7ccbe75c7b24fd8914379 (patch) | |
tree | 6d05b82261ce76e6ecf00131c667292b62d16adf /net/ipv6/syncookies.c | |
parent | 2a1d4bd46047efff513600d7ff422bc344f540a6 (diff) |
syncookies: avoid unneeded tcp header flag double check
caller: if (!th->rst && !th->syn && th->ack)
callee: if (!th->ack)
make the caller only check for !syn (common for 3whs), and move
the !rst / ack test to the callee.
Signed-off-by: Florian Westphal <fw@strlen.de>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv6/syncookies.c')
-rw-r--r-- | net/ipv6/syncookies.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/ipv6/syncookies.c b/net/ipv6/syncookies.c index 12383705dbad..9fcb3ec7c838 100644 --- a/net/ipv6/syncookies.c +++ b/net/ipv6/syncookies.c | |||
@@ -174,7 +174,7 @@ struct sock *cookie_v6_check(struct sock *sk, struct sk_buff *skb) | |||
174 | struct dst_entry *dst; | 174 | struct dst_entry *dst; |
175 | __u8 rcv_wscale; | 175 | __u8 rcv_wscale; |
176 | 176 | ||
177 | if (!sysctl_tcp_syncookies || !th->ack) | 177 | if (!sysctl_tcp_syncookies || !th->ack || th->rst) |
178 | goto out; | 178 | goto out; |
179 | 179 | ||
180 | if (tcp_synq_no_recent_overflow(sk) || | 180 | if (tcp_synq_no_recent_overflow(sk) || |