tree 2711f8d986bcdc4472cd61a8b5c6b81ebd07dd25
parent 0cb29005eb59ee416118335f4025fe629d73bd6d
author Willy Tarreau <w@1wt.eu> 1668621394 +0100
committer Christopher Faulet <cfaulet@haproxy.com> 1669373335 +0100

BUG/MEDIUM: ring: fix creation of server in uninitialized ring

If a "ring" section initialization fails (e.g. due to a duplicate name,
invalid chars, or missing memory), any subsequent "server" statement that
appears in the same section will crash the config parser by dereferencing
the currently NULL cfg_sink. E.g:

    ring x
    ring x                 # fails on "already exists"
       server srv 1.1.1.1  # crashes on cfg_sink==NULL

All other statements have a test for this but "server" was missing it,
so this patch adds it.

Thanks to Joel Hutchinson for reporting this issue.

This must be backported as far as 2.2.

(cherry picked from commit 1b662aabbfa32fb6ddeff4ff5f0e3031f12dafd3)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
(cherry picked from commit 857095c1213a24e9a48dc2e2ce05e3b01a3bc049)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
(cherry picked from commit ffc7214a73d8d734945d1abb81fa52f3081f5ed7)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>