Closed code423n4 closed 2 years ago
pauliax
Using the 'unchecked' keyword to avoid redundant arithmetic underflow/overflow checks to save gas when an underflow/overflow cannot happen. Unchecked can be applied here:
if (currentAllowance < amount) { // ... token.safeIncreaseAllowance(spender, amount - currentAllowance); }
Consider using unchecked maths where gas costs are important and overflow/underflow are not possible.
Handle
pauliax
Vulnerability details
Impact
Using the 'unchecked' keyword to avoid redundant arithmetic underflow/overflow checks to save gas when an underflow/overflow cannot happen. Unchecked can be applied here:
if (currentAllowance < amount) { // ... token.safeIncreaseAllowance(spender, amount - currentAllowance); }
Recommended Mitigation Steps
Consider using unchecked maths where gas costs are important and overflow/underflow are not possible.