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,
Thanx,
tomar
PS: Please copy to my email-id as I am not able to access the list,
currently.
-- You have moved the mouse. Windows must be restarted for the
changes to take effect.
-
: send the line "unsubscribe linux-net" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
