Merge pull request #476 from smoltcp-rs/fix-overflow
tcp rtte: fix "attempt to multiply with overflow". Fixes #468
This commit is contained in:
commit
74b124f4c9
|
@ -140,7 +140,7 @@ impl RttEstimator {
|
||||||
// all packets sent would incur a retransmit. To avoid this, force an estimate
|
// all packets sent would incur a retransmit. To avoid this, force an estimate
|
||||||
// increase if we see 3 consecutive retransmissions without any successful sample.
|
// increase if we see 3 consecutive retransmissions without any successful sample.
|
||||||
self.rto_count = 0;
|
self.rto_count = 0;
|
||||||
self.rtt *= 2;
|
self.rtt = RTTE_MAX_RTO.min(self.rtt*2);
|
||||||
let rto = self.retransmission_timeout().millis();
|
let rto = self.retransmission_timeout().millis();
|
||||||
net_trace!("rtte: too many retransmissions, increasing: rtt={:?} dev={:?} rto={:?}", self.rtt, self.deviation, rto);
|
net_trace!("rtte: too many retransmissions, increasing: rtt={:?} dev={:?} rto={:?}", self.rtt, self.deviation, rto);
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue