Do not modify devmap in place in NetworkInterfaces#filter_interfacetype

yast · May 18, 2016 · d85952e · d85952e
1 parent 22378e3
commit d85952e
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/library/network/src/modules/NetworkInterfaces.rb b/library/network/src/modules/NetworkInterfaces.rb
@@ -641,7 +641,8 @@ def CanonicalizeIP(ifcfg)
     # - using INTERFACETYPE is reported as a warning by wicked (it asks for reporting a bug)
     # - it is often ignored by wicked
     def filter_interfacetype(devmap)
-      devmap.delete_if { |k, v| k == "INTERFACETYPE" && !["lo", "dummy"].include?(v) }
+      ret = deep_copy(devmap)
+      ret.delete_if { |k, v| k == "INTERFACETYPE" && !["lo", "dummy"].include?(v) }
     end
 
     # Conceal secret information, such as WEP keys, so that the output