Skip to content

Commit

Permalink
fix(liquidation): add verification for weth liquidation (#304)
Browse files Browse the repository at this point in the history
  • Loading branch information
WalidOfNow authored Dec 27, 2022
1 parent f6d1675 commit 6e0dbb9
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 0 deletions.
2 changes: 2 additions & 0 deletions contracts/protocol/libraries/logic/LiquidationLogic.sol
Original file line number Diff line number Diff line change
Expand Up @@ -354,12 +354,14 @@ library LiquidationLogic {
collateralReserve,
DataTypes.ValidateLiquidateERC721Params({
liquidationAssetReserveCache: vars.liquidationAssetReserveCache,
liquidationAsset: params.liquidationAsset,
liquidator: params.liquidator,
borrower: params.borrower,
globalDebt: vars.userGlobalDebt,
actualLiquidationAmount: vars.actualLiquidationAmount,
maxLiquidationAmount: params.liquidationAmount,
healthFactor: vars.healthFactor,
weth: params.weth,
priceOracleSentinel: params.priceOracleSentinel,
collateralAsset: params.collateralAsset,
tokenId: params.collateralTokenId,
Expand Down
5 changes: 5 additions & 0 deletions contracts/protocol/libraries/logic/ValidationLogic.sol
Original file line number Diff line number Diff line change
Expand Up @@ -669,6 +669,11 @@ library ValidationLogic {
);
}

require(
msg.value == 0 || params.liquidationAsset == params.weth,
Errors.INVALID_LIQUIDATION_ASSET
);

require(
params.maxLiquidationAmount >= params.actualLiquidationAmount &&
(msg.value == 0 || msg.value >= params.maxLiquidationAmount),
Expand Down
2 changes: 2 additions & 0 deletions contracts/protocol/libraries/types/DataTypes.sol
Original file line number Diff line number Diff line change
Expand Up @@ -242,12 +242,14 @@ library DataTypes {

struct ValidateLiquidateERC721Params {
ReserveCache liquidationAssetReserveCache;
address liquidationAsset;
address liquidator;
address borrower;
uint256 globalDebt;
uint256 healthFactor;
address collateralAsset;
uint256 tokenId;
address weth;
uint256 actualLiquidationAmount;
uint256 maxLiquidationAmount;
uint256 auctionRecoveryHealthFactor;
Expand Down

0 comments on commit 6e0dbb9

Please sign in to comment.