[PATCH] eap-tls: Use the right variable in settings check

[PATCH] eap-tls: Use the right variable in settings check

[Andrew Zaborowski]

[-- Attachment #1: Type: text/plain, Size: 1072 bytes --]

---
 src/eap-tls.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/eap-tls.c b/src/eap-tls.c
index ac7650dc..b20a1b39 100644
--- a/src/eap-tls.c
+++ b/src/eap-tls.c
@@ -174,7 +174,7 @@ static int eap_tls_settings_check(struct l_settings *settings,
 	 * Check whether the combination of settings that are present/missing
 	 * makes sense before validating each setting.
 	 */
-	if (bundle_value && (priv_key_value || client_cert)) {
+	if (bundle_value && (priv_key_value || client_cert_value)) {
 		l_error("Either %s or %s/%s can be used, not both",
 			bundle_setting, priv_key_setting, client_cert_setting);
 		ret = -EEXIST;
@@ -184,7 +184,7 @@ static int eap_tls_settings_check(struct l_settings *settings,
 			priv_key_setting, client_cert_setting);
 		ret = -ENOENT;
 		goto done;
-	} else if (!priv_key_value && client_cert) {
+	} else if (!priv_key_value && client_cert_value) {
 		l_error("%s present but no client private key (%s)",
 			client_cert_setting, priv_key_setting);
 		ret = -ENOENT;
-- 
2.27.0

Re: [PATCH] eap-tls: Use the right variable in settings check

[Denis Kenzior]

[-- Attachment #1: Type: text/plain, Size: 188 bytes --]

Hi Andrew,

On 1/27/21 8:02 AM, Andrew Zaborowski wrote:
> ---
>   src/eap-tls.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)

Applied, thanks.

Regards,
-Denis