code-423n4 / 2021-04-marginswap-findings

1 stars 0 forks source link

No function for TOKEN_ADMIN in RoleAware.sol #11

Open code423n4 opened 3 years ago

code423n4 commented 3 years ago

Email address

mail@gpersoon.com

Handle

gpersoon

Eth address

gpersoon.eth

Vulnerability details

This is a minor suggestion.

RoleAware.sol contains functions for most of the constants. However the one exception is TOKEN_ADMIN.

Impact

The code is slightly longer than necessary.

Recommended mitigation steps

Remove the constant TOKEN_ADMIN, or provide a comment why it isn't used.

zscole commented 3 years ago

This submission has been judged as non-critical since this has no impact on the security or functionality of the contract.