From ab2aeec41a2ef34730b9dc8894b184e0bf00b147 Mon Sep 17 00:00:00 2001
From: Christian Breunig <christian@breunig.cc>
Date: Tue, 3 Oct 2023 20:51:21 +0200
Subject: pppoe: T5630: verify MRU is less or equal then MTU

(cherry picked from commit e357258e645cf85de0035d4ecfbf99db4dd90f7e)
---
 src/conf_mode/interfaces-pppoe.py | 5 +++++
 1 file changed, 5 insertions(+)

(limited to 'src')

diff --git a/src/conf_mode/interfaces-pppoe.py b/src/conf_mode/interfaces-pppoe.py
index 6c4c6c95b..49714c558 100755
--- a/src/conf_mode/interfaces-pppoe.py
+++ b/src/conf_mode/interfaces-pppoe.py
@@ -59,6 +59,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):
-- 
cgit v1.2.3