diff options
author | Ilpo Järvinen <ilpo.jarvinen@helsinki.fi> | 2007-03-28 15:06:37 -0400 |
---|---|---|
committer | David S. Miller <davem@sunset.davemloft.net> | 2007-10-10 19:47:51 -0400 |
commit | 19b2b486580f5939688d3e225acdc0f4b291ed0d (patch) | |
tree | 7d6803d9decb958769344a28b5ea8e8d65c0d9ca /net/ipv4/tcp_input.c | |
parent | d8f4f2235abc7b30cf447ca3e22ac28326b12f28 (diff) |
[TCP]: Rexmit hint must be cleared instead of setting it
Stupid error from my side. Even though now that I noticed this,
I hoped it would have been an optimization but no, the counter
hint is then incorrect. Thus clearing is necessary for now (I
still suspect though that this path is never executed).
Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@helsinki.fi>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4/tcp_input.c')
-rw-r--r-- | net/ipv4/tcp_input.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/ipv4/tcp_input.c b/net/ipv4/tcp_input.c index 7d843c429381..0aa17243d369 100644 --- a/net/ipv4/tcp_input.c +++ b/net/ipv4/tcp_input.c | |||
@@ -1776,7 +1776,7 @@ static void tcp_verify_retransmit_hint(struct tcp_sock *tp, | |||
1776 | if ((tp->retransmit_skb_hint != NULL) && | 1776 | if ((tp->retransmit_skb_hint != NULL) && |
1777 | before(TCP_SKB_CB(skb)->seq, | 1777 | before(TCP_SKB_CB(skb)->seq, |
1778 | TCP_SKB_CB(tp->retransmit_skb_hint)->seq)) | 1778 | TCP_SKB_CB(tp->retransmit_skb_hint)->seq)) |
1779 | tp->retransmit_skb_hint = skb; | 1779 | tp->retransmit_skb_hint = NULL; |
1780 | } | 1780 | } |
1781 | 1781 | ||
1782 | /* Mark head of queue up as lost. */ | 1782 | /* Mark head of queue up as lost. */ |