Closed jalextowle closed 1 week ago
Totals | |
---|---|
Change from base Build 9690425513: | 0.0% |
Covered Lines: | 1984 |
Relevant Lines: | 2141 |
Benchmark suite | Current: 016dd0b04d67fccf43b633f58b6c2c5c24aa9092 | Previous: e84bb23edd4b8d4b6ee42dac6c225bde9df2de3c | Deviation | Status |
---|---|---|---|---|
addLiquidity: min |
33937 gas |
33937 gas |
0% |
π° |
addLiquidity: avg |
156552 gas |
156075 gas |
0.3056% |
π¨ |
addLiquidity: max |
429262 gas |
429262 gas |
0% |
π° |
checkpoint: min |
40316 gas |
40316 gas |
0% |
π° |
checkpoint: avg |
144763 gas |
144745 gas |
0.0124% |
π¨ |
checkpoint: max |
255940 gas |
255940 gas |
0% |
π° |
closeLong: min |
31361 gas |
31361 gas |
0% |
π° |
closeLong: avg |
135570 gas |
135881 gas |
-0.2289% |
β |
closeLong: max |
2539376 gas |
2539376 gas |
0% |
π° |
closeShort: min |
31327 gas |
31327 gas |
0% |
π° |
closeShort: avg |
131201 gas |
131156 gas |
0.0343% |
π¨ |
closeShort: max |
271117 gas |
271117 gas |
0% |
π° |
initialize: min |
31349 gas |
31349 gas |
0% |
π° |
initialize: avg |
333412 gas |
333484 gas |
-0.0216% |
β |
initialize: max |
399746 gas |
399746 gas |
0% |
π° |
openLong: min |
33370 gas |
33370 gas |
0% |
π° |
openLong: avg |
172967 gas |
172902 gas |
0.0376% |
π¨ |
openLong: max |
333831 gas |
333831 gas |
0% |
π° |
openShort: min |
33936 gas |
33936 gas |
0% |
π° |
openShort: avg |
168219 gas |
168373 gas |
-0.0915% |
β |
openShort: max |
415052 gas |
414680 gas |
0.0897% |
π¨ |
redeemWithdrawalShares: min |
31251 gas |
31251 gas |
0% |
π° |
redeemWithdrawalShares: avg |
74389 gas |
75443 gas |
-1.3971% |
β |
redeemWithdrawalShares: max |
305459 gas |
305459 gas |
0% |
π° |
removeLiquidity: min |
31301 gas |
31301 gas |
0% |
π° |
removeLiquidity: avg |
215576 gas |
215556 gas |
0.0093% |
π¨ |
removeLiquidity: max |
403641 gas |
403858 gas |
-0.0537% |
β |
This comment was automatically generated by workflow using github-action-benchmark.
Resolved Issues
Fixes: #799.
Description
After some issues were encountered with the agent0 fuzzing that used the
MockLido
vault, it became apparent that the current methodology used by theMockLido
vault wasn't acceptable. This PR rewrites the mock yield source so that we can continue fuzzing the stETH integration.Review Checklists
Please check each item before approving the pull request. While going through the checklist, it is recommended to leave comments on items that are referenced in the checklist to make sure that they are reviewed. If there are multiple reviewers, copy the checklists into sections titled
## [Reviewer Name]
. If the PR doesn't touch Solidity, the corresponding checklist can be removed.[[Reviewer Name]]
approve
calls useforceApprove
?transfer
calls usesafeTransfer
?transferFrom
calls usemsg.sender
as thefrom
address?call
,delegatecall
,staticcall
,transfer
,send
)success
boolean checked to handle failed calls?delegatecall
, are there strict access controls on the addresses that can be called? It shouldn't be possible todelegatecall
arbitrary addresses, so the list of possible targets should either be immutable or tightly controlled by an admin.nonReentrant
?payable
functions restricted to avoid stuck ether?Safe
functions are altered, are potential underflows and overflows caught so that a failure flag can be thrown?