code-423n4 / 2021-04-maple-findings

0 stars 0 forks source link

Fields stuck after initialization #80

Closed code423n4 closed 3 years ago

code423n4 commented 3 years ago

Handle

a_delamo

Vulnerability details

Impact

If I am right, the impact could be middle or high.

Proof of Concept

Here you have a gist: https://gist.github.com/alexon1234/4320a1806f3071b0279e8ca46288b1f0

Tools Used

Vscode

Recommended Mitigation Steps

In order to fix the issue, we need to fetch the data from the loan wherever we need it in DebtLocker

lucas-manuel commented 3 years ago

Not a bug, this code works as intended. We want to only capture those values once, when they get updated to a non-zero value the first time.