Beedle - Oracle free perpetual lending

Summary

Vulnerability Details

block.timestamp and block.number are added to event information by default so adding them manually wastes gas

There are 4 instances of this issue:

• Borrowed(msg.sender,pool.lender,loans.length - 1,debt,collateral,pool.interestRate,block.timestamp) remove default fields(block.timestamp or block.number) in event to save gas.

• Borrowed(loan.borrower,pool.lender,loanId,loans[loanId].debt,loans[loanId].collateral,pool.interestRate,block.timestamp) remove default fields(block.timestamp or block.number) in event to save gas.

• AuctionStart(loan.borrower,loan.lender,loanId,loan.debt,loan.collateral,block.timestamp,loan.auctionLength) remove default fields(block.timestamp or block.number) in event to save gas.

• Borrowed(msg.sender,pool.lender,loanId,debt,collateral,pool.interestRate,block.timestamp) remove default fields(block.timestamp or block.number) in event to save gas.

Impact

Tools Used

Recommendations

Remove default fields(block.timestamp or block.number) in event.