Commit 26a399c1 authored by Jim Jagielski's avatar Jim Jagielski
Browse files

Because SSL's child init is run *after* we change uid/gid. So we need to ensure that file-based 

locks have the correct perms so that the child process
can access them


git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk/modules/ssl@99104 13f79535-47bb-0310-9956-ffa450edef68
parent c1a14427

mod_ssl.h

+1 −0
Changes for mod_ssl.h: 1 added line, 0 removed lines.
Original line number Diff line number Diff line
@@ -406,6 +406,7 @@ typedef struct { 
    ssl_mutexmode_t nMutexMode;

    apr_lockmech_e  nMutexMech;

    const char     *szMutexFile;

    BOOL           ChownMutexFile;

    apr_global_mutex_t   *pMutex;

    apr_array_header_t   *aRandSeed;

    apr_hash_t     *tVHostKeys;

ssl_engine_config.c

+3 −0
Changes for ssl_engine_config.c: 3 added lines, 0 removed lines.
Original line number Diff line number Diff line
@@ -101,6 +101,7 @@ SSLModConfigRec *ssl_config_global_create(server_rec *s) 
    mc->nMutexMode             = SSL_MUTEXMODE_UNSET;

    mc->nMutexMech             = APR_LOCK_DEFAULT;

    mc->szMutexFile            = NULL;

    mc->ChownMutexFile         = FALSE;

    mc->pMutex                 = NULL;

    mc->aRandSeed              = apr_array_make(pool, 4,

                                                sizeof(ssl_randseed_t));
@@ -400,6 +401,7 @@ const char *ssl_cmd_SSLMutex(cmd_parms *cmd, 
     */

    mc->nMutexMode  = SSL_MUTEXMODE_USED;

    mc->szMutexFile = NULL;

    mc->ChownMutexFile = FALSE;



    /* NOTE: previously, 'yes' implied 'sem' */

    if (!strcasecmp(meth, "default") || !strcasecmp(meth, "yes")) {
@@ -413,6 +415,7 @@ const char *ssl_cmd_SSLMutex(cmd_parms *cmd, 
#if APR_HAS_FLOCK_SERIALIZE

    else if ((!strcasecmp(meth, "flock") || !strcasecmp(meth, "file")) && file) {

        mc->nMutexMech = APR_LOCK_FLOCK;

        mc->ChownMutexFile = TRUE;

    }

#endif

#if APR_HAS_POSIXSEM_SERIALIZE

ssl_engine_mutex.c

+2 −0
Changes for ssl_engine_mutex.c: 2 added lines, 0 removed lines.
Original line number Diff line number Diff line
@@ -88,6 +88,8 @@ int ssl_mutex_init(server_rec *s, apr_pool_t *p) 
                         "Cannot create SSLMutex");

        return FALSE;

    }

    if (mc->szMutexFile && mc->ChownMutexFile == TRUE)

        chown(mc->szMutexFile, unixd_config.user_id, -1);



#if APR_USE_SYSVSEM_SERIALIZE

    rv = unixd_set_global_mutex_perms(mc->pMutex);