diff options
| author | Thomas Mangin <thomas.mangin@exa.net.uk> | 2020-02-29 00:57:58 +0000 |
|---|---|---|
| committer | Thomas Mangin <thomas.mangin@exa.net.uk> | 2020-02-29 00:57:58 +0000 |
| commit | 9213475ba47ee15e155e89026778f42135aead12 (patch) | |
| tree | 976d06dd151b669eca23293f8c7e979a0c311d5c | |
| parent | 7d3176beb07ab1faa156c2da76384186376cf25d (diff) | |
| download | vyos-1x-9213475ba47ee15e155e89026778f42135aead12.tar.gz vyos-1x-9213475ba47ee15e155e89026778f42135aead12.zip | |
ifconfig: T2082: fix checking of argument passed
| -rw-r--r-- | python/vyos/ifconfig.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/python/vyos/ifconfig.py b/python/vyos/ifconfig.py index 71e223a01..b1f9baabe 100644 --- a/python/vyos/ifconfig.py +++ b/python/vyos/ifconfig.py @@ -96,13 +96,13 @@ class Interface: self.config = deepcopy(self.default) self.config['ifname'] = ifname - for k in self.options: - if k not in kargs: + for k in kargs: + if k not in self.options: raise ConfigError('invalid option {} for {}'.format(k,self.__class__)) self.config[k] = kargs[k] for k in self.required: - if k not in self.config: + if k not in kargs: raise ConfigError('missing required option {} for {}'.format(k,self.__class__)) if not os.path.exists('/sys/class/net/{}'.format(self.config['ifname'])): |