svn commit: r775327 - /httpd/httpd/trunk/server/log.c

[wr...@apache.org]

Author: wrowe
Date: Fri May 15 20:53:33 2009
New Revision: 775327

URL: http://svn.apache.org/viewvc?rev=775327&view=rev
Log:
Force the current directory to the server root in the invocation of the piped
logger processes.

Looking for confirmation from Rainer that this resolves the ambiguity
he observed in Message-ID: <4A...@kippdata.de>.

Note that the shell may decide to ignore the chdir, so test with "|$cmd"
syntax to compare to the expected behavior from 5/14 or earlier. 

Modified:
    httpd/httpd/trunk/server/log.c

Modified: httpd/httpd/trunk/server/log.c
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/server/log.c?rev=775327&r1=775326&r2=775327&view=diff
==============================================================================
--- httpd/httpd/trunk/server/log.c (original)
+++ httpd/httpd/trunk/server/log.c Fri May 15 20:53:33 2009
@@ -309,6 +309,8 @@
     apr_file_t *errfile;
 
     if (((rc = apr_procattr_create(&procattr, p)) == APR_SUCCESS)
+        && ((rc = apr_procattr_dir_set(procattr,
+                                       ap_server_root)) == APR_SUCCESS)
         && ((rc = apr_procattr_cmdtype_set(procattr, cmdtype)) == APR_SUCCESS)
         && ((rc = apr_procattr_io_set(procattr,
                                       APR_FULL_BLOCK,
@@ -912,6 +914,8 @@
     apr_status_t status;
 
     if (((status = apr_procattr_create(&procattr, pl->p)) != APR_SUCCESS) ||
+        ((status = apr_procattr_dir_set(procattr, ap_server_root))
+         != APR_SUCCESS) ||
         ((status = apr_procattr_cmdtype_set(procattr, pl->cmdtype))
          != APR_SUCCESS) ||
         ((status = apr_procattr_child_in_set(procattr,