summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSamuel Jero <sj323707@ohio.edu>2010-12-30 12:15:16 +0100
committerGerrit Renker <gerrit@erg.abdn.ac.uk>2011-01-07 12:22:43 +0100
commit2cf5be93d1b704f342ad423a49f0e78d73939e66 (patch)
tree5d0e0435c71e775134d0e2374c97db78a46d3198
parentcassini: Use local-mac-address prom property for Cassini MAC address (diff)
downloadlinux-2cf5be93d1b704f342ad423a49f0e78d73939e66.tar.xz
linux-2cf5be93d1b704f342ad423a49f0e78d73939e66.zip
dccp: fix return value for sequence-invalid packets
Currently dccp_check_seqno returns 0 (indicating a valid packet) if the acknowledgment number is out of bounds and the sync that RFC 4340 mandates at this point is currently being rate-limited. This function should return -1, indicating an invalid packet. Signed-off-by: Samuel Jero <sj323707@ohio.edu> Acked-by: Gerrit Renker <gerrit@erg.abdn.ac.uk>
-rw-r--r--net/dccp/input.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/net/dccp/input.c b/net/dccp/input.c
index 15af247ea007..8cde009e8b85 100644
--- a/net/dccp/input.c
+++ b/net/dccp/input.c
@@ -260,7 +260,7 @@ static int dccp_check_seqno(struct sock *sk, struct sk_buff *skb)
*/
if (time_before(now, (dp->dccps_rate_last +
sysctl_dccp_sync_ratelimit)))
- return 0;
+ return -1;
DCCP_WARN("Step 6 failed for %s packet, "
"(LSWL(%llu) <= P.seqno(%llu) <= S.SWH(%llu)) and "