Bad collateral check in `SmartCommitmentsForwarder`
Closed this issue · 0 comments
sherlock-admin3 commented
Bad collateral check in SmartCommitmentsForwarder
Low/Info issue submitted by KiroBrejka
Summary
Vulnerability Detail
As CommitmentCollateralType
is an enum, the collateral token type can't be <= CommitmentCollateralType.ERC1155_ANY_ID
as it is right now
Impact
Code Snippet
Tool used
Manual Review
Recommendation
Change the <=
to !=