require should come first #93
Labels
bug
Something isn't working
G (Gas Optimization)
sponsor acknowledged
Technically the issue is correct, but we're not going to resolve it for XYZ reasons
Handle
pants
Vulnerability details
Vulnerability details
OverlayToken.sol line 131/180/254:
the require comes after the _transfer function, there is no reason for it to be after the function.
If the require is placed before the _transfer it will save gas if the require will fail.
Tool Used
Manual code review.
Recommended Mitigation Steps
place the _transfer after the require
The text was updated successfully, but these errors were encountered: