From 9cc88c3075ca13c90268eabc83ae121f47848f40 Mon Sep 17 00:00:00 2001 From: Willy Tarreau Date: Fri, 8 Apr 2022 14:34:31 +0200 Subject: [PATCH] BUG/MINOR: quic: set the source not the destination address on accept() When a QUIC connection is accepted, the wrong field is set from the client's source address, it's the destination instead of the source. No backport needed. --- src/quic_sock.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/quic_sock.c b/src/quic_sock.c index 9128fd7fc..f3faa64f5 100644 --- a/src/quic_sock.c +++ b/src/quic_sock.c @@ -98,10 +98,10 @@ static int new_quic_cli_conn(struct quic_conn *qc, struct listener *l, if (unlikely((cli_conn = conn_new(&l->obj_type)) == NULL)) goto out; - if (!sockaddr_alloc(&cli_conn->dst, saddr, sizeof *saddr)) + if (!sockaddr_alloc(&cli_conn->src, saddr, sizeof *saddr)) goto out_free_conn; - cli_conn->flags |= CO_FL_ADDR_TO_SET; + cli_conn->flags |= CO_FL_ADDR_FROM_SET; qc->conn = cli_conn; cli_conn->qc = qc;