r11811: Revert 11808, this broke the selftest. No idea why though. Why do we to the

resolve_name if it's not used?

I know this is my code, but I don't understand why it breaks tests.

Volker

diff --git a/source/lib/socket/connect.c b/source/lib/socket/connect.c
index f174b7c09d174c3b9492f873084ddc9f9a911138..244fdcbb0635169cc5032775179b8d666ce81efc 100644 (file)
--- a/source/lib/socket/connect.c
+++ b/source/lib/socket/connect.c
@@ -147,9 +147,9 @@ static void socket_connect_recv_addr(struct composite_context *ctx)
 {
        struct connect_state *state =
                talloc_get_type(ctx->async.private_data, struct connect_state);
+       const char *addr;
 
-       state->ctx->status = resolve_name_recv(ctx, state,
-                                              &state->server_address);
+       state->ctx->status = resolve_name_recv(ctx, state, &addr);
        if (!composite_is_ok(state->ctx)) return;
 
        ctx = talloc_zero(state, struct composite_context);