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
Handle already discarded packets in discard_excess_packets #22594
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.
nice catch
Codecov Report
@@ Coverage Diff @@
## master #22594 +/- ##
=========================================
+ Coverage 81.1% 81.5% +0.4%
=========================================
Files 560 553 -7
Lines 151206 149655 -1551
=========================================
- Hits 122633 122047 -586
+ Misses 28573 27608 -965 |
fd17320
to
6c68f66
Compare
(cherry picked from commit 38b02bb) # Conflicts: # core/src/sigverify_stage.rs
(cherry picked from commit 38b02bb)
(cherry picked from commit 38b02bb)
(cherry picked from commit 38b02bb)
(cherry picked from commit 38b02bb)
@Mergifyio backport v1.9 |
✅ Backports have been created
|
I just rediscovered this bug on v1.9.. turns out the v1.9 backport was never merged so v1.9 hasn't been running this fix. New PR here: #24926 |
Problem
Packets with .discard() == true are considered as valid by discard_excess_packets, but they have already been dropped as dupes.
Summary of Changes
Don't consider discarded packets
Fixes #