Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Only payload size #15

Closed
wants to merge 3 commits into from
Closed

Only payload size #15

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
a8a78cb
onlyPayloadSize
jbaylina May 8, 2017
b0b5eb2
compiled
jbaylina May 8, 2017
348f3cb
onlyPayloadSize
jbaylina May 16, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 13 additions & 4 deletions contracts/MiniMeToken.sol
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -122,6 +122,15 @@ contract MiniMeToken is Controlled {
// The factory used to create new clone tokens
MiniMeTokenFactory public tokenFactory;

////////////////
// Modifiers
////////////////

modifier onlyPayloadSize(uint numwords) {
assert(msg.data.length == numwords * 32 + 4);
_;
}

////////////////
// Constructor
////////////////
Expand Down Expand Up @@ -167,7 +176,7 @@ contract MiniMeToken is Controlled {
/// @param _to The address of the recipient
/// @param _amount The amount of tokens to be transferred
/// @return Whether the transfer was successful or not
function transfer(address _to, uint256 _amount) returns (bool success) {
function transfer(address _to, uint256 _amount) onlyPayloadSize(2) returns (bool success) {
if (!transfersEnabled) throw;
return doTransfer(msg.sender, _to, _amount);
}
Expand All @@ -179,7 +188,7 @@ contract MiniMeToken is Controlled {
/// @param _amount The amount of tokens to be transferred
/// @return True if the transfer was successful
function transferFrom(address _from, address _to, uint256 _amount
) returns (bool success) {
) onlyPayloadSize(3) returns (bool success) {

// The controller of this contract can move tokens around at will,
// this is important to recognize! Confirm that you trust the
Expand Down Expand Up @@ -412,7 +421,7 @@ contract MiniMeToken is Controlled {
/// @param _amount The quantity of tokens generated
/// @return True if the tokens are generated correctly
function generateTokens(address _owner, uint _amount
) onlyController returns (bool) {
) onlyController onlyPayloadSize(2) returns (bool) {
uint curTotalSupply = getValueAt(totalSupplyHistory, block.number);
if (curTotalSupply + _amount < curTotalSupply) throw; // Check for overflow
updateValueAtNow(totalSupplyHistory, curTotalSupply + _amount);
Expand All @@ -429,7 +438,7 @@ contract MiniMeToken is Controlled {
/// @param _amount The quantity of tokens to burn
/// @return True if the tokens are burned correctly
function destroyTokens(address _owner, uint _amount
) onlyController returns (bool) {
) onlyPayloadSize(2) onlyController returns (bool) {
uint curTotalSupply = getValueAt(totalSupplyHistory, block.number);
if (curTotalSupply < _amount) throw;
updateValueAtNow(totalSupplyHistory, curTotalSupply - _amount);
Expand Down
Loading