From d6baf248fea9256a8142fb66e2bf9b19e1831cea Mon Sep 17 00:00:00 2001 From: Christian Poessinger Date: Sun, 27 Oct 2019 23:12:10 +0100 Subject: snmp: fix verify() bail out early order (cherry picked from commit ed642ff5e958c6ef43dee5ef684fb5ccf85ad8cf) --- src/conf_mode/snmp.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/conf_mode/snmp.py b/src/conf_mode/snmp.py index fbffce751..edacc17ea 100755 --- a/src/conf_mode/snmp.py +++ b/src/conf_mode/snmp.py @@ -525,10 +525,6 @@ def verify(snmp): if snmp is None: return None - # bail out early if SNMP v3 is not configured - if not snmp['v3_enabled']: - return None - for listen in snmp['listen_address']: addr = listen[0] port = listen[1] @@ -547,6 +543,10 @@ def verify(snmp): else: print('WARNING: SNMP listen address {0} not configured!'.format(addr)) + # bail out early if SNMP v3 is not configured + if not snmp['v3_enabled']: + return None + if 'v3_groups' in snmp.keys(): for group in snmp['v3_groups']: # -- cgit v1.2.3