Unverified Commit 488c3de3 by Nicolás Venturo Committed by GitHub

Merge pull request #29 from nventuro/storage-slots

Added gaps at the end of the storage of each contract.
parents 6956b935 19de09af
...@@ -46,4 +46,6 @@ contract Initializable { ...@@ -46,4 +46,6 @@ contract Initializable {
assembly { cs := extcodesize(address) } assembly { cs := extcodesize(address) }
return cs == 0; return cs == 0;
} }
uint256[50] private ______gap;
} }
...@@ -44,4 +44,6 @@ contract CapperRole is Initializable { ...@@ -44,4 +44,6 @@ contract CapperRole is Initializable {
cappers.remove(account); cappers.remove(account);
emit CapperRemoved(account); emit CapperRemoved(account);
} }
uint256[50] private ______gap;
} }
...@@ -44,4 +44,6 @@ contract MinterRole is Initializable { ...@@ -44,4 +44,6 @@ contract MinterRole is Initializable {
minters.remove(account); minters.remove(account);
emit MinterRemoved(account); emit MinterRemoved(account);
} }
uint256[50] private ______gap;
} }
...@@ -44,4 +44,6 @@ contract PauserRole is Initializable { ...@@ -44,4 +44,6 @@ contract PauserRole is Initializable {
pausers.remove(account); pausers.remove(account);
emit PauserRemoved(account); emit PauserRemoved(account);
} }
uint256[50] private ______gap;
} }
...@@ -44,4 +44,6 @@ contract SignerRole is Initializable { ...@@ -44,4 +44,6 @@ contract SignerRole is Initializable {
signers.remove(account); signers.remove(account);
emit SignerRemoved(account); emit SignerRemoved(account);
} }
uint256[50] private ______gap;
} }
...@@ -232,4 +232,6 @@ contract Crowdsale is Initializable { ...@@ -232,4 +232,6 @@ contract Crowdsale is Initializable {
function _forwardFunds() internal { function _forwardFunds() internal {
_wallet.transfer(msg.value); _wallet.transfer(msg.value);
} }
uint256[50] private ______gap;
} }
...@@ -46,4 +46,6 @@ contract FinalizableCrowdsale is Initializable, TimedCrowdsale { ...@@ -46,4 +46,6 @@ contract FinalizableCrowdsale is Initializable, TimedCrowdsale {
function _finalization() internal { function _finalization() internal {
} }
uint256[50] private ______gap;
} }
...@@ -48,4 +48,6 @@ contract PostDeliveryCrowdsale is Initializable, TimedCrowdsale { ...@@ -48,4 +48,6 @@ contract PostDeliveryCrowdsale is Initializable, TimedCrowdsale {
_balances[beneficiary] = _balances[beneficiary].add(tokenAmount); _balances[beneficiary] = _balances[beneficiary].add(tokenAmount);
} }
uint256[50] private ______gap;
} }
...@@ -84,4 +84,6 @@ contract RefundableCrowdsale is Initializable, FinalizableCrowdsale { ...@@ -84,4 +84,6 @@ contract RefundableCrowdsale is Initializable, FinalizableCrowdsale {
_escrow.deposit.value(msg.value)(msg.sender); _escrow.deposit.value(msg.value)(msg.sender);
} }
uint256[50] private ______gap;
} }
...@@ -54,4 +54,6 @@ contract AllowanceCrowdsale is Initializable, Crowdsale { ...@@ -54,4 +54,6 @@ contract AllowanceCrowdsale is Initializable, Crowdsale {
{ {
token().safeTransferFrom(_tokenWallet, beneficiary, tokenAmount); token().safeTransferFrom(_tokenWallet, beneficiary, tokenAmount);
} }
uint256[50] private ______gap;
} }
...@@ -27,4 +27,6 @@ contract MintedCrowdsale is Initializable, Crowdsale { ...@@ -27,4 +27,6 @@ contract MintedCrowdsale is Initializable, Crowdsale {
require( require(
ERC20Mintable(address(token())).mint(beneficiary, tokenAmount)); ERC20Mintable(address(token())).mint(beneficiary, tokenAmount));
} }
uint256[50] private ______gap;
} }
...@@ -68,4 +68,6 @@ contract IncreasingPriceCrowdsale is Initializable, TimedCrowdsale { ...@@ -68,4 +68,6 @@ contract IncreasingPriceCrowdsale is Initializable, TimedCrowdsale {
return currentRate.mul(weiAmount); return currentRate.mul(weiAmount);
} }
uint256[50] private ______gap;
} }
...@@ -53,4 +53,6 @@ contract CappedCrowdsale is Initializable, Crowdsale { ...@@ -53,4 +53,6 @@ contract CappedCrowdsale is Initializable, Crowdsale {
require(weiRaised().add(weiAmount) <= _cap); require(weiRaised().add(weiAmount) <= _cap);
} }
uint256[50] private ______gap;
} }
...@@ -81,4 +81,6 @@ contract IndividuallyCappedCrowdsale is Initializable, Crowdsale, CapperRole { ...@@ -81,4 +81,6 @@ contract IndividuallyCappedCrowdsale is Initializable, Crowdsale, CapperRole {
weiAmount); weiAmount);
} }
uint256[50] private ______gap;
} }
...@@ -83,4 +83,6 @@ contract TimedCrowdsale is Initializable, Crowdsale { ...@@ -83,4 +83,6 @@ contract TimedCrowdsale is Initializable, Crowdsale {
super._preValidatePurchase(beneficiary, weiAmount); super._preValidatePurchase(beneficiary, weiAmount);
} }
uint256[50] private ______gap;
} }
...@@ -12,6 +12,8 @@ import "../../token/ERC20/IERC20.sol"; ...@@ -12,6 +12,8 @@ import "../../token/ERC20/IERC20.sol";
*/ */
contract ERC20TokenMetadata is Initializable, IERC20 { contract ERC20TokenMetadata is Initializable, IERC20 {
function tokenURI() external view returns (string); function tokenURI() external view returns (string);
uint256[50] private ______gap;
} }
...@@ -28,4 +30,6 @@ contract ERC20WithMetadata is Initializable, ERC20TokenMetadata { ...@@ -28,4 +30,6 @@ contract ERC20WithMetadata is Initializable, ERC20TokenMetadata {
function tokenURI() external view returns (string) { function tokenURI() external view returns (string) {
return _tokenURI; return _tokenURI;
} }
uint256[50] private ______gap;
} }
...@@ -99,4 +99,6 @@ contract ERC20Migrator is Initializable { ...@@ -99,4 +99,6 @@ contract ERC20Migrator is Initializable {
uint256 amount = Math.min(balance, allowance); uint256 amount = Math.min(balance, allowance);
migrate(account, amount); migrate(account, amount);
} }
uint256[50] private ______gap;
} }
...@@ -140,4 +140,6 @@ contract SignatureBouncer is Initializable, SignerRole { ...@@ -140,4 +140,6 @@ contract SignatureBouncer is Initializable, SignerRole {
return signer != address(0) && isSigner(signer); return signer != address(0) && isSigner(signer);
} }
uint256[50] private ______gap;
} }
...@@ -176,4 +176,6 @@ contract TokenVesting is Initializable, Ownable { ...@@ -176,4 +176,6 @@ contract TokenVesting is Initializable, Ownable {
return totalBalance.mul(block.timestamp.sub(_start)).div(_duration); return totalBalance.mul(block.timestamp.sub(_start)).div(_duration);
} }
} }
uint256[50] private ______gap;
} }
...@@ -25,6 +25,8 @@ contract SampleCrowdsaleToken is Initializable, ERC20Mintable { ...@@ -25,6 +25,8 @@ contract SampleCrowdsaleToken is Initializable, ERC20Mintable {
symbol = "SCT"; symbol = "SCT";
decimals = 18; decimals = 18;
} }
uint256[50] private ______gap;
} }
...@@ -67,4 +69,6 @@ contract SampleCrowdsale is Initializable, Crowdsale, CappedCrowdsale, Refundabl ...@@ -67,4 +69,6 @@ contract SampleCrowdsale is Initializable, Crowdsale, CappedCrowdsale, Refundabl
//the value needs to less or equal than a cap which is limit for accepted funds //the value needs to less or equal than a cap which is limit for accepted funds
require(goal <= cap); require(goal <= cap);
} }
uint256[50] private ______gap;
} }
...@@ -26,4 +26,6 @@ contract SimpleToken is Initializable, ERC20 { ...@@ -26,4 +26,6 @@ contract SimpleToken is Initializable, ERC20 {
_mint(sender, INITIAL_SUPPLY); _mint(sender, INITIAL_SUPPLY);
} }
uint256[50] private ______gap;
} }
...@@ -53,4 +53,6 @@ contract ERC165 is Initializable, IERC165 { ...@@ -53,4 +53,6 @@ contract ERC165 is Initializable, IERC165 {
require(interfaceId != 0xffffffff); require(interfaceId != 0xffffffff);
_supportedInterfaces[interfaceId] = true; _supportedInterfaces[interfaceId] = true;
} }
uint256[50] private ______gap;
} }
...@@ -56,4 +56,6 @@ contract Pausable is Initializable, PauserRole { ...@@ -56,4 +56,6 @@ contract Pausable is Initializable, PauserRole {
_paused = false; _paused = false;
emit Unpaused(); emit Unpaused();
} }
uint256[50] private ______gap;
} }
...@@ -76,4 +76,6 @@ contract Ownable is Initializable { ...@@ -76,4 +76,6 @@ contract Ownable is Initializable {
emit OwnershipTransferred(_owner, newOwner); emit OwnershipTransferred(_owner, newOwner);
_owner = newOwner; _owner = newOwner;
} }
uint256[50] private ______gap;
} }
...@@ -33,4 +33,6 @@ contract Secondary is Initializable { ...@@ -33,4 +33,6 @@ contract Secondary is Initializable {
_primary = recipient; _primary = recipient;
} }
uint256[50] private ______gap;
} }
...@@ -24,4 +24,6 @@ contract ConditionalEscrow is Initializable, Escrow { ...@@ -24,4 +24,6 @@ contract ConditionalEscrow is Initializable, Escrow {
require(withdrawalAllowed(payee)); require(withdrawalAllowed(payee));
super.withdraw(payee); super.withdraw(payee);
} }
uint256[50] private ______gap;
} }
...@@ -52,4 +52,6 @@ contract Escrow is Initializable, Secondary { ...@@ -52,4 +52,6 @@ contract Escrow is Initializable, Secondary {
emit Withdrawn(payee, payment); emit Withdrawn(payee, payment);
} }
uint256[50] private ______gap;
} }
...@@ -44,4 +44,6 @@ contract PullPayment is Initializable { ...@@ -44,4 +44,6 @@ contract PullPayment is Initializable {
function _asyncTransfer(address dest, uint256 amount) internal { function _asyncTransfer(address dest, uint256 amount) internal {
_escrow.deposit.value(amount)(dest); _escrow.deposit.value(amount)(dest);
} }
uint256[50] private ______gap;
} }
...@@ -87,4 +87,6 @@ contract RefundEscrow is Initializable, ConditionalEscrow { ...@@ -87,4 +87,6 @@ contract RefundEscrow is Initializable, ConditionalEscrow {
function withdrawalAllowed(address payee) public view returns (bool) { function withdrawalAllowed(address payee) public view returns (bool) {
return _state == State.Refunding; return _state == State.Refunding;
} }
uint256[50] private ______gap;
} }
...@@ -107,4 +107,6 @@ contract SplitPayment is Initializable { ...@@ -107,4 +107,6 @@ contract SplitPayment is Initializable {
_shares[account] = shares_; _shares[account] = shares_;
_totalShares = _totalShares.add(shares_); _totalShares = _totalShares.add(shares_);
} }
uint256[50] private ______gap;
} }
...@@ -210,4 +210,6 @@ contract ERC20 is Initializable, IERC20 { ...@@ -210,4 +210,6 @@ contract ERC20 is Initializable, IERC20 {
amount); amount);
_burn(account, amount); _burn(account, amount);
} }
uint256[50] private ______gap;
} }
...@@ -26,4 +26,6 @@ contract ERC20Burnable is Initializable, ERC20 { ...@@ -26,4 +26,6 @@ contract ERC20Burnable is Initializable, ERC20 {
function burnFrom(address from, uint256 value) public { function burnFrom(address from, uint256 value) public {
_burnFrom(from, value); _burnFrom(from, value);
} }
uint256[50] private ______gap;
} }
...@@ -47,4 +47,6 @@ contract ERC20Capped is Initializable, ERC20Mintable { ...@@ -47,4 +47,6 @@ contract ERC20Capped is Initializable, ERC20Mintable {
return super.mint(to, amount); return super.mint(to, amount);
} }
uint256[50] private ______gap;
} }
...@@ -41,4 +41,6 @@ contract ERC20Detailed is Initializable, IERC20 { ...@@ -41,4 +41,6 @@ contract ERC20Detailed is Initializable, IERC20 {
function decimals() public view returns(uint8) { function decimals() public view returns(uint8) {
return _decimals; return _decimals;
} }
uint256[50] private ______gap;
} }
...@@ -31,4 +31,6 @@ contract ERC20Mintable is Initializable, ERC20, MinterRole { ...@@ -31,4 +31,6 @@ contract ERC20Mintable is Initializable, ERC20, MinterRole {
_mint(to, amount); _mint(to, amount);
return true; return true;
} }
uint256[50] private ______gap;
} }
...@@ -70,4 +70,6 @@ contract ERC20Pausable is Initializable, ERC20, Pausable { ...@@ -70,4 +70,6 @@ contract ERC20Pausable is Initializable, ERC20, Pausable {
{ {
return super.decreaseAllowance(spender, subtractedValue); return super.decreaseAllowance(spender, subtractedValue);
} }
uint256[50] private ______gap;
} }
...@@ -69,4 +69,6 @@ contract TokenTimelock is Initializable { ...@@ -69,4 +69,6 @@ contract TokenTimelock is Initializable {
_token.safeTransfer(_beneficiary, amount); _token.safeTransfer(_beneficiary, amount);
} }
uint256[50] private ______gap;
} }
...@@ -327,4 +327,6 @@ contract ERC721 is Initializable, ERC165, IERC721 { ...@@ -327,4 +327,6 @@ contract ERC721 is Initializable, ERC165, IERC721 {
msg.sender, from, tokenId, _data); msg.sender, from, tokenId, _data);
return (retval == _ERC721_RECEIVED); return (retval == _ERC721_RECEIVED);
} }
uint256[50] private ______gap;
} }
...@@ -15,4 +15,6 @@ contract ERC721Burnable is Initializable, ERC721 { ...@@ -15,4 +15,6 @@ contract ERC721Burnable is Initializable, ERC721 {
require(_isApprovedOrOwner(msg.sender, tokenId)); require(_isApprovedOrOwner(msg.sender, tokenId));
_burn(ownerOf(tokenId), tokenId); _burn(ownerOf(tokenId), tokenId);
} }
uint256[50] private ______gap;
} }
...@@ -147,4 +147,6 @@ contract ERC721Enumerable is Initializable, ERC165, ERC721, IERC721Enumerable { ...@@ -147,4 +147,6 @@ contract ERC721Enumerable is Initializable, ERC165, ERC721, IERC721Enumerable {
_allTokensIndex[tokenId] = 0; _allTokensIndex[tokenId] = 0;
_allTokensIndex[lastToken] = tokenIndex; _allTokensIndex[lastToken] = tokenIndex;
} }
uint256[50] private ______gap;
} }
...@@ -21,4 +21,6 @@ contract ERC721Full is Initializable, ERC721, ERC721Enumerable, ERC721Metadata { ...@@ -21,4 +21,6 @@ contract ERC721Full is Initializable, ERC721, ERC721Enumerable, ERC721Metadata {
ERC721Enumerable.initialize(); ERC721Enumerable.initialize();
ERC721Metadata.initialize(name, symbol); ERC721Metadata.initialize(name, symbol);
} }
uint256[50] private ______gap;
} }
...@@ -16,4 +16,6 @@ contract ERC721Holder is Initializable, IERC721Receiver { ...@@ -16,4 +16,6 @@ contract ERC721Holder is Initializable, IERC721Receiver {
{ {
return this.onERC721Received.selector; return this.onERC721Received.selector;
} }
uint256[50] private ______gap;
} }
...@@ -89,4 +89,6 @@ contract ERC721Metadata is Initializable, ERC165, ERC721, IERC721Metadata { ...@@ -89,4 +89,6 @@ contract ERC721Metadata is Initializable, ERC165, ERC721, IERC721Metadata {
delete _tokenURIs[tokenId]; delete _tokenURIs[tokenId];
} }
} }
uint256[50] private ______gap;
} }
...@@ -35,4 +35,6 @@ contract ERC721MetadataMintable is Initializable, ERC721, ERC721Metadata, Minter ...@@ -35,4 +35,6 @@ contract ERC721MetadataMintable is Initializable, ERC721, ERC721Metadata, Minter
_setTokenURI(tokenId, tokenURI); _setTokenURI(tokenId, tokenURI);
return true; return true;
} }
uint256[50] private ______gap;
} }
...@@ -32,4 +32,6 @@ contract ERC721Mintable is Initializable, ERC721, MinterRole { ...@@ -32,4 +32,6 @@ contract ERC721Mintable is Initializable, ERC721, MinterRole {
_mint(to, tokenId); _mint(to, tokenId);
return true; return true;
} }
uint256[50] private ______gap;
} }
...@@ -45,4 +45,6 @@ contract ERC721Pausable is Initializable, ERC721, Pausable { ...@@ -45,4 +45,6 @@ contract ERC721Pausable is Initializable, ERC721, Pausable {
{ {
super.transferFrom(from, to, tokenId); super.transferFrom(from, to, tokenId);
} }
uint256[50] private ______gap;
} }
...@@ -25,5 +25,4 @@ library Address { ...@@ -25,5 +25,4 @@ library Address {
assembly { size := extcodesize(account) } assembly { size := extcodesize(account) }
return size > 0; return size > 0;
} }
} }
...@@ -32,4 +32,5 @@ contract ReentrancyGuard is Initializable { ...@@ -32,4 +32,5 @@ contract ReentrancyGuard is Initializable {
require(localCounter == _guardCounter); require(localCounter == _guardCounter);
} }
uint256[50] private ______gap;
} }
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment