Skip to content
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

[uci config] Drop unaccounted icmp6 packets #44

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

brada4
Copy link

@brada4 brada4 commented Feb 1, 2025

Drop ICMPv6 packets that are not explicitly allowed, like falling outside conntrack due to missing embedded state header like in referenced issue

Add older patch and omit singular echo reply as it is traditionally seen only in conntrack state only initiated by echo request.

Fixes: openwrt/openwrt#17800
Supersedes: openwrt/openwrt#17805
Supersedes: #38
Signed-off-by: Andris PE [email protected]

Drop ICMPv6 packets that are not explicitly allowed, like falling
outside conntrack due to missing embedded state header like in
referenced issue

Add older patch and omit singular echo reply as it is traditionally seen only in
conntrack state only initiated by echo request

Fixes: openwrt/openwrt#17800
Supersedes: openwrt/openwrt#17805
Supersedes: openwrt#38
Signed-off-by:
@brada4
Copy link
Author

brada4 commented Feb 1, 2025

Tested on both fw3 and fw4. Test machine with lan mtu 64 to chop icmp embedded extra header
@hauke @jow- this has to be pushed to fw3 and picked to both stable branches. Thank you in advance.

@f00b4r0
Copy link

f00b4r0 commented Feb 16, 2025

I'd suggest closing #38 if it's not longer relevant: there were no 3rd-party comments there and this will ease maintainer's workload.

@brada4
Copy link
Author

brada4 commented Feb 16, 2025

Kind of does not change much... I linked it to shoot all at once.

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.

ssh stuck at expecting SSH2_MSG_KEX_ECDH_REPLY
2 participants