Skip to content

core/txpool/legacypool: fix data race of pricedList access#31758

Merged
rjl493456442 merged 1 commit into
ethereum:masterfrom
mirokuratczyk:simulated_backend_fix
May 4, 2025
Merged

core/txpool/legacypool: fix data race of pricedList access#31758
rjl493456442 merged 1 commit into
ethereum:masterfrom
mirokuratczyk:simulated_backend_fix

Conversation

@mirokuratczyk
Copy link
Copy Markdown
Contributor

@mirokuratczyk mirokuratczyk commented May 2, 2025

I believe this was missed in #31641 where fixing one data race created another. Should only apply to the simulated backend.

go-ethereum v1.15.10

The race detector flagged this in some automated tests I run against the simulated backend.

==================
WARNING: DATA RACE
Write at 0x00c00177bba8 by goroutine 36573:
  github.com/ethereum/go-ethereum/core/txpool/legacypool.(*LegacyPool).Clear()
      /home/repo/vendor/github.com/ethereum/go-ethereum/core/txpool/legacypool/legacypool.go:1937 +0x44d
  github.com/ethereum/go-ethereum/core/txpool.(*TxPool).Clear()
      /home/repo/vendor/github.com/ethereum/go-ethereum/core/txpool/txpool.go:508 +0x1af
  github.com/ethereum/go-ethereum/eth/catalyst.(*SimulatedBeacon).Rollback()
      /home/repo/vendor/github.com/ethereum/go-ethereum/eth/catalyst/simulated_beacon.go:320 +0xed5
  github.com/ethereum/go-ethereum/ethclient/simulated.(*Backend).Rollback()
      /home/repo/vendor/github.com/ethereum/go-ethereum/ethclient/simulated/backend.go:164 +0xe0b
  ...

Previous read at 0x00c00177bba8 by goroutine 36536:
  github.com/ethereum/go-ethereum/core/txpool/legacypool.(*LegacyPool).loop()
      /home/repo/vendor/github.com/ethereum/go-ethereum/core/txpool/legacypool/legacypool.go:363 +0x93b
  github.com/ethereum/go-ethereum/core/txpool/legacypool.(*LegacyPool).Init.gowrap2()
      /home/repo/vendor/github.com/ethereum/go-ethereum/core/txpool/legacypool/legacypool.go:330 +0x33
  ...
==================

rjl493456442
rjl493456442 approved these changes May 4, 2025
View reviewed changes
Copy link
Copy Markdown
Member

@rjl493456442 rjl493456442 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@rjl493456442 rjl493456442 added this to the 1.15.11 milestone May 4, 2025
@rjl493456442 rjl493456442 merged commit 2d86a54 into ethereum:master May 4, 2025
3 of 4 checks passed
@mirokuratczyk mirokuratczyk deleted the simulated_backend_fix branch May 4, 2025 13:14
jakub-freebit pushed a commit to fblch/go-ethereum that referenced this pull request Jul 3, 2025
@mirokuratczyk @jakub-freebit
core/txpool/legacypool: fix data race of pricedList access (ethereum#…
2f2a515 
…31758)
howjmay pushed a commit to iotaledger/go-ethereum that referenced this pull request Aug 27, 2025
@mirokuratczyk @howjmay
core/txpool/legacypool: fix data race of pricedList access (ethereum#…
e319b2f 
…31758)
gballet pushed a commit to gballet/go-ethereum that referenced this pull request Sep 11, 2025
@mirokuratczyk @gballet
core/txpool/legacypool: fix data race of pricedList access (ethereum#…
2ea7ffd 
…31758)
eomti-wm added a commit to stable-net/go-stablenet that referenced this pull request Jan 5, 2026
@eomti-wm
fix: fix data race of pricedList access
ed80b8a 
Use Reheap() instead of assigning a new pricedList to avoid
data race when clearing pool state.

Based on ethereum/go-ethereum#31758
@eomti-wm eomti-wm mentioned this pull request Jan 8, 2026
Merged
eomti-wm added a commit to stable-net/go-stablenet that referenced this pull request Jan 14, 2026
@eomti-wm
feat: EIP-7702 Set Code Transaction support (#42)
f32d12e 
* chore: update Go version to 1.21

* feat: implement EIP-7702 set code transaction type

Add support for EIP-7702 (SetCodeTx) which allows EOAs to temporarily
delegate their code execution to another account.

Changes include:
- Add SetCodeTx (type 0x04) transaction type and Authorization struct
- Add anzeonSigner for EIP-7702 transaction signing
- Implement delegation designator resolution in EVM
- Add EIP-7702 specific gas calculation for CALL variants
- Update EXTCODE* opcodes to handle delegation designators
- Add prestate tracer support for authorization list
- Update t8n tool and tests for SetCodeTx

Based on ethereum/go-ethereum#30078

* fix: complete EIP-7702 authorization marshaling and eth_call support

Based on ethereum/go-ethereum#30926

* refactor: improve EIP-7702 API naming

- Rename Authorization to SetCodeAuthorization
- Rename SignAuth to SignSetCode with key-first parameter order

Based on ethereum/go-ethereum#30933

* fix: rename SetCodeAuthorization 'v' to 'yParity'

The API spec requires the name yParity.

Based on:
- ethereum/go-ethereum@73a4ecf
- ethereum/go-ethereum#30936

* refactor: rename AuthList to SetCodeAuthorizations

Use SetCode prefix consistently in internal APIs for the authorization
list, as a follow-up to SetCodeAuthorization type renaming.

Based on ethereum/go-ethereum#30935

* refactor: change SetCodeTx.ChainID to uint256

Change ChainID field type from uint64 to uint256.Int in SetCodeTx and
SetCodeAuthorization for consistency with other chain ID handling.

Also removes unused json/gencodec tags from signature value fields
in DynamicFeeTx, BlobTx, SetCodeTx, and FeeDelegateDynamicFeeTx.

Based on ethereum/go-ethereum#30982

* fix: correct chainId check for anzeonSigner

Use zero value check for SetCodeTx chainId validation.
This aligns with cancunSigner and londonSigner as well.

Based on ethereum/go-ethereum#31032

* fix: initialize ChainID in SetCodeTx copy method

ChainID should be properly initialized and copied in the copy() method
to prevent nil pointer issues during transaction processing.

Based on ethereum/go-ethereum#31054

* refactor: remove EXTCODE* special handling for EIP-7702

EXTCODE* opcodes no longer need special overrides for EIP-7702 delegation
designators. The delegation designator is now returned as-is.

Implements EIPs#9248.

Based on ethereum/go-ethereum#31089

* feat: add EIP-7702 SetCode transaction pool support

Add SetCode transaction handling to legacypool:
- SetCodeTxType validation with Anzeon fork check
- Empty authorization rejection
- Authority tracking and conflict prevention
- Account slot limit for delegated accounts and pending authorizations

Note: tx replacement is not supported in go-stablenet. Tests are adjusted
to expect ErrAccountLimitExceeded instead of successful replacement.

Based on ethereum/go-ethereum#31073

* test: add setCodeTx reorg test

Based on ethereum/go-ethereum#31206

* refactor: move setcode tx validation into legacyPool

Move authorization-related validations from common validation to legacyPool
since these checks are specific to SetCode transactions.

Based on ethereum/go-ethereum#31209

* fix: fix error logs flood caused by removeAuthorities

Fix removeAuthorities to only iterate tx's authorities instead of
all authorities in the pool.

Based on ethereum/go-ethereum#31249

* refactor: reduce allocs in transaction signing

Add sigHash method to TxData interface and move hash computation
into each tx type, reducing allocations during signature operations.

Based on ethereum/go-ethereum#31258

* fix: reject gapped tx from delegated account

Add checkDelegationLimit to reject transactions with gapped nonces
from delegated accounts. Only one in-flight executable transaction
is allowed for senders with delegation or pending delegation.

Based on ethereum/go-ethereum#31430

* fix: exclude 7702 authorities from eth_createAccessList result

Exclude valid EIP-7702 authorities from access list generation result.

Based on ethereum/go-ethereum#31336
(which resolves ethereum/go-ethereum#31335)

* fix: data race in checkDelegationLimit

Add delegationTxsCount method with proper lock to fix data race
when accessing auths map in checkDelegationLimit.

Based on ethereum/go-ethereum#31475

* feat: add EIP-7702 protection to blobpool

Add delegation limit checks to blobpool:
- Limit senders with delegation/pending auth to one in-flight tx
- Check reserver for authority conflicts
- Refactor AddressReserver to Reserver struct with Hold/Release/Has

Based on ethereum/go-ethereum#31526

* fix: slot limit check and update comments

- Skip slot limit validation for tx replacements (same nonce reuses slot)
- Clarify cumulative balance validation skip reason: fee delegation compatibility

* fix: allow tx and authority regardless of admission order

Check reserver for authority conflicts to make authorization validation
independent of admission order. Authorities must not conflict with
addresses reserved by other subpools.

Based on ethereum/go-ethereum#31373

* fix: fix data race of txlookup access in test

Reset txlookup fields directly with proper locking instead of
assigning a new lookup object to avoid data race.

Based on ethereum/go-ethereum#31641

* feat: allow passing AuthorizationList to eth_call and eth_estimateGas

Add AuthorizationList field to CallMsg and toCallArg to support
SetCodeTx in eth_call and eth_estimateGas APIs.

Based on ethereum/go-ethereum#31198

* fix: fix data race of pricedList access

Use Reheap() instead of assigning a new pricedList to avoid
data race when clearing pool state.

Based on ethereum/go-ethereum#31758

* fix: resolve incorrect prestate for EIP-7702 authority accounts

Before: Authority accounts' prestate showed post-authorization code/storage
        and incorrect balance/nonce due to wrong capture timing
After:  Authority accounts' prestate correctly reflects state before
        SetCodeAuthorization is applied

- Refactor EVMLogger to pass env in CaptureTxStart instead of CaptureStart
- Capture authority accounts before delegation code (0xef0100...) is set
- Preserve original code/storage while restoring only gas from balance

* feat: prestate tracer lookup EIP-7702 delegation account

Parse and lookup the delegation account if EIP-7702 is enabled
in prestate tracer.

Implements ethereum/go-ethereum#32078
Based on ethereum/go-ethereum#32080

* refactor: expose sigHash as SigHash for SetCodeAuthorization

Rename the private sigHash() method to public SigHash() method
to enable external signing (e.g., MPC signing).

Based on ethereum/go-ethereum#32298
(which resolves ethereum/go-ethereum#32297)

* fix: resolve test failures in tracer and blobpool

- Add tracer lifecycle calls in runtime package
- Fix EIP-7702 and blobpool test configs

* Revert "chore: update Go version to 1.21"

This reverts commit 71e90d7.

* fix: support AuthorizationList in simulated backend gas estimation
eomti-wm added a commit to wemixarchive/go-wbft that referenced this pull request Mar 4, 2026
@eomti-wm
feat: EIP-7702 Set Code Transaction support (#42)
472009c 
* chore: update Go version to 1.21

* feat: implement EIP-7702 set code transaction type

Add support for EIP-7702 (SetCodeTx) which allows EOAs to temporarily
delegate their code execution to another account.

Changes include:
- Add SetCodeTx (type 0x04) transaction type and Authorization struct
- Add anzeonSigner for EIP-7702 transaction signing
- Implement delegation designator resolution in EVM
- Add EIP-7702 specific gas calculation for CALL variants
- Update EXTCODE* opcodes to handle delegation designators
- Add prestate tracer support for authorization list
- Update t8n tool and tests for SetCodeTx

Based on ethereum/go-ethereum#30078

* fix: complete EIP-7702 authorization marshaling and eth_call support

Based on ethereum/go-ethereum#30926

* refactor: improve EIP-7702 API naming

- Rename Authorization to SetCodeAuthorization
- Rename SignAuth to SignSetCode with key-first parameter order

Based on ethereum/go-ethereum#30933

* fix: rename SetCodeAuthorization 'v' to 'yParity'

The API spec requires the name yParity.

Based on:
- ethereum/go-ethereum@73a4ecf
- ethereum/go-ethereum#30936

* refactor: rename AuthList to SetCodeAuthorizations

Use SetCode prefix consistently in internal APIs for the authorization
list, as a follow-up to SetCodeAuthorization type renaming.

Based on ethereum/go-ethereum#30935

* refactor: change SetCodeTx.ChainID to uint256

Change ChainID field type from uint64 to uint256.Int in SetCodeTx and
SetCodeAuthorization for consistency with other chain ID handling.

Also removes unused json/gencodec tags from signature value fields
in DynamicFeeTx, BlobTx, SetCodeTx, and FeeDelegateDynamicFeeTx.

Based on ethereum/go-ethereum#30982

* fix: correct chainId check for anzeonSigner

Use zero value check for SetCodeTx chainId validation.
This aligns with cancunSigner and londonSigner as well.

Based on ethereum/go-ethereum#31032

* fix: initialize ChainID in SetCodeTx copy method

ChainID should be properly initialized and copied in the copy() method
to prevent nil pointer issues during transaction processing.

Based on ethereum/go-ethereum#31054

* refactor: remove EXTCODE* special handling for EIP-7702

EXTCODE* opcodes no longer need special overrides for EIP-7702 delegation
designators. The delegation designator is now returned as-is.

Implements EIPs#9248.

Based on ethereum/go-ethereum#31089

* feat: add EIP-7702 SetCode transaction pool support

Add SetCode transaction handling to legacypool:
- SetCodeTxType validation with Anzeon fork check
- Empty authorization rejection
- Authority tracking and conflict prevention
- Account slot limit for delegated accounts and pending authorizations

Note: tx replacement is not supported in go-stablenet. Tests are adjusted
to expect ErrAccountLimitExceeded instead of successful replacement.

Based on ethereum/go-ethereum#31073

* test: add setCodeTx reorg test

Based on ethereum/go-ethereum#31206

* refactor: move setcode tx validation into legacyPool

Move authorization-related validations from common validation to legacyPool
since these checks are specific to SetCode transactions.

Based on ethereum/go-ethereum#31209

* fix: fix error logs flood caused by removeAuthorities

Fix removeAuthorities to only iterate tx's authorities instead of
all authorities in the pool.

Based on ethereum/go-ethereum#31249

* refactor: reduce allocs in transaction signing

Add sigHash method to TxData interface and move hash computation
into each tx type, reducing allocations during signature operations.

Based on ethereum/go-ethereum#31258

* fix: reject gapped tx from delegated account

Add checkDelegationLimit to reject transactions with gapped nonces
from delegated accounts. Only one in-flight executable transaction
is allowed for senders with delegation or pending delegation.

Based on ethereum/go-ethereum#31430

* fix: exclude 7702 authorities from eth_createAccessList result

Exclude valid EIP-7702 authorities from access list generation result.

Based on ethereum/go-ethereum#31336
(which resolves ethereum/go-ethereum#31335)

* fix: data race in checkDelegationLimit

Add delegationTxsCount method with proper lock to fix data race
when accessing auths map in checkDelegationLimit.

Based on ethereum/go-ethereum#31475

* feat: add EIP-7702 protection to blobpool

Add delegation limit checks to blobpool:
- Limit senders with delegation/pending auth to one in-flight tx
- Check reserver for authority conflicts
- Refactor AddressReserver to Reserver struct with Hold/Release/Has

Based on ethereum/go-ethereum#31526

* fix: slot limit check and update comments

- Skip slot limit validation for tx replacements (same nonce reuses slot)
- Clarify cumulative balance validation skip reason: fee delegation compatibility

* fix: allow tx and authority regardless of admission order

Check reserver for authority conflicts to make authorization validation
independent of admission order. Authorities must not conflict with
addresses reserved by other subpools.

Based on ethereum/go-ethereum#31373

* fix: fix data race of txlookup access in test

Reset txlookup fields directly with proper locking instead of
assigning a new lookup object to avoid data race.

Based on ethereum/go-ethereum#31641

* feat: allow passing AuthorizationList to eth_call and eth_estimateGas

Add AuthorizationList field to CallMsg and toCallArg to support
SetCodeTx in eth_call and eth_estimateGas APIs.

Based on ethereum/go-ethereum#31198

* fix: fix data race of pricedList access

Use Reheap() instead of assigning a new pricedList to avoid
data race when clearing pool state.

Based on ethereum/go-ethereum#31758

* fix: resolve incorrect prestate for EIP-7702 authority accounts

Before: Authority accounts' prestate showed post-authorization code/storage
        and incorrect balance/nonce due to wrong capture timing
After:  Authority accounts' prestate correctly reflects state before
        SetCodeAuthorization is applied

- Refactor EVMLogger to pass env in CaptureTxStart instead of CaptureStart
- Capture authority accounts before delegation code (0xef0100...) is set
- Preserve original code/storage while restoring only gas from balance

* feat: prestate tracer lookup EIP-7702 delegation account

Parse and lookup the delegation account if EIP-7702 is enabled
in prestate tracer.

Implements ethereum/go-ethereum#32078
Based on ethereum/go-ethereum#32080

* refactor: expose sigHash as SigHash for SetCodeAuthorization

Rename the private sigHash() method to public SigHash() method
to enable external signing (e.g., MPC signing).

Based on ethereum/go-ethereum#32298
(which resolves ethereum/go-ethereum#32297)

* fix: resolve test failures in tracer and blobpool

- Add tracer lifecycle calls in runtime package
- Fix EIP-7702 and blobpool test configs

* Revert "chore: update Go version to 1.21"

This reverts commit 71e90d700023db9fc1e0a272d7ffb9207ede738f.

* fix: support AuthorizationList in simulated backend gas estimation
eomti-wm added a commit to wemixarchive/go-wbft that referenced this pull request Mar 5, 2026
@eomti-wm @vickkkkkyy
feat: EIP-7702 Set Code Transaction support (#170)
a802074 
* feat: EIP-7702 Set Code Transaction support (#42)

* chore: update Go version to 1.21

* feat: implement EIP-7702 set code transaction type

Add support for EIP-7702 (SetCodeTx) which allows EOAs to temporarily
delegate their code execution to another account.

Changes include:
- Add SetCodeTx (type 0x04) transaction type and Authorization struct
- Add anzeonSigner for EIP-7702 transaction signing
- Implement delegation designator resolution in EVM
- Add EIP-7702 specific gas calculation for CALL variants
- Update EXTCODE* opcodes to handle delegation designators
- Add prestate tracer support for authorization list
- Update t8n tool and tests for SetCodeTx

Based on ethereum/go-ethereum#30078

* fix: complete EIP-7702 authorization marshaling and eth_call support

Based on ethereum/go-ethereum#30926

* refactor: improve EIP-7702 API naming

- Rename Authorization to SetCodeAuthorization
- Rename SignAuth to SignSetCode with key-first parameter order

Based on ethereum/go-ethereum#30933

* fix: rename SetCodeAuthorization 'v' to 'yParity'

The API spec requires the name yParity.

Based on:
- ethereum/go-ethereum@73a4ecf
- ethereum/go-ethereum#30936

* refactor: rename AuthList to SetCodeAuthorizations

Use SetCode prefix consistently in internal APIs for the authorization
list, as a follow-up to SetCodeAuthorization type renaming.

Based on ethereum/go-ethereum#30935

* refactor: change SetCodeTx.ChainID to uint256

Change ChainID field type from uint64 to uint256.Int in SetCodeTx and
SetCodeAuthorization for consistency with other chain ID handling.

Also removes unused json/gencodec tags from signature value fields
in DynamicFeeTx, BlobTx, SetCodeTx, and FeeDelegateDynamicFeeTx.

Based on ethereum/go-ethereum#30982

* fix: correct chainId check for anzeonSigner

Use zero value check for SetCodeTx chainId validation.
This aligns with cancunSigner and londonSigner as well.

Based on ethereum/go-ethereum#31032

* fix: initialize ChainID in SetCodeTx copy method

ChainID should be properly initialized and copied in the copy() method
to prevent nil pointer issues during transaction processing.

Based on ethereum/go-ethereum#31054

* refactor: remove EXTCODE* special handling for EIP-7702

EXTCODE* opcodes no longer need special overrides for EIP-7702 delegation
designators. The delegation designator is now returned as-is.

Implements EIPs#9248.

Based on ethereum/go-ethereum#31089

* feat: add EIP-7702 SetCode transaction pool support

Add SetCode transaction handling to legacypool:
- SetCodeTxType validation with Anzeon fork check
- Empty authorization rejection
- Authority tracking and conflict prevention
- Account slot limit for delegated accounts and pending authorizations

Note: tx replacement is not supported in go-stablenet. Tests are adjusted
to expect ErrAccountLimitExceeded instead of successful replacement.

Based on ethereum/go-ethereum#31073

* test: add setCodeTx reorg test

Based on ethereum/go-ethereum#31206

* refactor: move setcode tx validation into legacyPool

Move authorization-related validations from common validation to legacyPool
since these checks are specific to SetCode transactions.

Based on ethereum/go-ethereum#31209

* fix: fix error logs flood caused by removeAuthorities

Fix removeAuthorities to only iterate tx's authorities instead of
all authorities in the pool.

Based on ethereum/go-ethereum#31249

* refactor: reduce allocs in transaction signing

Add sigHash method to TxData interface and move hash computation
into each tx type, reducing allocations during signature operations.

Based on ethereum/go-ethereum#31258

* fix: reject gapped tx from delegated account

Add checkDelegationLimit to reject transactions with gapped nonces
from delegated accounts. Only one in-flight executable transaction
is allowed for senders with delegation or pending delegation.

Based on ethereum/go-ethereum#31430

* fix: exclude 7702 authorities from eth_createAccessList result

Exclude valid EIP-7702 authorities from access list generation result.

Based on ethereum/go-ethereum#31336
(which resolves ethereum/go-ethereum#31335)

* fix: data race in checkDelegationLimit

Add delegationTxsCount method with proper lock to fix data race
when accessing auths map in checkDelegationLimit.

Based on ethereum/go-ethereum#31475

* feat: add EIP-7702 protection to blobpool

Add delegation limit checks to blobpool:
- Limit senders with delegation/pending auth to one in-flight tx
- Check reserver for authority conflicts
- Refactor AddressReserver to Reserver struct with Hold/Release/Has

Based on ethereum/go-ethereum#31526

* fix: slot limit check and update comments

- Skip slot limit validation for tx replacements (same nonce reuses slot)
- Clarify cumulative balance validation skip reason: fee delegation compatibility

* fix: allow tx and authority regardless of admission order

Check reserver for authority conflicts to make authorization validation
independent of admission order. Authorities must not conflict with
addresses reserved by other subpools.

Based on ethereum/go-ethereum#31373

* fix: fix data race of txlookup access in test

Reset txlookup fields directly with proper locking instead of
assigning a new lookup object to avoid data race.

Based on ethereum/go-ethereum#31641

* feat: allow passing AuthorizationList to eth_call and eth_estimateGas

Add AuthorizationList field to CallMsg and toCallArg to support
SetCodeTx in eth_call and eth_estimateGas APIs.

Based on ethereum/go-ethereum#31198

* fix: fix data race of pricedList access

Use Reheap() instead of assigning a new pricedList to avoid
data race when clearing pool state.

Based on ethereum/go-ethereum#31758

* fix: resolve incorrect prestate for EIP-7702 authority accounts

Before: Authority accounts' prestate showed post-authorization code/storage
        and incorrect balance/nonce due to wrong capture timing
After:  Authority accounts' prestate correctly reflects state before
        SetCodeAuthorization is applied

- Refactor EVMLogger to pass env in CaptureTxStart instead of CaptureStart
- Capture authority accounts before delegation code (0xef0100...) is set
- Preserve original code/storage while restoring only gas from balance

* feat: prestate tracer lookup EIP-7702 delegation account

Parse and lookup the delegation account if EIP-7702 is enabled
in prestate tracer.

Implements ethereum/go-ethereum#32078
Based on ethereum/go-ethereum#32080

* refactor: expose sigHash as SigHash for SetCodeAuthorization

Rename the private sigHash() method to public SigHash() method
to enable external signing (e.g., MPC signing).

Based on ethereum/go-ethereum#32298
(which resolves ethereum/go-ethereum#32297)

* fix: resolve test failures in tracer and blobpool

- Add tracer lifecycle calls in runtime package
- Fix EIP-7702 and blobpool test configs

* Revert "chore: update Go version to 1.21"

This reverts commit 71e90d700023db9fc1e0a272d7ffb9207ede738f.

* fix: support AuthorizationList in simulated backend gas estimation

* internal/ethapi: include AuthorizationList in gas estimation (#33849)

Fixes an issue where AuthorizationList wasn't copied over when
estimating gas for a user-provided transaction.

* test: fix EIP-7702 t8n test expected output

* refactor: remove redundant nil check

---------

Co-authored-by: vickkkkkyy <vickyaviles847@gmail.com>
gzliudan added a commit to gzliudan/XDPoSChain that referenced this pull request Mar 17, 2026
@gzliudan
fix(core/txpool/legacypool): fix data race of pricedList access ether…
967e322 
…eum#31758
@gzliudan gzliudan mentioned this pull request Mar 17, 2026
Merged
25 tasks
gzliudan added a commit to gzliudan/XDPoSChain that referenced this pull request Mar 17, 2026
@gzliudan
fix(core/txpool/legacypool): fix data race of pricedList access ether…
7128100 
…eum#31758
gzliudan added a commit to gzliudan/XDPoSChain that referenced this pull request Mar 18, 2026
@gzliudan
fix(core/txpool/legacypool): fix data race of pricedList access ether…
17d0484 
…eum#31758
gzliudan added a commit to gzliudan/XDPoSChain that referenced this pull request Mar 24, 2026
@gzliudan
fix(core/txpool/legacypool): fix data race of pricedList access ether…
13d5504 
…eum#31758
gzliudan added a commit to gzliudan/XDPoSChain that referenced this pull request Mar 24, 2026
@gzliudan
fix(core/txpool/legacypool): fix data race of pricedList access ether…
1bc881d 
…eum#31758
AnilChinchawale pushed a commit to XinFinOrg/XDPoSChain that referenced this pull request Mar 24, 2026
@gzliudan
fix(core/txpool/legacypool): fix data race of pricedList access ether…
83d644b 
…eum#31758 (#2203)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rjl493456442 rjl493456442 rjl493456442 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

1.15.11

Development

Successfully merging this pull request may close these issues.

2 participants

@mirokuratczyk @rjl493456442