Merge pull request #377 from yaronvel/master
fix for issue 375: transferFrom allows sending to self more than self balance
This commit is contained in:
@ -29,8 +29,8 @@ contract StandardToken is ERC20, BasicToken {
|
|||||||
// Check is not needed because sub(_allowance, _value) will already throw if this condition is not met
|
// Check is not needed because sub(_allowance, _value) will already throw if this condition is not met
|
||||||
// require (_value <= _allowance);
|
// require (_value <= _allowance);
|
||||||
|
|
||||||
balances[_to] = balances[_to].add(_value);
|
|
||||||
balances[_from] = balances[_from].sub(_value);
|
balances[_from] = balances[_from].sub(_value);
|
||||||
|
balances[_to] = balances[_to].add(_value);
|
||||||
allowed[_from][msg.sender] = _allowance.sub(_value);
|
allowed[_from][msg.sender] = _allowance.sub(_value);
|
||||||
Transfer(_from, _to, _value);
|
Transfer(_from, _to, _value);
|
||||||
return true;
|
return true;
|
||||||
|
|||||||
Reference in New Issue
Block a user