From e2c29255d9a390d081c45e7c7f16dd325add329f Mon Sep 17 00:00:00 2001 From: clabby <ben@clab.by> Date: Sun, 3 Sep 2023 16:58:09 -0400 Subject: [PATCH] Fix prestate hash --- op-challenger/game/fault/player.go | 2 -- op-challenger/game/fault/player_test.go | 2 -- 2 files changed, 4 deletions(-) diff --git a/op-challenger/game/fault/player.go b/op-challenger/game/fault/player.go index c4a189e1c..4d3eca4fb 100644 --- a/op-challenger/game/fault/player.go +++ b/op-challenger/game/fault/player.go @@ -5,7 +5,6 @@ import ( "context" "fmt" - "github.com/ethereum-optimism/optimism/cannon/mipsevm" "github.com/ethereum-optimism/optimism/op-bindings/bindings" "github.com/ethereum-optimism/optimism/op-challenger/config" "github.com/ethereum-optimism/optimism/op-challenger/game/fault/responder" @@ -170,7 +169,6 @@ func ValidateAbsolutePrestate(ctx context.Context, trace types.TraceProvider, lo return fmt.Errorf("failed to get the trace provider's absolute prestate: %w", err) } providerPrestateHash := crypto.Keccak256(providerPrestate) - providerPrestateHash[0] = mipsevm.VMStatusUnfinished onchainPrestate, err := loader.FetchAbsolutePrestateHash(ctx) if err != nil { return fmt.Errorf("failed to get the onchain absolute prestate: %w", err) diff --git a/op-challenger/game/fault/player_test.go b/op-challenger/game/fault/player_test.go index f74dfa19c..b31e9edfd 100644 --- a/op-challenger/game/fault/player_test.go +++ b/op-challenger/game/fault/player_test.go @@ -6,7 +6,6 @@ import ( "fmt" "testing" - "github.com/ethereum-optimism/optimism/cannon/mipsevm" "github.com/ethereum-optimism/optimism/op-challenger/game/fault/types" "github.com/ethereum-optimism/optimism/op-node/testlog" "github.com/ethereum/go-ethereum/common" @@ -120,7 +119,6 @@ func TestValidateAbsolutePrestate(t *testing.T) { t.Run("ValidPrestates", func(t *testing.T) { prestate := []byte{0x00, 0x01, 0x02, 0x03} prestateHash := crypto.Keccak256(prestate) - prestateHash[0] = mipsevm.VMStatusUnfinished mockTraceProvider := newMockTraceProvider(false, prestate) mockLoader := newMockPrestateLoader(false, common.BytesToHash(prestateHash)) err := ValidateAbsolutePrestate(context.Background(), mockTraceProvider, mockLoader) -- 2.23.0