firewall4 icon indicating copy to clipboard operation
firewall4 copied to clipboard

Use symbolic ICMP types in MLD rule

Open jonathanunderwood opened this issue 2 years ago • 3 comments

Symbolic ICMP types for MLD were added in commit e6e82a55206cf7017f26b92f7097f779161b5cac. This commit updates the config file to use them.

jonathanunderwood avatar Jan 07 '24 23:01 jonathanunderwood

@jow- unsure if this needs tests updates, but I am struggling to parse the tests - any pointers much appreciated.

jonathanunderwood avatar Jan 08 '24 00:01 jonathanunderwood

Build and install ucode and run sh script in top level.

brada4 avatar Jan 08 '24 00:01 brada4

@jow- any chance we can get this merged for 24.10?

jonathanunderwood avatar Oct 08 '24 09:10 jonathanunderwood

@jow- would be great if we could get this into 24.10 (now on rc4) - it's a minimal change and has been working fine for me locally.

jonathanunderwood avatar Dec 29 '24 09:12 jonathanunderwood

Does it work with firewall3?

brada4 avatar Dec 29 '24 13:12 brada4

From rfc4890 - Recommendations for Filtering ICMPv6 Messages in Firewalls

4.3. Recommendations for ICMPv6 Transit Traffic

4.3.3. Traffic That Will Be Dropped Anyway -- No Special Attention Needed

Link-local multicast receiver notification messages (must have link-local source address): o Listener Query (Type 130) o Listener Report (Type 131) o Listener Done (Type 132) o Listener Report v2 (Type 143)

4.4. Recommendations for ICMPv6 Local Configuration Traffic

4.4.1. Traffic That Must Not Be Dropped

Link-Local Multicast Receiver Notification messages: o Listener Query (Type 130) o Listener Report (Type 131) o Listener Done (Type 132) o Listener Report v2 (Type 143)

According to the content of this document, it seems that these types of ICMP packets coming from the WAN can be safely dropped. On the other hand, for packets coming from the LAN, they need to remain in the Accept state. However, since the current firewall policy allows all traffic from the LAN, there is no issue.

CallMeR avatar Dec 30 '24 16:12 CallMeR

https://github.com/openwrt/firewall3/blob/master/icmp_codes.h

brada4 avatar Jan 13 '25 20:01 brada4

Merged, thanks.

jow- avatar Mar 17 '25 15:03 jow-