Skip to content

core/txpool: ignore nil sub when subpool have been shut down#1915

Merged
buddh0 merged 1 commit intobnb-chain:masterfrom
node-real:ignore_nil_sub_subpool_down
Oct 12, 2023
Merged

core/txpool: ignore nil sub when subpool have been shut down#1915
buddh0 merged 1 commit intobnb-chain:masterfrom
node-real:ignore_nil_sub_subpool_down

Conversation

@NathanBSC
Copy link
Copy Markdown
Contributor

@NathanBSC NathanBSC commented Oct 11, 2023

Description

core/txpool: ignore nil sub when subpool have been shut down

Rationale

keep continue subscribing result of txs,
when stop geth, report error as following

panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x2 addr=0x18 pc=0x1003ea414]

goroutine 25748 [running]:
github.com/ethereum/go-ethereum/event.JoinSubscriptions.func1.2({0x0?, 0x0?})
	github.com/ethereum/go-ethereum/event/multisub.go:34 +0x34
created by github.com/ethereum/go-ethereum/event.JoinSubscriptions.func1
	github.com/ethereum/go-ethereum/event/multisub.go:32 +0x8c

It's caused by nil subscription when subpool has been shut down.

Example

add an example CLI or API response...

Changes

Notable changes:

  • add each change in a bullet point here
  • ...

@buddh0 buddh0 merged commit b86459a into bnb-chain:master Oct 12, 2023
@NathanBSC NathanBSC deleted the ignore_nil_sub_subpool_down branch November 7, 2023 10:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants