Skip to content

fix(levm): fix transfer value in call, organize call opcodes and other fixes#1492

Merged
JereSalo merged 15 commits into
mainfrom
levm/fix_transfer_value
Dec 16, 2024
Merged

fix(levm): fix transfer value in call, organize call opcodes and other fixes#1492
JereSalo merged 15 commits into
mainfrom
levm/fix_transfer_value

Conversation

@JereSalo

@JereSalo JereSalo commented Dec 12, 2024

Copy link
Copy Markdown
Contributor

Motivation

Description

  • Removes in generic_call double definition of should_transfer_value

  • Organizes call opcodes

  • Now we don't return early when bytecode is empty. This was inaccurate behavior

Closes #issue_number

@JereSalo JereSalo added the levm Lambda EVM implementation label Dec 12, 2024
@JereSalo JereSalo self-assigned this Dec 12, 2024
@JereSalo JereSalo changed the title fix(levm): fix transfer value in call fix(levm): fix transfer value in call, organize call opcodes and other fixes Dec 12, 2024
@JereSalo JereSalo marked this pull request as ready for review December 13, 2024 15:45
@JereSalo JereSalo requested a review from a team as a code owner December 13, 2024 15:45

@maximopalopoli maximopalopoli left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Now you can also move generic_call to system.rs, as you said before.

@LeanSerra LeanSerra left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, left a small comment

Comment thread crates/vm/levm/src/opcode_handlers/system.rs Outdated
@JereSalo JereSalo added this pull request to the merge queue Dec 16, 2024
Merged via the queue into main with commit d38ee81 Dec 16, 2024
@JereSalo JereSalo deleted the levm/fix_transfer_value branch December 16, 2024 12:04
pedrobergamini pushed a commit to pedrobergamini/ethrex that referenced this pull request Aug 24, 2025
…r fixes (lambdaclass#1492)

**Motivation**

<!-- Why does this pull request exist? What are its goals? -->

**Description**
- Removes in `generic_call` double definition of `should_transfer_value`

- Organizes call opcodes

- Now we don't return early when bytecode is empty. This was inaccurate
behavior


<!-- A clear and concise general description of the changes this PR
introduces -->

<!-- Link to issues: Resolves lambdaclass#111, Resolves lambdaclass#222 -->

Closes #issue_number
edg-l added a commit that referenced this pull request May 13, 2026
ethrex ab15f37 hardened the HTTP JSON-RPC defaults, gating
`admin_*`/`debug_*`/`txpool_*` behind a new `--http.api` allowlist.
Hive c4d839b3 (#1485) feature-detects the flag in `clients/ethrex/
ethrex.sh` and opts those namespaces back in. The pinned version
`1c27560a` predates that fix, so hive runs against current main start
ethrex without the namespace allowlist and any test touching the
gated namespaces gets `MethodNotFound`.

Bump to current hive HEAD (3b6bde03), which includes the fix plus a
Cargo.lock dependency-drift safeguard (#1492).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

levm Lambda EVM implementation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants