-
Notifications
You must be signed in to change notification settings - Fork 20.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
eth/downloader: ignore zero size header batch for importing #24569
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, but we might want to also add an early return in ValidateHeaderChain
. If chain is empty, return an error
a94c2bd
to
320e881
Compare
|
…#24569) * eth/downloader: ignore zero size header batch for importing * core, light: reject empty header batch for importing
…#24569) * eth/downloader: ignore zero size header batch for importing * core, light: reject empty header batch for importing
…#24569) * eth/downloader: ignore zero size header batch for importing * core, light: reject empty header batch for importing
…#24569) * eth/downloader: ignore zero size header batch for importing * core, light: reject empty header batch for importing
This PR fixes a panic by importing zero size header batch. It can happen when the imported headers are crossing the ttd border and all the synced headers are post-merge.