From f916f40ee9a726159c063952b879f8bce90c378c Mon Sep 17 00:00:00 2001 From: sarthurdev <965089+sarthurdev@users.noreply.github.com> Date: Sat, 3 Dec 2022 12:20:33 +0100 Subject: firewall: T478: Fix firewall group circular dependency check --- src/conf_mode/firewall.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'src/conf_mode/firewall.py') diff --git a/src/conf_mode/firewall.py b/src/conf_mode/firewall.py index 9fee20358..ae3dd6414 100755 --- a/src/conf_mode/firewall.py +++ b/src/conf_mode/firewall.py @@ -279,6 +279,8 @@ def verify_nested_group(group_name, group, groups, seen): if 'include' not in group: return + seen.append(group_name) + for g in group['include']: if g not in groups: raise ConfigError(f'Nested group "{g}" does not exist') @@ -286,8 +288,6 @@ def verify_nested_group(group_name, group, groups, seen): if g in seen: raise ConfigError(f'Group "{group_name}" has a circular reference') - seen.append(g) - if 'include' in groups[g]: verify_nested_group(g, groups[g], groups, seen) -- cgit v1.2.3