unchecked return value from approve() #233
Labels
1 (Low Risk)
Assets are not at risk. State handling, function incorrect as to spec, issues with comments
bug
Something isn't working
duplicate
This issue or pull request already exists
Handle
JMukesh
Vulnerability details
Impact
due to lack of check for the return value of approve() , function may silently fail
Proof of Concept
https://github.com/code-423n4/2021-12-amun/blob/98f6e2ff91f5fcebc0489f5871183566feaec307/contracts/basket/contracts/singleJoinExit/SingleNativeTokenExitV2.sol#L55
Tools Used
manual review
Recommended Mitigation Steps
use safeApprove() from oz library
The text was updated successfully, but these errors were encountered: