fix(rpc): eth_call over-reports gas for native precompiles#1183
Conversation
* buildTraceCtx installs a finite gas meter so OOM is detected past gasLimit * preserves KvGasConfig only when to is native precompile, so cosmos state ops charge same as EthCall, pure-EVM keeps KV zeroed for SLOAD/SSTORE * add recover for cosmos meter OOM panics as "raise gas" signals for binary search
|
PR author is not in the allowed authors list. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1183 +/- ##
==========================================
- Coverage 66.99% 66.98% -0.02%
==========================================
Files 320 320
Lines 23436 23437 +1
==========================================
- Hits 15702 15699 -3
- Misses 6576 6580 +4
Partials 1158 1158
🚀 New features to boost your workflow:
|
|
two quick fixes before merge:
|
aljo242
left a comment
There was a problem hiding this comment.
changelog links to wrong PR number; EthCall fix has no direct test coverage.
|
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days-before-close if no further activity occurs. |
Description
Closes: #1182
Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...
mainbranch