[coap] fix order of operations in TxParameters::IsValid() (#6027)

To match CalculateInitialRetransmissionTimeout() implementation.
This commit is contained in:
Jonathan Hui
2021-01-05 09:35:45 -08:00
committed by GitHub
parent df047ed5cb
commit f64c597fc6
+1 -1
View File
@@ -950,8 +950,8 @@ bool TxParameters::IsValid(void) const
// Calulate exchange lifetime step by step and verify no overflow.
uint32_t tmp = Multiply(mAckTimeout, (1U << (mMaxRetransmit + 1)) - 1);
tmp /= mAckRandomFactorDenominator;
tmp = Multiply(tmp, mAckRandomFactorNumerator);
tmp /= mAckRandomFactorDenominator;
rval = (tmp != 0 && (tmp + mAckTimeout + 2 * kDefaultMaxLatency) > tmp);
}