Skip to content

Improve security of Comptroller#_supportMarket - #186

Open
TRiLeZ wants to merge 2 commits into
compound-finance:masterfrom
TRiLeZ:support-market-cToken-admin-check
Open

TRiLeZ wants to merge 2 commits into
compound-finance:masterfrom
TRiLeZ:support-market-cToken-admin-check

Conversation

@TRiLeZ

@TRiLeZ TRiLeZ commented Feb 3, 2022

Copy link
Copy Markdown
Contributor

Adds an additional check to ensure that the admin of the cToken is the same as the admin of the Comptroller when supporting a new market.

Test status: in-progress

- Adds an additional check to ensure that the admin of the cToken is the same as the admin of the Comptroller.
- Oops forgot the brackets
@TRiLeZ

TRiLeZ commented Feb 3, 2022

Copy link
Copy Markdown
Contributor Author

All existing test cases pass!

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

Successfully merging this pull request may close these issues.

1 participant