auth: auth-request - Fix success response data without policy lookup

In that case the success data was discarded and not passed to the client.

diff --git a/src/auth/auth-request.c b/src/auth/auth-request.c
index a7db045f8afd90240530a7acf9f5335aaaadbf57..3c563afa03c58e4dfdcd8f5d9231968d5b572c11 100644 (file)
--- a/src/auth/auth-request.c
+++ b/src/auth/auth-request.c
@@ -185,23 +185,20 @@ void auth_request_success(struct auth_request *request,
        if (request->fields.userdb_reply != NULL)
                auth_fields_snapshot(request->fields.userdb_reply);
 
+       struct auth_policy_check_ctx *ctx;
+
+       ctx = p_new(request->pool, struct auth_policy_check_ctx, 1);
+       ctx->request = request;
+       ctx->success_data = buffer_create_dynamic(request->pool, data_size);
+       buffer_append(ctx->success_data, data, data_size);
+       ctx->type = AUTH_POLICY_CHECK_TYPE_SUCCESS;
+
        if (!request->set->policy_check_after_auth) {
-               struct auth_policy_check_ctx *ctx =
-                       p_new(request->pool, struct auth_policy_check_ctx, 1);
-               ctx->success_data = buffer_create_dynamic(request->pool, 1);
-               ctx->request = request;
-               ctx->type = AUTH_POLICY_CHECK_TYPE_SUCCESS;
                auth_request_policy_check_callback(0, ctx);
                return;
        }
 
        /* perform second policy lookup here */
-       struct auth_policy_check_ctx *ctx =
-               p_new(request->pool, struct auth_policy_check_ctx, 1);
-       ctx->request = request;
-       ctx->success_data = buffer_create_dynamic(request->pool, data_size);
-       buffer_append(ctx->success_data, data, data_size);
-       ctx->type = AUTH_POLICY_CHECK_TYPE_SUCCESS;
        auth_policy_check(request, request->mech_password,
                          auth_request_policy_check_callback, ctx);
 }