Add log messages to the certificate callback path.
Logs are meant to test 2 theories regarding the
SIGSEGV in the attached bug:
1. Failure is happening somewhere in the middle
of the call path from tls_openssl to AIDL.
2. sm struct becomes stale before the callback is
used.
Bug: 275293332
Test: Connect to WPA-Enterprise network and verify logs.
Change-Id: I7b654d2a63f7f453f05b586f84c42702ac899d1e
diff --git a/wpa_supplicant/aidl/aidl.cpp b/wpa_supplicant/aidl/aidl.cpp
index 5d578dd..b4518bf 100644
--- a/wpa_supplicant/aidl/aidl.cpp
+++ b/wpa_supplicant/aidl/aidl.cpp
@@ -1069,5 +1069,7 @@
if (!aidl_manager)
return -1;
+ wpa_printf(MSG_INFO, "Requesting certificate from framework");
+
return aidl_manager->getCertificate(alias, value);
}
diff --git a/wpa_supplicant/notify.c b/wpa_supplicant/notify.c
index 32ddf1f..3122c58 100644
--- a/wpa_supplicant/notify.c
+++ b/wpa_supplicant/notify.c
@@ -1360,6 +1360,7 @@
ssize_t wpas_get_certificate(const char *alias, uint8_t** value)
{
+ wpa_printf(MSG_INFO, "wpas_get_certificate");
return wpas_aidl_get_certificate(alias, value);
}
diff --git a/wpa_supplicant/wpas_glue.c b/wpa_supplicant/wpas_glue.c
index 7d5ea4c..0921f52 100644
--- a/wpa_supplicant/wpas_glue.c
+++ b/wpa_supplicant/wpas_glue.c
@@ -1187,6 +1187,7 @@
static ssize_t wpa_supplicant_get_certificate_cb(
const char* alias, uint8_t** value)
{
+ wpa_printf(MSG_INFO, "wpa_supplicant_get_certificate_cb");
return wpas_get_certificate(alias, value);
}