diff options
| author | Christophe Jaillet <jailletc36@apache.org> | 2022-02-21 22:07:35 +0100 |
|---|---|---|
| committer | Christophe Jaillet <jailletc36@apache.org> | 2022-02-21 22:07:35 +0100 |
| commit | bf6c900db7ec99d7b0b4bd6e007acb541ca848d4 (patch) | |
| tree | f32ee6ef13f63b5b926a04fd2183834a8ea806a2 /server/util_expr_eval.c | |
| parent | Remove some APR 0.x stuff. (diff) | |
| download | apache2-bf6c900db7ec99d7b0b4bd6e007acb541ca848d4.tar.xz apache2-bf6c900db7ec99d7b0b4bd6e007acb541ca848d4.zip | |
There is no point in calling ap_varbuf_grow() here, it is already
called from within ap_varbuf_strmemcat().
Moreover, 2nd parameter should be the minimum total new length, not
the amount of the growth. So this call is likely to be a no-op.
git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1898286 13f79535-47bb-0310-9956-ffa450edef68
Diffstat (limited to '')
| -rw-r--r-- | server/util_expr_eval.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/server/util_expr_eval.c b/server/util_expr_eval.c index 823978d2f3..9c71e865b0 100644 --- a/server/util_expr_eval.c +++ b/server/util_expr_eval.c @@ -14,7 +14,7 @@ * limitations under the License. */ -/* _ _ +/* * ap_expr_eval.c, based on ssl_expr_eval.c from mod_ssl */ @@ -124,7 +124,6 @@ static const char *ap_expr_list_pstrcat(apr_pool_t *p, for (i = 0; i < n; ++i) { val = APR_ARRAY_IDX(list, i, const char*); vlen = strlen(val); - ap_varbuf_grow(&vb, vlen + slen + 1); ap_varbuf_strmemcat(&vb, val, vlen); ap_varbuf_strmemcat(&vb, sep, slen); } |