DeFiHardhatFoundry
250,000 USDC
View results
Submission Details
Severity: high
Valid

ReseedSilo .sol forgets to set Roots balances to users and protocol during migration

Summary

ReseedSilo.reseedSiloDeposit() completely misses functionality regarding Roots. It means Beanstalk state will be corrupted and all the users lose Roots after migration.

Roots is core concept of Silo module described in docs:
https://docs.bean.money/developers/misc/faq#what-are-roots

Impact

Beanstalk state will be corrupted and all the users lose Roots after migration.

Tools Used

Manual Review

Recommendations

Add missing functionality regarding Roots to ReseedSilo.

Updates

Lead Judging Commences

inallhonesty Lead Judge about 1 year ago
Submission Judgement Published
Validated
Assigned finding tags:

`ReseedSilo#reseedSiloDeposit` does not credit the user any `roots`

Support

FAQs

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