]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
CLEANUP: spoe: use pool_zalloc() instead of pool_alloc+memset
authorWilly Tarreau <w@1wt.eu>
Mon, 22 Mar 2021 20:04:50 +0000 (21:04 +0100)
committerWilly Tarreau <w@1wt.eu>
Mon, 22 Mar 2021 22:18:26 +0000 (23:18 +0100)
Two places used to alloc then zero the area, let's have the allocator do it.

src/flt_spoe.c

index 2d7b3b82805022401da282b8dbba4f61a6d2a7f8..5688ca99182c9917c40df7355d4af95bfbf3d2ba 100644 (file)
@@ -1965,10 +1965,9 @@ spoe_create_appctx(struct spoe_config *conf)
        if ((appctx = appctx_new(&spoe_applet, tid_bit)) == NULL)
                goto out_error;
 
-       appctx->ctx.spoe.ptr = pool_alloc(pool_head_spoe_appctx);
+       appctx->ctx.spoe.ptr = pool_zalloc(pool_head_spoe_appctx);
        if (SPOE_APPCTX(appctx) == NULL)
                goto out_free_appctx;
-       memset(appctx->ctx.spoe.ptr, 0, pool_head_spoe_appctx->size);
 
        appctx->st0 = SPOE_APPCTX_ST_CONNECT;
        if ((SPOE_APPCTX(appctx)->task = task_new(tid_bit)) == NULL)
@@ -2870,11 +2869,10 @@ spoe_create_context(struct stream *s, struct filter *filter)
        struct spoe_config  *conf = FLT_CONF(filter);
        struct spoe_context *ctx;
 
-       ctx = pool_alloc(pool_head_spoe_ctx);
+       ctx = pool_zalloc(pool_head_spoe_ctx);
        if (ctx == NULL) {
                return NULL;
        }
-       memset(ctx, 0, sizeof(*ctx));
        ctx->filter      = filter;
        ctx->state       = SPOE_CTX_ST_NONE;
        ctx->status_code = SPOE_CTX_ERR_NONE;