BUG in tcp_timer.c:tcp_retransmit_timer()

From: Nagendra Singh Tomar
Date: Mon Mar 29 2004 - 04:09:50 EST

While reading the code of tcp_retransmit_timer(), I came across something
which looks liks a BUG.

The following line

if (tcp_retransmit_skb(sk, skb_peek(&sk->write_queue)) > 0)

should correctly read as

if (tcp_retransmit_skb(sk, skb_peek(&sk->write_queue)) < 0)

Note the check for error condition errorneously done as '> 0'

The bad effects that could be introduced due to this BUG, might be lesser
number of retransmissions (actual retransmissions on the wire) before the
TCP timeout. In the (unlikely) worst case we might end up timing out the
connection without a single retransmission on the wire,


PS: Please copy to my email-id as I am not able to access the list,

-- You have moved the mouse. Windows must be restarted for the
changes to take effect.

To unsubscribe from this list: send the line "unsubscribe linux-net" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html