Closed code423n4 closed 2 years ago
pants
These functions contain require statement with messages longer than 32 bytes, meaning they cant get into bytes32 which will save gas:
OverlayTokenNew._approve()
When transactions revert, the require will be forced to use bytes instead of bytes32 and will waste more gas.
Manual code review.
Reduce the message by a few letters to fit it into 32 chars.
duplicate #65
mikeyrf
commented
2 years ago mikeyrf
commented
2 years ago
Handle
pants
Vulnerability details
Vulnerability details
These functions contain require statement with messages longer than 32 bytes, meaning they cant get into bytes32 which will save gas:
OverlayTokenNew._approve()line 403, 404Impact
When transactions revert, the require will be forced to use bytes instead of bytes32 and will waste more gas.
Tool Used
Manual code review.
Recommended Mitigation Steps
Reduce the message by a few letters to fit it into 32 chars.