Skip to content

L-03: UniswapV3 liquidity can be deposited on donated or outdated positions#93

Closed
davidlee1435 wants to merge 2 commits intodevfrom
david/l-03
Closed

L-03: UniswapV3 liquidity can be deposited on donated or outdated positions#93
davidlee1435 wants to merge 2 commits intodevfrom
david/l-03

Conversation

@davidlee1435
Copy link
Copy Markdown
Collaborator

Tracks whether controller minted the tokenId to prevent a malicious relayer from donating positions outside of governance bounds to the proxy and adding liquidity to it

@github-actions
Copy link
Copy Markdown

Coverage after merging david/l-03 into dev will be

97.69%

Coverage Report
FileStmtsBranchesFuncsLinesUncovered Lines
deploy
   ControllerDeploy.sol100%100%100%100%
   ForeignControllerInit.sol100%100%100%100%
   MainnetControllerInit.sol97.56%94.12%100%100%156, 92
src
   ALMProxy.sol100%100%100%100%
   ForeignController.sol96.96%86.67%97.87%98.39%162–163, 267, 374–375, 681, 981
   MainnetController.sol99.07%88.46%100%100%1048, 264, 530
   RateLimitHelpers.sol100%100%100%100%
   RateLimits.sol100%100%100%100%
src/libraries
   CCTPLib.sol100%100%100%100%
   CentrifugeLib.sol100%100%100%100%
   CurveLib.sol100%100%100%100%
   MerklLib.sol100%100%100%100%
   PSMLib.sol100%100%100%100%
   PendleLib.sol100%100%100%100%
   UniswapV3Lib.sol97.22%90.48%100%100%120, 150–151, 364–365, 91
src/libraries/common
   ERC20Lib.sol100%100%100%100%
   MathLib.sol100%100%100%100%
src/libraries/uniswap-v3
   UniV3OracleLib.sol59.26%40%50%65%23, 51, 57, 60, 60, 60–62, 66–67
   UniV3UtilsLib.sol41.18%16.67%33.33%50%102, 107–108, 34–35, 35, 35–36, 45, 47, 58, 63, 81, 90, 95–96

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants