code-423n4 / 2021-09-wildcredit-findings

0 stars 0 forks source link

Consider adding `account` parameter to event WithdrawUniPosition #51

Open code423n4 opened 3 years ago

code423n4 commented 3 years ago

Handle

WatchPug

Vulnerability details

https://github.com/code-423n4/2021-09-wildcredit/blob/main/contracts/LendingPair.sol#L121

While the event DepositUniPosition has a parameter _account, the WithdrawUniPosition event doesn't have one.

Consider adding account parameter to event WithdrawUniPosition for consistency and easier off-chain monitoring.