Add deep_copy calls missing because of a Y2R bug

See yast/y2r#47.
yast · Oct 18, 2013 · e0a92ff · e0a92ff
1 parent 6e78b05
commit e0a92ff
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/src/clients/ldap_browser.rb b/src/clients/ldap_browser.rb
@@ -203,7 +203,7 @@ def main
               )
             ) do |c|
               @i = Ops.add(@i, 1)
-              @i == @conf ? @configuration : c
+              @i == @conf ? deep_copy(@configuration) : deep_copy(c)
             end
           end
         end

diff --git a/src/modules/Ldap.rb b/src/modules/Ldap.rb
@@ -3236,7 +3236,9 @@ def Write(abort)
     def WriteNow
       abort = lambda { false }
 
-      needed_packages = @sssd ? @sssd_packages : @pam_nss_packages
+      needed_packages = @sssd ?
+        deep_copy(@sssd_packages) :
+        deep_copy(@pam_nss_packages)
       if @sssd_with_krb
         needed_packages = Convert.convert(
           Builtins.union(needed_packages, @kerberos_packages),

diff --git a/src/modules/LdapPopup.rb b/src/modules/LdapPopup.rb
@@ -222,7 +222,7 @@ def BrowseTree(root_dn)
     def InitAndBrowseTree(root_dn, connection)
       connection = deep_copy(connection)
       args = Ops.greater_than(Builtins.size(connection), 0) ?
-        connection :
+        deep_copy(connection) :
         {
           "hostname"   => Ldap.GetFirstServer(Ldap.server),
           "port"       => Ldap.GetFirstPort(Ldap.server),