From d9bcba347b4658a543d61a63a8b2d13198cac859 Mon Sep 17 00:00:00 2001 From: Thomas Haller Date: Tue, 4 May 2021 22:58:13 +0200 Subject: [PATCH] firewall: fix adding duplicate iptables rules for shared mode nm_act_request_set_shared() already calls nm_utils_share_rules_apply(). Calling it twice, is pretty bad because during deactivate we will only remove one of each duplicate rule. Fixes: 701654b93064 ('core: refactor tracking of shared-rules to use NMUtilsShareRules') (cherry picked from commit 60744889e22bec3fd03a59c66619294a9e0437f4) --- src/core/devices/nm-device.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/core/devices/nm-device.c b/src/core/devices/nm-device.c index 163fb8a175..086fd4ddfb 100644 --- a/src/core/devices/nm-device.c +++ b/src/core/devices/nm-device.c @@ -11678,8 +11678,6 @@ start_sharing(NMDevice *self, NMIP4Config *config, GError **error) nm_utils_share_rules_add_all_rules(share_rules, ip_iface, ip4_addr->address, ip4_addr->plen); - nm_utils_share_rules_apply(share_rules, TRUE); - nm_act_request_set_shared(req, share_rules); conn = nm_act_request_get_applied_connection(req);