Merge r1558483 from trunk:

follow-up to r1554161, r1554168, r1554170, r1554175, r1554188, and r1554195:

axe unnecessary initialization/pool allocation

Submitted by: trawick
Reviewed/backported by: jim

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/branches/2.4.x@1560083 13f79535-47bb-0310-9956-ffa450edef68

diff --git a/STATUS b/STATUS
index 522949cae62f60e91bad632a6c82d6b4a77e889d..5b260c5be1f20772afebb75ec8c71b01f6d44e42 100644 (file)
--- a/STATUS
+++ b/STATUS
@@ -98,11 +98,6 @@ RELEASE SHOWSTOPPERS:
 PATCHES ACCEPTED TO BACKPORT FROM TRUNK:
   [ start all new proposals below, under PATCHES PROPOSED. ]
 
-  * mod_auth*z scan-build warnings in new code for 2.4.8
-    trunk patch: https://svn.apache.org/r1558483
-    2.4.x patch: trunk patch works
-    +1: trawick, covener, jim
-
   * ssl_util_ocsp scan-build warning in old code
     (the code is clearly wrong but completely harmless AFAICT)
     trunk patch: https://svn.apache.org/r1534890

diff --git a/modules/aaa/mod_authnz_ldap.c b/modules/aaa/mod_authnz_ldap.c
index 50921cde3dd8f97212528d6854ca52768fadc412..53a6085372a9013fd6680b40cd344b3c1c3acc63 100644 (file)
--- a/modules/aaa/mod_authnz_ldap.c
+++ b/modules/aaa/mod_authnz_ldap.c
@@ -1377,7 +1377,7 @@ static const char *ldap_parse_config(cmd_parms *cmd, const char *require_line,
                                      const void **parsed_require_line)
 {
     const char *expr_err = NULL;
-    ap_expr_info_t *expr = apr_pcalloc(cmd->pool, sizeof(*expr));
+    ap_expr_info_t *expr;
 
     expr = ap_expr_parse_cmd(cmd, require_line, AP_EXPR_FLAG_STRING_RESULT,
             &expr_err, NULL);

diff --git a/modules/aaa/mod_authz_dbd.c b/modules/aaa/mod_authz_dbd.c
index 8da0ccaec340fc501d655886bd367303c9ab5263..30749914a7d7778e4479f79cf4cb2585240144fd 100644 (file)
--- a/modules/aaa/mod_authz_dbd.c
+++ b/modules/aaa/mod_authz_dbd.c
@@ -327,7 +327,7 @@ static const char *dbd_parse_config(cmd_parms *cmd, const char *require_line,
                                      const void **parsed_require_line)
 {
     const char *expr_err = NULL;
-    ap_expr_info_t *expr = apr_pcalloc(cmd->pool, sizeof(*expr));
+    ap_expr_info_t *expr;
 
     expr = ap_expr_parse_cmd(cmd, require_line, AP_EXPR_FLAG_STRING_RESULT,
             &expr_err, NULL);

diff --git a/modules/aaa/mod_authz_dbm.c b/modules/aaa/mod_authz_dbm.c
index cabe65601be7f40690a49c64b5dadb82965f9ea7..c329eacd340549d0122caacdfab1e1c952b0bd21 100644 (file)
--- a/modules/aaa/mod_authz_dbm.c
+++ b/modules/aaa/mod_authz_dbm.c
@@ -280,7 +280,7 @@ static const char *dbm_parse_config(cmd_parms *cmd, const char *require_line,
                                      const void **parsed_require_line)
 {
     const char *expr_err = NULL;
-    ap_expr_info_t *expr = apr_pcalloc(cmd->pool, sizeof(*expr));
+    ap_expr_info_t *expr;
 
     expr = ap_expr_parse_cmd(cmd, require_line, AP_EXPR_FLAG_STRING_RESULT,
             &expr_err, NULL);

diff --git a/modules/aaa/mod_authz_groupfile.c b/modules/aaa/mod_authz_groupfile.c
index 590e8da296b00fc1bae2f2cc004bf9fca99a1984..70e6815305d5eec1ad174a2b719d5443dbd87167 100644 (file)
--- a/modules/aaa/mod_authz_groupfile.c
+++ b/modules/aaa/mod_authz_groupfile.c
@@ -274,7 +274,7 @@ static const char *groupfile_parse_config(cmd_parms *cmd, const char *require_li
                                           const void **parsed_require_line)
 {
     const char *expr_err = NULL;
-    ap_expr_info_t *expr = apr_pcalloc(cmd->pool, sizeof(*expr));
+    ap_expr_info_t *expr;
 
     expr = ap_expr_parse_cmd(cmd, require_line, AP_EXPR_FLAG_STRING_RESULT,
             &expr_err, NULL);

diff --git a/modules/aaa/mod_authz_host.c b/modules/aaa/mod_authz_host.c
index f5f1a0ddbca3194598a985b537b1866cd6983e8d..83fc6e6c71a3faace71b33c0355e8c46b64bd07d 100644 (file)
--- a/modules/aaa/mod_authz_host.c
+++ b/modules/aaa/mod_authz_host.c
@@ -228,7 +228,7 @@ static const char *host_parse_config(cmd_parms *cmd, const char *require_line,
                                      const void **parsed_require_line)
 {
     const char *expr_err = NULL;
-    ap_expr_info_t *expr = apr_pcalloc(cmd->pool, sizeof(*expr));
+    ap_expr_info_t *expr;
 
     expr = ap_expr_parse_cmd(cmd, require_line, AP_EXPR_FLAG_STRING_RESULT,
             &expr_err, NULL);

diff --git a/modules/aaa/mod_authz_user.c b/modules/aaa/mod_authz_user.c
index 0f45395ed81e7f21444cf4bacb9ec1ac353ba193..881f77f1109a2f10c7dfe5b990890aca1bfaed27 100644 (file)
--- a/modules/aaa/mod_authz_user.c
+++ b/modules/aaa/mod_authz_user.c
@@ -97,7 +97,7 @@ static const char *user_parse_config(cmd_parms *cmd, const char *require_line,
                                      const void **parsed_require_line)
 {
     const char *expr_err = NULL;
-    ap_expr_info_t *expr = apr_pcalloc(cmd->pool, sizeof(*expr));
+    ap_expr_info_t *expr;
 
     expr = ap_expr_parse_cmd(cmd, require_line, AP_EXPR_FLAG_STRING_RESULT,
             &expr_err, NULL);