Skip to content
Projects
Groups
Snippets
Help
Loading...
Help
Support
Submit feedback
Contribute to GitLab
Sign in
Toggle navigation
N
nebula
Project
Project
Details
Activity
Releases
Cycle Analytics
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Charts
Issues
0
Issues
0
List
Boards
Labels
Milestones
Merge Requests
0
Merge Requests
0
CI / CD
CI / CD
Pipelines
Jobs
Schedules
Charts
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Charts
Create a new issue
Jobs
Commits
Issue Boards
Open sidebar
exchain
nebula
Commits
dd4dce86
Unverified
Commit
dd4dce86
authored
Feb 01, 2023
by
mergify[bot]
Committed by
GitHub
Feb 01, 2023
Browse files
Options
Browse Files
Download
Plain Diff
Merge pull request #4822 from mdehoog/fix-proof
[op-node] Fix ReadStorageAt for 0-prefixed values
parents
56efac5b
5f203dee
Changes
2
Hide whitespace changes
Inline
Side-by-side
Showing
2 changed files
with
3 additions
and
2 deletions
+3
-2
account_proof.go
op-node/eth/account_proof.go
+1
-1
eth_client.go
op-node/sources/eth_client.go
+2
-1
No files found.
op-node/eth/account_proof.go
View file @
dd4dce86
...
...
@@ -15,7 +15,7 @@ import (
type
StorageProofEntry
struct
{
Key
common
.
Hash
`json:"key"`
Value
hexutil
.
B
ytes
`json:"value"`
Value
hexutil
.
B
ig
`json:"value"`
Proof
[]
hexutil
.
Bytes
`json:"proof"`
}
...
...
op-node/sources/eth_client.go
View file @
dd4dce86
...
...
@@ -359,7 +359,8 @@ func (s *EthClient) ReadStorageAt(ctx context.Context, address common.Address, s
if
err
:=
result
.
Verify
(
block
.
Root
());
err
!=
nil
{
return
common
.
Hash
{},
fmt
.
Errorf
(
"failed to verify retrieved proof against state root: %w"
,
err
)
}
return
common
.
BytesToHash
(
result
.
StorageProof
[
0
]
.
Value
),
nil
value
:=
result
.
StorageProof
[
0
]
.
Value
.
ToInt
()
return
common
.
BytesToHash
(
value
.
Bytes
()),
nil
}
func
(
s
*
EthClient
)
Close
()
{
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment