diff options
| author | Jose Phillips <jose@latinol.com> | 2026-05-07 16:01:29 -0500 |
|---|---|---|
| committer | Jose Phillips <jose@latinol.com> | 2026-05-13 19:36:11 +0000 |
| commit | df2a81b27971ec9bb2596adfa25405e537516ac4 (patch) | |
| tree | 3398167b5d688ffe8a10b2ece5ae073586d0562c /src | |
| parent | 72bf69a9eaeeaecd86b39d536582a204029af72b (diff) | |
| download | vyos-1x-df2a81b27971ec9bb2596adfa25405e537516ac4.tar.gz vyos-1x-df2a81b27971ec9bb2596adfa25405e537516ac4.zip | |
T8823: Removed nullchecks because dict_Search return None if value not set
Diffstat (limited to 'src')
| -rwxr-xr-x | src/conf_mode/policy.py | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/src/conf_mode/policy.py b/src/conf_mode/policy.py index 90ecc4a4c..171ef23fd 100755 --- a/src/conf_mode/policy.py +++ b/src/conf_mode/policy.py @@ -170,20 +170,18 @@ def verify(config_dict): mask_len = int(rule_config['prefix'].split('/')[1]) ge = dict_search('ge', rule_config) le = dict_search('le', rule_config) - ge = int(ge) if ge is not None else None - le = int(le) if le is not None else None - if ge is not None and ge < mask_len: + if ge and int(ge) < mask_len: raise ConfigError( f'{policy_hr} {instance} rule {rule}: "ge" ({ge}) must be >= ' f'prefix length ({mask_len})' ) - if le is not None and le < mask_len: + if le and int(le) < mask_len: raise ConfigError( f'{policy_hr} {instance} rule {rule}: "le" ({le}) must be >= ' f'prefix length ({mask_len})' ) - if ge is not None and le is not None and ge > le: + if ge and le and int(ge) > int(le): raise ConfigError( f'{policy_hr} {instance} rule {rule}: "ge" ({ge}) must be <= ' f'"le" ({le})' |
