Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

execution: Fix request append out of range in block building #13625

Merged
merged 1 commit into from
Jan 31, 2025

Conversation

somnathb1
Copy link
Contributor

@somnathb1 somnathb1 commented Jan 30, 2025

Fixes an issue reported for devnet-5 with trace as follows

[INFO] [01-29|21:33:12.026] Received GetPayloadV4                    payloadId=369
[EROR] [01-29|21:33:12.026] RPC method engine_getPayloadV4 crashed: runtime error: index out of range [0] with length 0
[service.go:223 panic.go:770 panic.go:114 block_building.go:223 execution_client.go:53 engine_server.go:475 engine_api_methods.go:74 value.go:596 value.go:380 service.go:228 handler.go:534 handler.go:484 handler.go:425 handler.go:245 handler.go:338 asm_amd64.s:1695]

@somnathb1 somnathb1 merged commit 113a320 into main Jan 31, 2025
12 of 13 checks passed
@somnathb1 somnathb1 deleted the som/fix_req_block_building branch January 31, 2025 08:44
somnathb1 added a commit that referenced this pull request Feb 3, 2025
Fixes an issue reported for `devnet-5` with trace as follows
```
[INFO] [01-29|21:33:12.026] Received GetPayloadV4                    payloadId=369
[EROR] [01-29|21:33:12.026] RPC method engine_getPayloadV4 crashed: runtime error: index out of range [0] with length 0
[service.go:223 panic.go:770 panic.go:114 block_building.go:223 execution_client.go:53 engine_server.go:475 engine_api_methods.go:74 value.go:596 value.go:380 service.go:228 handler.go:534 handler.go:484 handler.go:425 handler.go:245 handler.go:338 asm_amd64.s:1695]
```
@VBulikov VBulikov mentioned this pull request Feb 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants