Page MenuHomeFreeBSD
Differential D44480

icmp6: rate limit our echo replies
ClosedPublic
Actions

Authored by glebius on Mar 22 2024, 9:49 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Dec 14, 8:09 AM
Unknown Object (File)
Sun, Nov 23, 5:36 AM
Unknown Object (File)
Sat, Nov 22, 11:55 AM
Unknown Object (File)
Wed, Nov 19, 7:06 PM
Unknown Object (File)
Oct 23 2025, 10:33 PM
Unknown Object (File)
Oct 20 2025, 5:05 PM
Unknown Object (File)
Oct 17 2025, 3:44 AM
Unknown Object (File)
Oct 14 2025, 11:36 PM
View All 93 Files
Subscribers
ae
imp
melifaro
zlei

Details

Reviewers
tuexen
zlei
Group Reviewers
network
Commits
rGa7ad6437193a: icmp6: rate limit our echo replies
rG90ecc3fc679d: icmp6: rate limit our echo replies
rGf6561ff8ee10: icmp6: rate limit our echo replies
rG32aeee8ce7e7: icmp6: rate limit our echo replies
Summary

The generation of ICMP6_ECHO_REPLY bypasses icmp6_error(), thus rate
limit was not applied.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 56751
Build 53639: arc lint + arc unit

Event Timeline

glebius created this revision.Mar 22 2024, 9:49 PM
Herald added subscribers: melifaro, ae, imp. · View Herald TranscriptMar 22 2024, 9:49 PM
glebius requested review of this revision.Mar 22 2024, 9:49 PM
Harbormaster completed remote builds in B56751: Diff 136109.Mar 22 2024, 9:49 PM
glebius added a parent revision: D44479: icmp6: make icmp6_ratelimit() responsible to update the stats counter.Mar 22 2024, 9:49 PM
glebius added a child revision: D44481: icmp6: move ICMPv6 related tunables to the files where they are used.
tuexen accepted this revision.Mar 23 2024, 2:21 PM
This revision is now accepted and ready to land.Mar 23 2024, 2:21 PM
zlei accepted this revision.Mar 24 2024, 2:59 PM
zlei added a subscriber: zlei.

Generally looks good to me.

sys/netinet6/icmp6.c
549

And do not forget to update statistic icp6s_toofreq if D44479 is abandoned.

Closed by commit rG32aeee8ce7e7: icmp6: rate limit our echo replies (authored by glebius). · Explain WhyMar 24 2024, 4:22 PM
This revision was automatically updated to reflect the committed changes.
glebius added a commit: rG32aeee8ce7e7: icmp6: rate limit our echo replies.
zlei added a commit: rGf6561ff8ee10: icmp6: rate limit our echo replies.Jun 26 2024, 4:52 AM
tuexen added a commit: rG90ecc3fc679d: icmp6: rate limit our echo replies.Feb 12 2025, 10:29 AM
tuexen added a commit: rGa7ad6437193a: icmp6: rate limit our echo replies.Feb 13 2025, 4:47 PM

Revision Contents
Changeset List

PathSize
sys/
netinet6/
2 lines

Diff 136109

View Options

sys/netinet6/icmp6.c

Loading...