NFTBridge
60,000 USDC
View results
Submission Details
Severity: low
Invalid

isWhitelisted function in bridge.sol returns true for all collections if globally

Code Links

https://github.com/Cyfrin/2024-07-ark-project/blob/main/apps/blockchain/ethereum/src/Bridge.sol#L334-L338

Summary

the _isWhitelisted() function is meant to check if a collection is whitelisted or not and return true or false

Vulnerability Details

The whiteListEnabled variable is used to indicate whether the whitelisting is enabled globally or not, when it is set to false it means the whitelisting is disabled globally. Hiwever when it is disabled the _isWhitelisted returns true

Impact

The function returns true, even if the collection is not whitelisted when the _whiteListEnabled variable is false. This allows even non whitelisted collections

Tools Used

Manual Review

Recommendations

Updates

Lead Judging Commences

n0kto Lead Judge 10 months ago
Submission Judgement Published
Invalidated
Reason: Design choice

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.