There is no way for MANAGER to remove an asset from supportedAssets.
Proof of Concept
MANAGER adds a new supported asset to the protocol (let's say asset X).
If something unexpected happens to the asset X (for example eigenlayer governance decides to remove an asset), MANAGER is not able to remove that asset (asset X) from supportedAssets.
Tools Used
Manual Review
Recommended Mitigation Steps
Consider adding a function that MANAGER is able to remove some asset from supportedAssets.
Lines of code
https://github.com/code-423n4/2023-11-kelp/blob/c5fdc2e62c5e1d78769f44d6e34a6fb9e40c00f0/src/LRTConfig.sol#L80-L89 https://github.com/code-423n4/2023-11-kelp/blob/c5fdc2e62c5e1d78769f44d6e34a6fb9e40c00f0/src/LRTConfig.sol#L73-L75
Vulnerability details
Impact
There is no way for MANAGER to remove an asset from supportedAssets.
Proof of Concept
X
).X
(for example eigenlayer governance decides to remove an asset), MANAGER is not able to remove that asset (assetX
) from supportedAssets.Tools Used
Manual Review
Recommended Mitigation Steps
Consider adding a function that MANAGER is able to remove some asset from supportedAssets.
Assessed type
Context