BUG/MEDIUM: ssl: Fix crash when calling "update ssl ocsp-response" when an update...
authorRemi Tricot-Le Breton <rlebreton@haproxy.com>
Fri, 9 Feb 2024 15:13:43 +0000 (16:13 +0100)
committerChristopher Faulet <cfaulet@haproxy.com>
Wed, 14 Feb 2024 15:39:46 +0000 (16:39 +0100)
commit38c93f27bac14b82ca7565fa7fc0263f6ecef494
treecb71322681fbf0641354b5ac67e6408213d7bec1
parent7bd8817e3c253e1090e0c10cc126a08538ecec7c
BUG/MEDIUM: ssl: Fix crash when calling "update ssl ocsp-response" when an update is ongoing

The CLI command "update ssl ocsp-response" was forcefully removing an
OCSP response from the update tree regardless of whether it used to be
in it beforehand or not. But since the main OCSP upate task works by
removing the entry being currently updated from the update tree and then
reinserting it when the update process is over, it meant that in the CLI
command code we were modifying a structure that was already being used.

These concurrent accesses were not properly locked on the "regular"
update case because it was assumed that once an entry was removed from
the update tree, the update task was the only one able to work on it.

Rather than locking the whole update process, an "updating" flag was
added to the certificate_ocsp in order to prevent the "update ssl
ocsp-response" command from trying to update a response already being
updated.

An easy way to reproduce this crash was to perform two "simultaneous"
calls to "update ssl ocsp-response" on the same certificate. It would
then crash on an eb64_delete call in the main ocsp update task function.

This patch can be backported up to 2.8.

(cherry picked from commit 5e66bf26ecbf6439fafc8ef8857abe22e0874f4d)
Signed-off-by: Christopher Faulet <cfaulet@haproxy.com>
include/haproxy/ssl_ocsp-t.h
src/ssl_ocsp.c