Merge pull request #3084 from craigcomstock/ent-13530/3.24

Fixed cfruncommand for Windows causing "Too many arguments" error (3.24)

Author: craigcomstock

CHANGELOG.md

@@ -1,4 +1,5 @@
 3.24.3:
+	- Fixed cfruncommand for Windows causing "Too many arguments" error (ENT-13530)
 	- Added dmidecode to well known paths for Red Hat (ENT-12988)
 	- Added recommendation about nfs server and consistent use of root dot
 	  (ENT-13223)

controls/cf_serverd.cf

@@ -68,15 +68,18 @@ body server control
 
       allowusers => { @(def.control_server_allowusers) };
 
+    # In 3.10 the quotation is properly closed when EOF is reached. It is left
+    # open so that arguments (like -K and --remote-bundles) can be appended.
+    # 3.10.x does not automatically append -I -Dcfruncommand
+
     windows::
-      cfruncommand => "$(sys.cf_agent) -I -D cf_runagent_initiated -f \"$(sys.update_policy_path)\"  &
-                       $(sys.cf_agent) -I -D cf_runagent_initiated";
 
-    !windows::
+        # /s removes the first and last quote characters and aids in specifying paths with surrounding quotes
+        cfruncommand => "$(def.cf_runagent_shell) /s /c \"
+                           $(sys.cf_agent) -I -D cf_runagent_initiated -f $(sys.update_policy_path)  &
+                           $(sys.cf_agent) -I -D cf_runagent_initiated";
 
-      # In 3.10 the quotation is properly closed when EOF is reached. It is left
-      # open so that arguments (like -K and --remote-bundles) can be appended.
-      # 3.10.x does not automatically append -I -Dcfruncommand
+    !windows::
 
         cfruncommand => "$(def.cf_runagent_shell) -c \'
                            $(sys.cf_agent) -I -D cf_runagent_initiated -f $(sys.update_policy_path)  ;
@@ -172,7 +175,7 @@ bundle server mpf_default_access_rules()
         shortcut => "hub_cmdb",
         admit_keys => { $(connection.key) };
 
-    !windows::
+    any::
       "$(def.cf_runagent_shell)" -> { "ENT-6673" }
       handle => "server_access_grant_access_shell_cmd",
       comment => "Grant access to shell for cfruncommand",

controls/def.cf

@@ -523,6 +523,12 @@ bundle common def
         comment => "Define path to shell used by cf-runagent",
         handle  => "common_def_vars_solaris_cf_runagent_shell";
 
+    windows::
+      "cf_runagent_shell"
+        string  => "${sys.winsysdir}${const.dirsep}cmd.exe",
+        comment => "Define path to shell used by cf-runagent",
+        handle  => "common_def_vars_windows_cf_runagent_shell";
+
     !(windows|solaris)::
       "cf_runagent_shell"
         string  => "/bin/sh",