Fix BeaconBlockRootHandler

[yerke26]

Fixes #7594

Changes

• add check for eip4788Account existence

Types of changes

What types of changes does your code introduce?

• Bugfix (a non-breaking change that fixes an issue)
• New feature (a non-breaking change that adds functionality)
• Breaking change (a change that causes existing functionality not to work as expected)
• Optimization
• Refactoring
• Documentation update
• Build-related changes
• Other: Description

Testing

Requires testing

• Yes
• No

If yes, did you write tests?

• Yes
• No

Notes on testing

Optional. Remove if not applicable.

Documentation

Requires documentation update

• Yes
• No

If yes, link the PR to the docs update or the issue with the details labeled docs. Remove if not applicable.

Requires explanation in Release Notes

• Yes
• No

If yes, fill in the details here. Remove if not applicable.

Remarks

Optional. Remove if not applicable.

[LukaszRozmej]

Can you explain why this is the issue? It is just AccessList creation, which shouldn't make a difference here if there is no contract?

[yerke26]

Can you explain why this is the issue? It is just AccessList creation, which shouldn't make a difference here if there is no contract?

I checked geth implementation:
Evm call is called here with BeaconRootsAddress, which fits this condition and does not create account with BeaconRootsAddress, which does not change state root. I think the issue that we create account with BeaconRootsAddress.

[LukaszRozmej]

Can you explain why this is the issue? It is just AccessList creation, which shouldn't make a difference here if there is no contract?

I checked geth implementation: Evm call is called here with BeaconRootsAddress, which fits this condition and does not create account with BeaconRootsAddress, which does not change state root. I think the issue that we create account with BeaconRootsAddress.

Ok but Geth also uses access list but doesn't create the account? Something is off.

[yerke26]

Can you explain why this is the issue? It is just AccessList creation, which shouldn't make a difference here if there is no contract?

I checked geth implementation: Evm call is called here with BeaconRootsAddress, which fits this condition and does not create account with BeaconRootsAddress, which does not change state root. I think the issue that we create account with BeaconRootsAddress.

Ok but Geth also uses access list but doesn't create the account? Something is off.

Yes, here, before Evm call

[MarekM25]

can you pass StateProvider in constructor?