diff options
| author | Martin Basti <mbasti@redhat.com> | 2017-01-26 10:25:46 +0100 |
|---|---|---|
| committer | Jan Cholasta <jcholast@redhat.com> | 2017-02-08 08:32:44 +0100 |
| commit | 03d0a55e8a21a334ca4dc625527cae93633a7314 (patch) | |
| tree | cd45652ee6a1184293240732c8b6f5afcada693a /ipaserver/plugins/dns.py | |
| parent | a3d3b0ad2537c9d11d9c6108c31e079f0dfcf31c (diff) | |
| download | freeipa-03d0a55e8a21a334ca4dc625527cae93633a7314.tar.gz freeipa-03d0a55e8a21a334ca4dc625527cae93633a7314.tar.xz freeipa-03d0a55e8a21a334ca4dc625527cae93633a7314.zip | |
py3: DNS: get_record_entry_attrs: do not modify dict during iteration
In py3 keys() doesn't return list but iterator so it must be transformed
to tuple otherwise iterator will be broken.
https://fedorahosted.org/freeipa/ticket/4985
Reviewed-By: Jan Cholasta <jcholast@redhat.com>
Diffstat (limited to 'ipaserver/plugins/dns.py')
| -rw-r--r-- | ipaserver/plugins/dns.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ipaserver/plugins/dns.py b/ipaserver/plugins/dns.py index 0838161c3..97f6527d0 100644 --- a/ipaserver/plugins/dns.py +++ b/ipaserver/plugins/dns.py @@ -3195,7 +3195,7 @@ class dnsrecord(LDAPObject): def get_record_entry_attrs(self, entry_attrs): entry_attrs = entry_attrs.copy() - for attr in entry_attrs.keys(): + for attr in tuple(entry_attrs.keys()): if attr not in self.params or self.params[attr].primary_key: del entry_attrs[attr] return entry_attrs |