BUG/MINOR: ssl/cli: duplicate cleaning code in cli_parse_del_crtlist
authorWilliam Lallemand <wlallemand@haproxy.com>
Tue, 27 Feb 2024 16:22:15 +0000 (17:22 +0100)
committerWilliam Lallemand <wlallemand@haproxy.com>
Tue, 27 Feb 2024 17:10:43 +0000 (18:10 +0100)
commitbb7af8b2f169647286cfefe516c07d6d3a1fe156
tree9a4317cbe58ae05b136096dd945e447bffe0b5e1
parent8a31783b647f3bef171a7faf074da2b1563f8a80
BUG/MINOR: ssl/cli: duplicate cleaning code in cli_parse_del_crtlist

Since 23cab33 ("BUG/MINOR: ssl: Clear the ckch instance when deleting a
crt-list line"), LIST_DELETE is done twice, one time in
cli_parse_del_crtlist() and another time in ckch_inst_free().

It could trigger a crash with -DDEBUG_LIST.

This isn't a major problem since the ptr is not freed in the meantime so
it will only trigger with the debug.

This patch removes the LIST_DELETE as well as the loop done on link_ref
which is also don in ckch_inst_free()

Could be backported as far as 2.4. 2.4 version does not have a link_ref
loop.
src/ssl_crtlist.c