You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This mitigation succesfully mitigates the original issue. The address of the token is used, which will succesfully get the underlying token amount from the amount of shares + queued withdrawal shares.
Suggestion
n/a
Conclusion
LGTM
The text was updated successfully, but these errors were encountered:
Lines of code
Vulnerability details
Original Issue Summary
The
OperatorDelegator.getTokenBalanceFromStrategy()
incorrectly usesaddress(this)
to check for the queued amount instead ofaddress(token)
.address(this)
will always return0
, which means thatqueuedShares[address(token)]
will not be accounted for.Mitigation
This mitigation proposes the usage of
address(token)
instead ofaddress(this)
:Comments
This mitigation succesfully mitigates the original issue. The address of the token is used, which will succesfully get the underlying token amount from the amount of shares + queued withdrawal shares.
Suggestion
n/a
Conclusion
LGTM
The text was updated successfully, but these errors were encountered: