Skip to content
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

s0x0mtee - Blacklisted addresses can still call/interact with Telcoin core/swap functions #232

Open
sherlock-admin2 opened this issue Nov 13, 2024 · 0 comments

Comments

@sherlock-admin2
Copy link
Contributor

sherlock-admin2 commented Nov 13, 2024

s0x0mtee

Medium

Blacklisted addresses can still call/interact with Telcoin core/swap functions

Summary

Stablecoin.sol enforeces blacklist functionality by restricting the _update() for users who have been added to blacklist hence any transaction to and from their address will be restricted, but blacklisted users can still make calls to swap functions such as stablecoinSwap() with the help of delegate calls through contracts that have been funded with eXYZ tokens or other DeFi tokens. The blacklisted wallets will be able to by pass the onlyRole(SWAPPER_ROLE) since the modifer uses msg.sender(which would be the blacklisted address and not the external delegate contract) to check if addresses calling these functions are allowed to, which blacklisted addresses still have.https://github.com/sherlock-audit/2024-11-telcoin/blob/b9c751b59e78a7123a636e31ecafc9147046f190/telcoin-audit/contracts/stablecoin/StablecoinHandler.sol#L144-L181

Root Cause

  • Lack of checks for calls made by blacklisted addresses in each swap function.

Internal pre-conditions

  1. Blacklisted address uses an external funded address to make delegate calls to Telcoins functions

External pre-conditions

No response

Attack Path

No response

Impact

The blacklisted address can still carry out nefarious activities in the telcoin ecosystem

PoC

No response

Mitigation

No response

@sherlock-admin3 sherlock-admin3 changed the title Sour Foggy Quail - Blacklisted addresses can still call/interact with Telcoin core/swap functions s0x0mtee - Blacklisted addresses can still call/interact with Telcoin core/swap functions Nov 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant