summaryrefslogtreecommitdiff
path: root/src/conf_mode
diff options
context:
space:
mode:
authorChristian Breunig <christian@breunig.cc>2023-10-03 20:51:21 +0200
committerMergify <37929162+mergify[bot]@users.noreply.github.com>2023-10-07 17:15:00 +0000
commit713647429b981f35822e8b4c9fafe990d120643c (patch)
tree24212ec13751537430b99d63d283620cdc0acc73 /src/conf_mode
parente4fabffe740800d2ad9ded4aec4b768485289653 (diff)
downloadvyos-1x-713647429b981f35822e8b4c9fafe990d120643c.tar.gz
vyos-1x-713647429b981f35822e8b4c9fafe990d120643c.zip
pppoe: T5630: verify MRU is less or equal then MTU
(cherry picked from commit e357258e645cf85de0035d4ecfbf99db4dd90f7e)
Diffstat (limited to 'src/conf_mode')
-rwxr-xr-xsrc/conf_mode/interfaces-pppoe.py5
1 files changed, 5 insertions, 0 deletions
diff --git a/src/conf_mode/interfaces-pppoe.py b/src/conf_mode/interfaces-pppoe.py
index fca91253c..0a03a172c 100755
--- a/src/conf_mode/interfaces-pppoe.py
+++ b/src/conf_mode/interfaces-pppoe.py
@@ -77,6 +77,11 @@ def verify(pppoe):
if {'connect_on_demand', 'vrf'} <= set(pppoe):
raise ConfigError('On-demand dialing and VRF can not be used at the same time')
+ # both MTU and MRU have default values, thus we do not need to check
+ # if the key exists
+ if int(pppoe['mru']) > int(pppoe['mtu']):
+ raise ConfigError('PPPoE MRU needs to be lower then MTU!')
+
return None
def generate(pppoe):