Skip to content

Commit

Permalink
chore(security): slither annotations
Browse files Browse the repository at this point in the history
  • Loading branch information
vzctl committed Dec 21, 2022
1 parent 9fa519c commit 2b662ce
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 0 deletions.
3 changes: 3 additions & 0 deletions etc/eth-contracts/contracts/AdminControlled.sol
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ contract AdminControlled {
uint public paused;

constructor(address _admin, uint flags) {
// slither-disable-next-line missing-zero-check
admin = _admin;

// Add the possibility to set pause flags on the initialization
Expand Down Expand Up @@ -34,12 +35,14 @@ contract AdminControlled {
}

function adminSendEth(address payable destination, uint amount) public onlyAdmin {
// slither-disable-next-line missing-zero-check
destination.transfer(amount);
}

function adminReceiveEth() public payable onlyAdmin {}

function adminDelegatecall(address target, bytes memory data) public payable onlyAdmin returns (bytes memory) {
// slither-disable-next-line controlled-delegatecall,low-level-calls,missing-zero-check
(bool success, bytes memory rdata) = target.delegatecall(data);
require(success);
return rdata;
Expand Down
2 changes: 2 additions & 0 deletions etc/eth-contracts/contracts/EvmErc20.sol
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ contract EvmErc20 is ERC20, AdminControlled, IExit {
string private _symbol;
uint8 private _decimals;

// slither-disable-next-line shadowing-local
constructor (string memory metadata_name, string memory metadata_symbol, uint8 metadata_decimals, address admin)
ERC20(metadata_name, metadata_symbol)
AdminControlled(admin, 0)
Expand All @@ -38,6 +39,7 @@ contract EvmErc20 is ERC20, AdminControlled, IExit {
return _decimals;
}

// slither-disable-next-line events-maths
function setMetadata(string memory metadata_name, string memory metadata_symbol, uint8 metadata_decimals) external onlyAdmin {
_name = metadata_name;
_symbol = metadata_symbol;
Expand Down
2 changes: 2 additions & 0 deletions etc/eth-contracts/contracts/EvmErc20V2.sol
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ contract EvmErc20V2 is ERC20, AdminControlled, IExit {
string private _symbol;
uint8 private _decimals;

// slither-disable-next-line shadowing-local
constructor (string memory metadata_name, string memory metadata_symbol, uint8 metadata_decimals, address admin)
ERC20(metadata_name, metadata_symbol)
AdminControlled(admin, 0)
Expand All @@ -38,6 +39,7 @@ contract EvmErc20V2 is ERC20, AdminControlled, IExit {
return _decimals;
}

// slither-disable-next-line events-maths
function setMetadata(string memory metadata_name, string memory metadata_symbol, uint8 metadata_decimals) external onlyAdmin {
_name = metadata_name;
_symbol = metadata_symbol;
Expand Down

0 comments on commit 2b662ce

Please sign in to comment.