Merge pull request #2094 from CortexFoundation/dev

core/types: don't modify signature V when reading large chainID
CortexFoundation · Jul 17, 2024 · 444e6d3 · 444e6d3
2 parents cf14d92 + a7a9d8c
commit 444e6d3
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/core/types/transaction_signing.go b/core/types/transaction_signing.go
@@ -269,6 +269,6 @@ func deriveChainId(v *big.Int) *big.Int {
 		}
 		return new(big.Int).SetUint64((v - 35) / 2)
 	}
-	v.Sub(v, big.NewInt(35))
-	return v.Rsh(v, 1)
+	vCopy := new(big.Int).Sub(v, big.NewInt(35))
+	return vCopy.Rsh(vCopy, 1)
 }