NoBey/openzeppelin-contracts
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #231 from maraoz/fix-vested-token

Browse Source 
Implement VestedToken bug mitigation #3
This commit is contained in:
Manuel Aráoz
2017-05-29 18:27:44 -03:00
committed by GitHub
parent 7f6921e787 7100e30bed
commit 2bcb54b3cb
1 changed files with 5 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
contracts/token/VestedToken.sol
View File

@ -8,6 +8,9 @@ import "./LimitedTransferToken.sol";
* @dev Tokens that can be vested for a group of addresses. * @dev Tokens that can be vested for a group of addresses.
*/ */
contract VestedToken is StandardToken, LimitedTransferToken { contract VestedToken is StandardToken, LimitedTransferToken {
uint256 MAX_GRANTS_PER_ADDRESS = 20;
struct TokenGrant { struct TokenGrant {
address granter; // 20 bytes address granter; // 20 bytes
uint256 value; // 32 bytes uint256 value; // 32 bytes
@ -45,6 +48,8 @@ contract VestedToken is StandardToken, LimitedTransferToken {
throw; throw;
} }
if (tokenGrantsCount(_to) > MAX_GRANTS_PER_ADDRESS) throw; // To prevent a user being spammed and have his balance locked (out of gas attack when calculating vesting).
uint count = grants[_to].push( uint count = grants[_to].push(
TokenGrant( TokenGrant(
_revokable ? msg.sender : 0, // avoid storing an extra 20 bytes when it is non-revokable _revokable ? msg.sender : 0, // avoid storing an extra 20 bytes when it is non-revokable