Beedle - Oracle free perpetual lending

Summary

Vulnerability Details

You can cut out 10 opcodes in the creation-time EVM bytecode
if you declare a constructor payable.
Making the constructor payable eliminates the need for an initial check
of msg.value == 0 and saves 13 gas on deployment with no security risks.

There are 5 instances of this issue:

• Beedle.constructor() should be set to payable

• Ownable.constructor(address) should be set to payable

• Fees.constructor(address,address) should be set to payable

• Staking.constructor(address,address) should be set to payable

• Lender.constructor() should be set to payable

Impact

Tools Used

Recommendations

Set the constructor to payable.