summaryrefslogtreecommitdiff
path: root/src/conf_mode
diff options
context:
space:
mode:
authorViacheslav Hletenko <v.gletenko@vyos.io>2022-07-12 08:39:36 +0000
committerChristian Poessinger <christian@poessinger.com>2022-07-18 08:14:33 +0200
commit3a1c690e22e397bbdecf47364c3f885df7a8e8ef (patch)
treebae5603042684e17cb2a77959fd63e6af1bcd975 /src/conf_mode
parent51455fc033cd41e1a105800b2a901e4a32df6058 (diff)
downloadvyos-1x-3a1c690e22e397bbdecf47364c3f885df7a8e8ef.tar.gz
vyos-1x-3a1c690e22e397bbdecf47364c3f885df7a8e8ef.zip
vrf: T4527: Prevent to create VRF with reserved names
VRF names: "add, all, broadcast, default, delete, dev, get, inet, mtu, link, type, vrf" are reserved and cannot be used for vrf name (cherry picked from commit 52342f389af2da2995b858d026e6fbcad5c8bfaa)
Diffstat (limited to 'src/conf_mode')
-rwxr-xr-xsrc/conf_mode/vrf.py6
1 files changed, 6 insertions, 0 deletions
diff --git a/src/conf_mode/vrf.py b/src/conf_mode/vrf.py
index fb2182fff..def4cc70d 100755
--- a/src/conf_mode/vrf.py
+++ b/src/conf_mode/vrf.py
@@ -108,8 +108,14 @@ def verify(vrf):
f'static routes installed!')
if 'name' in vrf:
+ reserved_names = ["add", "all", "broadcast", "default", "delete", "dev", "get", "inet", "mtu", "link", "type",
+ "vrf"]
table_ids = []
for name, config in vrf['name'].items():
+ # Reserved VRF names
+ if name in reserved_names:
+ raise ConfigError(f'VRF name "{name}" is reserved and connot be used!')
+
# table id is mandatory
if 'table' not in config:
raise ConfigError(f'VRF "{name}" table id is mandatory!')