Merge pull request #412 from thomas-mangin/T2475

flake8: T2475: fix a number of issue reported by flake8

3 files changed, 5 insertions, 7 deletions

diff --git a/src/conf_mode/interfaces-tunnel.py b/src/conf_mode/interfaces-tunnel.py
index 5261b6e7d..8e9bb069e 100755
--- a/src/conf_mode/interfaces-tunnel.py
+++ b/src/conf_mode/interfaces-tunnel.py
@@ -222,7 +222,7 @@ class ConfigurationState(Config):
         remove all the values which were not changed from the default
         """
         for option in options:
-            if self.exists(option) and self_return_value(option) != self.default[option]:
+            if self.exists(option) and self.self_return_value(option) != self.default[option]:
                 continue
             del self.options[option]
 
@@ -555,7 +555,7 @@ def verify(conf):
     # source-interface check
 
     if tun_dev and tun_dev not in options['interfaces']:
-        raise ConfigError(f'device "{dev}" does not exist')
+        raise ConfigError(f'device "{tun_dev}" does not exist')
 
     # tunnel encapsulation check
 
diff --git a/src/conf_mode/ipsec-settings.py b/src/conf_mode/ipsec-settings.py
index 3398bcdf2..6282c2cc7 100755
--- a/src/conf_mode/ipsec-settings.py
+++ b/src/conf_mode/ipsec-settings.py
@@ -99,7 +99,7 @@ def get_config():
 def write_ipsec_secrets(c):
     if c.get("ipsec_l2tp_auth_mode") == "pre-shared-secret":
         secret_txt = "{0}\n{1} %any : PSK \"{2}\"\n{3}\n".format(delim_ipsec_l2tp_begin, c['outside_addr'], c['ipsec_l2tp_secret'], delim_ipsec_l2tp_end)
-    elif data.get("ipsec_l2tp_auth_mode") == "x509":
+    elif c.get("ipsec_l2tp_auth_mode") == "x509":
         secret_txt = "{0}\n: RSA {1}\n{2}\n".format(delim_ipsec_l2tp_begin, c['server_key_file_copied'], delim_ipsec_l2tp_end)
 
     old_umask = os.umask(0o077)
diff --git a/src/op_mode/show_interfaces.py b/src/op_mode/show_interfaces.py
index 7041c7e16..2f0f8a1c9 100755
--- a/src/op_mode/show_interfaces.py
+++ b/src/op_mode/show_interfaces.py
@@ -267,17 +267,15 @@ def run_show_counters(ifnames, iftypes, vif, vrrp):
 
 
 @register('clear')
-def run_clear_intf(intf, iftypes, vif, vrrp):
+def run_clear_intf(ifnames, iftypes, vif, vrrp):
     for interface in filtered_interfaces(ifnames, iftypes, vif, vrrp):
         print(f'Clearing {interface.ifname}')
-        interface = Interface(ifname, create=False, debug=False)
         interface.operational.clear_counters()
 
 
 @register('reset')
-def run_reset_intf(intf, iftypes, vif, vrrp):
+def run_reset_intf(ifnames, iftypes, vif, vrrp):
     for interface in filtered_interfaces(ifnames, iftypes, vif, vrrp):
-        interface = Interface(ifname, create=False, debug=False)
         interface.operational.reset_counters()