Beginner FriendlyFoundryDeFi
100 EXP
View results
Submission Details
Severity: low
Invalid

Should check weth address when deploying Steaking contract.

Summary

Should check weth address when deploying Steaking contract.

Vulnerability Details

As weth address is immutable, if the owner inputs the wrong address as weth, the users's eth will stuck in this steaking protocol when use call depositIntoVault.

Impact

  1. Users' s eth will stuck in this steaking protocol when use call depositIntoVault.

  2. The owner no ways dealing with this issues

Tools Used

Manual

Recommendations

When deploy the contract, check the _weth address. If want more more security, can add one funciton the owner can set the WETH address.

@deploy
def __init__(_weth: address):
assert _weth != ADDRESS_ZERO, STEAK__ADDRESS_ZERO
WETH = _weth
self.startTimestamp = block.timestamp
self.owner = msg.sender
Updates

Lead Judging Commences

inallhonesty Lead Judge 11 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

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