]> granicus.if.org Git - apache/commitdiff
Constify and save a few bytes in the conf pool when parsing "UserDir" directive
authorChristophe Jaillet <jailletc36@apache.org>
Sat, 28 May 2016 05:54:19 +0000 (05:54 +0000)
committerChristophe Jaillet <jailletc36@apache.org>
Sat, 28 May 2016 05:54:19 +0000 (05:54 +0000)
git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1745835 13f79535-47bb-0310-9956-ffa450edef68

modules/mappers/mod_userdir.c

index b181e278d815ae1109190f1669dcd3352956ad61..1ec0e9010d259dca56681e2a97f963b192c87655 100644 (file)
@@ -89,7 +89,7 @@ module AP_MODULE_DECLARE_DATA userdir_module;
 
 typedef struct {
     int globally_disabled;
-    char *userdir;
+    const char *userdir;
     apr_table_t *enabled_users;
     apr_table_t *disabled_users;
 } userdir_config;
@@ -137,7 +137,7 @@ static const char *set_user_dir(cmd_parms *cmd, void *dummy, const char *arg)
                                                  &userdir_module);
     char *username;
     const char *usernames = arg;
-    char *kw = ap_getword_conf(cmd->pool, &usernames);
+    char *kw = ap_getword_conf(cmd->temp_pool, &usernames);
     apr_table_t *usertable;
 
     /* Since we are a raw argument, it is possible for us to be called with
@@ -173,7 +173,7 @@ static const char *set_user_dir(cmd_parms *cmd, void *dummy, const char *arg)
          * If the first (only?) value isn't one of our keywords, just copy
          * the string to the userdir string.
          */
-        s_cfg->userdir = apr_pstrdup(cmd->pool, arg);
+        s_cfg->userdir = arg;
         return NULL;
     }
     /*
@@ -182,7 +182,7 @@ static const char *set_user_dir(cmd_parms *cmd, void *dummy, const char *arg)
      */
     while (*usernames) {
         username = ap_getword_conf(cmd->pool, &usernames);
-        apr_table_setn(usertable, username, kw);
+        apr_table_setn(usertable, username, "1");
     }
     return NULL;
 }