UNI-1017 Loan can be written off by anybody before overdue delay expires #64
Conversation
contracts/market/UToken.sol
Outdated
| accountBorrows[borrower].lastRepay = 0; | ||
| } else { | ||
| // Still tracking the exisiting loan. | ||
| accountBorrows[borrower].lastRepay = getBlockNumber(); |
There was a problem hiding this comment.
I don't think accountBorrows[borrower].lastRepay = getBlockNumber() should be set without liquidation being complete. Because the borrower still has debts and he has not made repayments, his behavior is still a bad borrower. This will cause other guarantors to be unable to liquidate, because the test is no longer in default
There was a problem hiding this comment.
I thought the staker could write off a small amount to re-validate the borrower's loan. If that's not the case, I don't know why anyone would want to just write off part of the loan.
There was a problem hiding this comment.
I mean when there are multiple stakers and one of them liquidates their vouch, other stakers can no longer liquidate their vouch because lastRepay is updated
There was a problem hiding this comment.
Ok, that makes sense. I've updated it.
…/uni-1017-115-loan-can-be-written-off-by-anybody
maxweng
deleted the
findings/uni-1017-115-loan-can-be-written-off-by-anybody
branch
No description provided.