]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
BUG/MINOR: sink: Use the right field in appctx context in release callback
authorChristopher Faulet <cfaulet@haproxy.com>
Fri, 14 Jan 2022 14:03:22 +0000 (15:03 +0100)
committerChristopher Faulet <cfaulet@haproxy.com>
Fri, 28 Jan 2022 16:56:18 +0000 (17:56 +0100)
In the release callback, ctx.peers was used instead of ctx.sft. Concretly,
it is not an issue because the appctx context is an union and these both
fields are structures with a unique pointer. But it will be a problem if
that changes.

This patch must be backported as far as 2.2.

src/sink.c

index 9e4da9c1cd309617bafce5c7782349bcfeff16e2..c1da18c8d464ba1da19acb225bdfcfdcb1308499 100644 (file)
@@ -603,7 +603,7 @@ void __sink_forward_session_deinit(struct sink_forward_target *sft)
 
 static void sink_forward_session_release(struct appctx *appctx)
 {
-       struct sink_forward_target *sft = appctx->ctx.peers.ptr;
+       struct sink_forward_target *sft = appctx->ctx.sft.ptr;
 
        if (!sft)
                return;