Fixed for the recent apr-ized logfile changes. The Win32 service never

  has a STDERR channel in the first place, so the first ap_open_stderr_log()
  fails.  We need to open it after we've initialized the service control
  manager and created the stderr pipe thread that logs stderr output over
  in our Application Event Log.

  The failure message still looks pretty danged stupid on WinXP however;
  "The Apache2 service on Local Computer started and then stopped."
  [ok, that part is straightforward.]  "Some services stop automatically
  if they have no work to do, for example, the Performance Logs and
  Alerts service."  [that's the bogus bit.]

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@93241 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/server/mpm/winnt/mpm_winnt.c b/server/mpm/winnt/mpm_winnt.c
index 00d6c0aa78a200babb3e3c22fe850c56f361e5a3..9ecfcd99b36bae7064493d3b9bf283924e69a734 100644 (file)
--- a/server/mpm/winnt/mpm_winnt.c
+++ b/server/mpm/winnt/mpm_winnt.c
@@ -1895,8 +1895,10 @@ void winnt_rewrite_args(process_rec *process)
          */
         if (osver.dwPlatformId == VER_PLATFORM_WIN32_NT) {
             service_to_start_success = mpm_service_to_start(&service_name);
-            if (service_to_start_success == APR_SUCCESS)
+            if (service_to_start_success == APR_SUCCESS) {
                 service_set = APR_SUCCESS;
+                ap_open_stderr_log(process->pool);
+            }
         }
     }