Merge branch '170-move-mint-after-the-transfer-in-deposit' into 'master'

Resolve "Move mint() after the transfer in deposit()"

Closes #170

See merge request aave-tech/protocol-v2!191
This commit is contained in:
Ernesto Boado 2020-11-26 15:17:03 +00:00
commit 3b85560340

View File

@ -117,6 +117,8 @@ contract LendingPool is VersionedInitializable, ILendingPool, LendingPoolStorage
reserve.updateState();
reserve.updateInterestRates(asset, aToken, amount, 0);
IERC20(asset).safeTransferFrom(msg.sender, aToken, amount);
bool isFirstDeposit = IAToken(aToken).mint(onBehalfOf, amount, reserve.liquidityIndex);
if (isFirstDeposit) {
@ -124,8 +126,6 @@ contract LendingPool is VersionedInitializable, ILendingPool, LendingPoolStorage
emit ReserveUsedAsCollateralEnabled(asset, onBehalfOf);
}
IERC20(asset).safeTransferFrom(msg.sender, aToken, amount);
emit Deposit(asset, msg.sender, onBehalfOf, amount, referralCode);
}