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

Merge pull request #1647 from nventuro/safeerc20-bugfix

Browse Source 
Fix SafeERC20.safeApprove bug
This commit is contained in:
Nicolás Venturo
2019-02-26 12:39:02 -03:00
committed by GitHub
parent 14ec984cc4 b14c9f4b68
commit 3111291b4a
2 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
contracts/mocks/SafeERC20Helper.sol
View File

@ -32,7 +32,7 @@ contract ERC20FailingMock {
}
contract ERC20SucceedingMock {
uint256 private _allowance;
mapping (address => uint256) private _allowances;
// IERC20's functions are not pure, but these mock implementations are: to prevent Solidity from issuing warnings,
// we write to a dummy state variable.
@ -54,11 +54,11 @@ contract ERC20SucceedingMock {
}
function setAllowance(uint256 allowance_) public {
_allowance = allowance_;
_allowances[msg.sender] = allowance_;
}
function allowance(address, address) public view returns (uint256) {
return _allowance;
function allowance(address owner, address) public view returns (uint256) {
return _allowances[owner];
}
}

2
contracts/token/ERC20/SafeERC20.sol
View File

@ -24,7 +24,7 @@ library SafeERC20 {
// safeApprove should only be called when setting an initial allowance,
// or when resetting it to zero. To increase and decrease it, use
// 'safeIncreaseAllowance' and 'safeDecreaseAllowance'
require((value == 0) || (token.allowance(msg.sender, spender) == 0));
require((value == 0) || (token.allowance(address(this), spender) == 0));
require(token.approve(spender, value));
}