Project
One World
NFTDeFi
15,000 USDC
View results
Previous Next
Submission Details
Severity: low
Invalid

[M-5] lack of check of return value in addCurrency::CurrencyManager and removeCurrency::CurrencyManager

oluwaseyisekoni

Summary

CurrencyManager.addCurrency::CurrencyManager.sol ignores return value by _whitelistedCurrencies.add(currency)
CurrencyManager.removeCurrency::CurrencyManager.sol ignores return value by _whitelistedCurrencies.remove(currency)::CurrencyManager.sol

Impact

Unsuccessful Removal or addition goes undetected

Tools Used

Manual review

Recommendations

return values of _whitelistedCurrencies.remove(currency)and _whitelistedCurrencies.add(currency)should be checked

Updates

Lead Judging Commences

0xbrivan2 Lead Judge 9 months ago
Submission Judgement Published
Invalidated
Reason: Incorrect statement
0xbrivan2 Lead Judge 9 months ago
Submission Judgement Published
Invalidated
Reason: Incorrect statement

Support

FAQs

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