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.96% -0.03%
==========================================
Files 320 320
Lines 23433 23434 +1
==========================================
- Hits 15698 15692 -6
- Misses 6578 6583 +5
- Partials 1157 1159 +2
🚀 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.
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