From ced5f529710be4323b1f3b0838238a696ce8da35 Mon Sep 17 00:00:00 2001 From: Angus Antley Date: Thu, 2 May 2019 14:22:52 -0700 Subject: [PATCH] fixed possible divide by zero error in setpacketsendperiod in CongestionControl.cpp --- libraries/networking/src/udt/CongestionControl.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libraries/networking/src/udt/CongestionControl.cpp b/libraries/networking/src/udt/CongestionControl.cpp index c0ad89e804..5661f3ab83 100644 --- a/libraries/networking/src/udt/CongestionControl.cpp +++ b/libraries/networking/src/udt/CongestionControl.cpp @@ -29,7 +29,7 @@ void CongestionControl::setMaxBandwidth(int maxBandwidth) { void CongestionControl::setPacketSendPeriod(double newSendPeriod) { Q_ASSERT_X(newSendPeriod >= 0, "CongestionControl::setPacketPeriod", "Can not set a negative packet send period"); - auto packetsPerSecond = (double)_maxBandwidth / (BITS_PER_BYTE * _mss); + auto packetsPerSecond = _mss > 0.0f ? (double)_maxBandwidth / (BITS_PER_BYTE * _mss) : -1.0f; if (packetsPerSecond > 0.0) { // anytime the packet send period is about to be increased, make sure it stays below the minimum period, // calculated based on the maximum desired bandwidth