-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Lack of incentive to liquidate small positions #916
Labels
2 (Med Risk)
Assets not at direct risk, but function/availability of the protocol could be impacted or leak value
bug
Something isn't working
duplicate-175
🤖_11_group
AI based duplicate group recommendation
satisfactory
satisfies C4 submission criteria; eligible for awards
sufficient quality report
This report is of sufficient quality
Comments
JustDravee marked the issue as duplicate of #1258 |
JustDravee marked the issue as sufficient quality report |
koolexcrypto changed the severity to QA (Quality Assurance) |
koolexcrypto marked the issue as grade-c |
This previously downgraded issue has been upgraded by koolexcrypto |
koolexcrypto marked the issue as satisfactory |
koolexcrypto marked the issue as duplicate of #175 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
2 (Med Risk)
Assets not at direct risk, but function/availability of the protocol could be impacted or leak value
bug
Something isn't working
duplicate-175
🤖_11_group
AI based duplicate group recommendation
satisfactory
satisfies C4 submission criteria; eligible for awards
sufficient quality report
This report is of sufficient quality
Lines of code
https://github.com/code-423n4/2024-04-dyad/blob/main/src/core/VaultManagerV2.sol#L26
Vulnerability details
Impact
Ethereum mainnet currently has an average transaction cost of approximately $10,considering that the REWARD RATE is 20%, positions with less than 150 USD in collateral are not profitable to liquidate. This threshold could increase rapidly as transaction costs on Ethereum rise.
Tools Used
Manual review
Recommended Mitigation Steps
Due to the computational complexity of the liquidate function, which increases transaction gas costs, and the fact that funds cannot be withdrawn by the user, adding an option for donations that uses fewer opcodes would be a good choice. However, it is essential to consider potential future vulnerabilities that such a function may introduce regarding health factor concerns.
Assessed type
Other
The text was updated successfully, but these errors were encountered: