Skip to content
This repository has been archived by the owner on Nov 26, 2023. It is now read-only.

mahdikarimi - removed collateral can't be accessible #911

Closed
sherlock-admin opened this issue May 24, 2023 · 0 comments
Closed

mahdikarimi - removed collateral can't be accessible #911

sherlock-admin opened this issue May 24, 2023 · 0 comments
Labels
Duplicate A valid issue that is a duplicate of an issue with `Has Duplicates` label Medium A valid Medium severity issue Reward A payout will be made for this issue

Comments

@sherlock-admin
Copy link
Contributor

sherlock-admin commented May 24, 2023

mahdikarimi

medium

removed collateral can't be accessible

Summary

If admin remove a collateral , the collateral can't be accessible anymore .

Vulnerability Detail

Since the collateral will be removed from CollateralInfo array so it can't be used for re-balancing therefore removing collateral leads to just ignoring it while re-balancing , while there should be a mechanism to swap it to other collaterals .

Impact

removed collateral won't be accessible

Code Snippet

https://github.com/sherlock-audit/2023-05-USSD/blob/main/ussd-contracts/contracts/USSD.sol#L120-L123

Tool used

Manual Review

Recommendation

Add a mechanism to swap collateral to another allowed collateral after removing .

Duplicate of #341

@github-actions github-actions bot closed this as completed Jun 5, 2023
@github-actions github-actions bot added Medium A valid Medium severity issue Duplicate A valid issue that is a duplicate of an issue with `Has Duplicates` label labels Jun 5, 2023
@sherlock-admin sherlock-admin added the Reward A payout will be made for this issue label Jun 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Duplicate A valid issue that is a duplicate of an issue with `Has Duplicates` label Medium A valid Medium severity issue Reward A payout will be made for this issue
Projects
None yet
Development

No branches or pull requests

1 participant