prefork MPM: Use the right permissions for the directory created

for gprof support.

Submitted by:         Jim Carlson <jcarlson@jnous.com>
Reviewed by:          Jeff Trawick

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

diff --git a/CHANGES b/CHANGES
index 99f279c60ac59891499a5ae28381609d9b307af9..38b9b2f4e13b56ec9c43dac70b565a01fde3a32c 100644 (file)
--- a/CHANGES
+++ b/CHANGES
@@ -2,6 +2,9 @@ Changes with Apache 2.1.0-dev
 
   [Remove entries to the current 2.0 section below, when backported]
 
+  *) prefork MPM: Use the right permissions for the directory created 
+     for gprof support.  [Jim Carlson <jcarlson@jnous.com>]
+
   *) By default, use the same CC and CPP with which APR was built.
      The user can override with CC and CPP environment variables.
      [Jeff Trawick]

diff --git a/server/mpm/prefork/prefork.c b/server/mpm/prefork/prefork.c
index a1612712bf1e386722308998164aef10b9927e40..699366532d1378d772dbfe88968a9eb42c6e17d9 100644 (file)
--- a/server/mpm/prefork/prefork.c
+++ b/server/mpm/prefork/prefork.c
@@ -205,7 +205,10 @@ static void chdir_for_gprof(void)
            apr_snprintf(buf, sizeof(buf), "%sgprof.%d", dir, (int)getpid());
        } 
        use_dir = ap_server_root_relative(pconf, buf[0] ? buf : dir);
-       res = apr_dir_make(use_dir, 0755, pconf);
+       res = apr_dir_make(use_dir,
+                           APR_UREAD | APR_UWRITE | APR_UEXECUTE |
+                           APR_GREAD | APR_GEXECUTE |
+                           APR_WREAD | APR_WEXECUTE, pconf);
        if(res != APR_SUCCESS && !APR_STATUS_IS_EEXIST(res)) {
            ap_log_error(APLOG_MARK, APLOG_ERR, res, ap_server_conf,
                         "gprof: error creating directory %s", dir);