Skip to content

RDKB-66007: iptables error in MAP-T + bridgemode#368

Merged
snayak002c merged 1 commit into
developfrom
fix/RDKB-66007
Jul 16, 2026
Merged

RDKB-66007: iptables error in MAP-T + bridgemode#368
snayak002c merged 1 commit into
developfrom
fix/RDKB-66007

Conversation

@sivaraj-comcast

Copy link
Copy Markdown
Contributor

RDKB-66007: iptables error in MAP-T + bridgemode causing increase in SYS_SH_IPTableFW_restart marker

Reason for change: Declare postrouting chains for MAPT + bridgemode path
Test Procedure: Verify there is no iptables error on iptable restart and SNAT rules are getting added when in MAP-T + Bridgemode
Risks: Low
Priority: P1
Signed-off-by: Sivaraj_Sivalingam@comcast.com

RDKB-66007:  Declare postrouting chains for MAPT + bridgemode path
Copilot AI review requested due to automatic review settings July 16, 2026 16:05
@sivaraj-comcast
sivaraj-comcast requested review from a team as code owners July 16, 2026 16:05

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Addresses RDKB-66007 by preventing iptables-restore errors during IPv4 firewall restarts when MAP‑T is active in bridge mode, by ensuring required MAP‑T NAT chains exist in the “firewall disabled” generation path.

Changes:

  • Declare MAP‑T IPv4 postrouting protocol-specific chains (TCP/UDP/ICMP) in the disabled IPv4 firewall NAT table when MAP‑T is ready.
  • Ensure POSTROUTING jump to the MAP‑T postrouting chain remains present for the NAT46 interface in this path.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread source/firewall/firewall.c
@snayak002c
snayak002c merged commit a8113e0 into develop Jul 16, 2026
13 of 14 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 16, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants