Use a two-step ownership transfer approach
Summary
Use a two-step ownership transfer approach
Vulnerability Details
There is method with the onlyOwner modifier in Beedle.sol and Staking.sol also uses Ownalbe library which shows that the owner role is an important one.Make sure to use a two-step ownership transfer approach by using Ownable2Step from OpenZeppelin as opposed to Ownable as it gives you the security of not unintentionally sending the owner role to an address you do not control.
setFeeReceiver function in Lender.sol also uses single step tranfer of feeReceiver role.Use two-step transfer for important roles.
Impact
Low
Tools Used
Recommendations
use a two-step onwership transfer approach by using Owanble2Step from OpenZeppelin as opposed to Ownable.
Prize pool breakdown
Live
The contest is live. Earn rewards by submitting a finding.
Appeals
This is your time to appeal against judgements on your submissions.
Appeals Review
Appeals are being carefully reviewed by our judges.