diff options
| author | Viacheslav Hletenko <v.gletenko@vyos.io> | 2022-09-16 07:13:59 +0000 | 
|---|---|---|
| committer | Viacheslav Hletenko <v.gletenko@vyos.io> | 2022-09-16 07:13:59 +0000 | 
| commit | 1b3ded9fb2846e20f3bec2a0e21dc4cec3a09139 (patch) | |
| tree | 2fc0ea8d42a8f535d161df047c4870b76c6acc1a | |
| parent | 7a2ad35ec8ecd84d162a3e5dc41b50f31d18dc10 (diff) | |
| download | vyos-1x-1b3ded9fb2846e20f3bec2a0e21dc4cec3a09139.tar.gz vyos-1x-1b3ded9fb2846e20f3bec2a0e21dc4cec3a09139.zip | |
policy-route: T4697: Add missing rule_id for verify_rule func
There is a missing 'rule_id' in verify_rule() function
We call it from the loop but don't provide argument 'rule_id'
It cause "NameError: name 'rule_id' is not defined"
Fix it
| -rwxr-xr-x | src/conf_mode/policy-route.py | 4 | 
1 files changed, 2 insertions, 2 deletions
| diff --git a/src/conf_mode/policy-route.py b/src/conf_mode/policy-route.py index 9fddbd2c6..00539b9c7 100755 --- a/src/conf_mode/policy-route.py +++ b/src/conf_mode/policy-route.py @@ -92,7 +92,7 @@ def get_config(config=None):      return policy -def verify_rule(policy, name, rule_conf, ipv6): +def verify_rule(policy, name, rule_conf, ipv6, rule_id):      icmp = 'icmp' if not ipv6 else 'icmpv6'      if icmp in rule_conf:          icmp_defined = False @@ -166,7 +166,7 @@ def verify(policy):              for name, pol_conf in policy[route].items():                  if 'rule' in pol_conf:                      for rule_id, rule_conf in pol_conf['rule'].items(): -                        verify_rule(policy, name, rule_conf, ipv6) +                        verify_rule(policy, name, rule_conf, ipv6, rule_id)      for ifname, if_policy in policy['interfaces'].items():          name = dict_search_args(if_policy, 'route') | 
