Fixed a condition that was always true.

The impact of the bug was small because other code queried DNS anyway and filtered out the nonexistent host name.
yast · Jan 8, 2015 · c422332 · c422332
1 parent 682f6fb
commit c422332
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/include/nfs/ui.rb b/src/include/nfs/ui.rb
@@ -147,7 +147,7 @@ def ProposeHostname
       ret = ""
       cur_domain = Hostname.CurrentDomain
 
-      ret = Ops.add("nfs.", cur_domain) if cur_domain || cur_domain != ""
+      ret = "nfs.#{cur_domain}" if cur_domain && cur_domain != ""
       ret
     end
 

diff --git a/src/modules/Nfs.rb b/src/modules/Nfs.rb
@@ -387,7 +387,7 @@ def WriteOnly
           Convert.convert(
             Builtins.union(
               entry,
-              "freq" => 0, "passno" => 0 # "vfstype": "nfs",
+              "freq" => 0, "passno" => 0
             ),
             :from => "map",
             :to   => "map <string, any>"