| From 6cd9e8e15750b91f58bf4a1035256973d293c3a4 Mon Sep 17 00:00:00 2001 |
| From: Eric Dumazet <edumazet@google.com> |
| Date: Mon, 1 May 2017 15:29:48 -0700 |
| Subject: [PATCH] tcp: fix wraparound issue in tcp_lp |
| |
| commit a9f11f963a546fea9144f6a6d1a307e814a387e7 upstream. |
| |
| Be careful when comparing tcp_time_stamp to some u32 quantity, |
| otherwise result can be surprising. |
| |
| Fixes: 7c106d7e782b ("[TCP]: TCP Low Priority congestion control") |
| Signed-off-by: Eric Dumazet <edumazet@google.com> |
| Signed-off-by: David S. Miller <davem@davemloft.net> |
| Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com> |
| |
| diff --git a/net/ipv4/tcp_lp.c b/net/ipv4/tcp_lp.c |
| index c67ece1390c2..7d86fc505397 100644 |
| --- a/net/ipv4/tcp_lp.c |
| +++ b/net/ipv4/tcp_lp.c |
| @@ -264,13 +264,15 @@ static void tcp_lp_pkts_acked(struct sock *sk, const struct ack_sample *sample) |
| { |
| struct tcp_sock *tp = tcp_sk(sk); |
| struct lp *lp = inet_csk_ca(sk); |
| + u32 delta; |
| |
| if (sample->rtt_us > 0) |
| tcp_lp_rtt_sample(sk, sample->rtt_us); |
| |
| /* calc inference */ |
| - if (tcp_time_stamp > tp->rx_opt.rcv_tsecr) |
| - lp->inference = 3 * (tcp_time_stamp - tp->rx_opt.rcv_tsecr); |
| + delta = tcp_time_stamp - tp->rx_opt.rcv_tsecr; |
| + if ((s32)delta > 0) |
| + lp->inference = 3 * delta; |
| |
| /* test if within inference */ |
| if (lp->last_drop && (tcp_time_stamp - lp->last_drop < lp->inference)) |
| -- |
| 2.12.0 |
| |