Open jalextowle opened 2 weeks ago
Benchmark suite | Current: 9a76fb63af26848667eb1adbaaaa50b7af200d1e | Previous: 7bc914ff4f80a017d14a35d9cb5c5dd99987bd20 | Deviation | Status |
---|---|---|---|---|
addLiquidity: min |
33937 gas |
33937 gas |
0% |
🟰 |
addLiquidity: avg |
156398 gas |
155757 gas |
0.4115% |
🚨 |
addLiquidity: max |
429290 gas |
429262 gas |
0.0065% |
🚨 |
checkpoint: min |
40316 gas |
40316 gas |
0% |
🟰 |
checkpoint: avg |
144694 gas |
144703 gas |
-0.0062% |
✅ |
checkpoint: max |
255940 gas |
255940 gas |
0% |
🟰 |
closeLong: min |
31361 gas |
31361 gas |
0% |
🟰 |
closeLong: avg |
135976 gas |
135769 gas |
0.1525% |
🚨 |
closeLong: max |
2539376 gas |
2539376 gas |
0% |
🟰 |
closeShort: min |
31327 gas |
31327 gas |
0% |
🟰 |
closeShort: avg |
131140 gas |
131221 gas |
-0.0617% |
✅ |
closeShort: max |
271599 gas |
271348 gas |
0.0925% |
🚨 |
initialize: min |
31349 gas |
31349 gas |
0% |
🟰 |
initialize: avg |
333507 gas |
333439 gas |
0.0204% |
🚨 |
initialize: max |
400255 gas |
399978 gas |
0.0693% |
🚨 |
openLong: min |
33370 gas |
33370 gas |
0% |
🟰 |
openLong: avg |
173293 gas |
173218 gas |
0.0433% |
🚨 |
openLong: max |
334313 gas |
334062 gas |
0.0751% |
🚨 |
openShort: min |
33936 gas |
33936 gas |
0% |
🟰 |
openShort: avg |
168403 gas |
168476 gas |
-0.0433% |
✅ |
openShort: max |
414974 gas |
415110 gas |
-0.0328% |
✅ |
redeemWithdrawalShares: min |
31251 gas |
31251 gas |
0% |
🟰 |
redeemWithdrawalShares: avg |
74940 gas |
75114 gas |
-0.2316% |
✅ |
redeemWithdrawalShares: max |
305949 gas |
305698 gas |
0.0821% |
🚨 |
removeLiquidity: min |
31301 gas |
31301 gas |
0% |
🟰 |
removeLiquidity: avg |
215568 gas |
215048 gas |
0.2418% |
🚨 |
removeLiquidity: max |
403870 gas |
403641 gas |
0.0567% |
🚨 |
This comment was automatically generated by workflow using github-action-benchmark.
Resolved Issues
Fixes: https://github.com/spearbit-audits/delv-week-review/issues/15
Description
This PR adds a knob that can be used to protect against k-inflation attacks that can occur with pools that have been configured with high time stretches.
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?