stake.link

stake.link
DeFiHardhatBridge
27,500 USDC
View results
Submission Details
Severity: low
Invalid

The unnecessary use of the `onlyOwner` modifier

Summary

Based on the description of the WrappedTokenBridge contract, all the bridge functions inside are intended for users. There are two functions with token bridging functionality: onTokenTransfer() and transferTokens(). The transferTokens() function differs from onTokenTransfer() only in its ability to pay bridge fees in native tokens, suggesting that this functionality may not necessarily need owner administration.

Impact

The purpose of the transferTokens() function is for anyone to use, but with the onlyOwner modifier, it is restricted to only the owner being able to use it.

Recommendations

Remove the onlyOwner modifier.

Updates

Lead Judging Commences

0kage Lead Judge over 1 year ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

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