From: Marc Becker Date: Sun, 11 Dec 2022 19:09:13 +0000 (+0100) Subject: unify code path for adding PKCS#11 providers X-Git-Tag: v2.6_beta2~6 X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=bcceded96775cc5a131bb9ab11ba855c7576603d;p=thirdparty%2Fopenvpn.git unify code path for adding PKCS#11 providers Use existing wrapper for pkcs11h_addProvider to have arguments with "magic values" for pkcs11-helper call in a central place. Slot event argument to pkcs11h_addProvider has NOT been a boolean for at least 15 years. Luckily the default is PKCS11H_SLOTEVENT_METHOD_AUTO=0=FALSE. Signed-off-by: Marc Becker Acked-by: Frank Lichtenheld Message-Id: <20221211190913.190-1-marc.becker@astos.de> URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg25642.html Signed-off-by: Gert Doering (cherry picked from commit 8958a365479348c1500dee44e1a8b27e7e35a96f) --- diff --git a/src/openvpn/pkcs11.c b/src/openvpn/pkcs11.c index 507af17c0..fbc4c4728 100644 --- a/src/openvpn/pkcs11.c +++ b/src/openvpn/pkcs11.c @@ -853,19 +853,9 @@ show_pkcs11_ids( goto cleanup; } - if ( - (rv = pkcs11h_addProvider( - provider, - provider, - TRUE, - 0, - FALSE, - 0, - cert_private ? TRUE : FALSE - )) != CKR_OK - ) + if (!pkcs11_addProvider(provider, TRUE, 0, cert_private ? TRUE : FALSE)) { - msg(M_FATAL, "PKCS#11: Cannot add provider '%s' %ld-'%s'", provider, rv, pkcs11h_getMessage(rv)); + msg(M_FATAL, "Failed to add PKCS#11 provider '%s", provider); goto cleanup; }