Summary

https://github.com/Cyfrin/2024-05-beanstalk-the-finale/blob/4e0ad0b964f74a1b4880114f4dd5b339bc69cd3e/protocol/contracts/beanstalk/barn/FertilizerFacet.sol#L47

functions like claimFertilized should have nonReenter guard

as we never know how safeTransfer will act especially that it might trigger return bomb, reenter atack in malicious implemented whitelisted token or wallet smartcontract.

Vulnerability Details

Withdraw funds with noReentry guard implemented.

Impact

Can be opening for malicious actions in protocol.

Tools Used

slither, code analyze.

Recommendations

add modifier nonReentry() to claimFerilized