From c8bb1539cb5f644295141c5219942ff02c279e8a Mon Sep 17 00:00:00 2001 From: William Dauchy Date: Sun, 24 Nov 2019 15:04:20 +0100 Subject: [PATCH] CLEANUP: ssl: check if a transaction exists once before setting it MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit trivial patch to fix issue #351 Fixes: bc6ca7ccaa72 ("MINOR: ssl/cli: rework 'set ssl cert' as 'set/commit'") Reported-by: Илья Шипицин Signed-off-by: William Dauchy --- src/ssl_sock.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/ssl_sock.c b/src/ssl_sock.c index cd04c344a..62b9de95c 100644 --- a/src/ssl_sock.c +++ b/src/ssl_sock.c @@ -10468,7 +10468,7 @@ static int cli_parse_set_cert(char **args, char *payload, struct appctx *appctx, /* we succeed, we can save the ckchs in the transaction */ /* if there wasn't a transaction, update the old ckchs */ - if (!ckchs_transaction.old_ckchs && !ckchs_transaction.old_ckchs) { + if (!ckchs_transaction.old_ckchs) { ckchs_transaction.old_ckchs = appctx->ctx.ssl.old_ckchs; ckchs_transaction.path = appctx->ctx.ssl.path; err = memprintf(&err, "Transaction created for certificate %s!\n", ckchs_transaction.path);