From: James Prestwood <prestwoj@gmail.com>
To: iwd@lists.linux.dev
Cc: Denis Kenzior <denkenz@gmail.com>, James Prestwood <prestwoj@gmail.com>
Subject: [PATCH 09/15] handshake: Add pmksa setter & stealer
Date: Fri, 22 Nov 2024 07:15:45 -0800 [thread overview]
Message-ID: <20241122151551.286355-10-prestwoj@gmail.com> (raw)
In-Reply-To: <20241122151551.286355-1-prestwoj@gmail.com>
From: Denis Kenzior <denkenz@gmail.com>
The majority of this patch was authored by Denis Kenzior, but
I have appended setting the PMK inside handshake_state_set_pmksa
as well as checking if the pmkid exists in
handshake_state_steal_pmkid.
Authored-by: Denis Kenzior <denkenz@gmail.com>
Authored-by: James Prestwood <prestwoj@gmail.com>
---
Makefile.am | 4 +++
src/handshake.c | 77 +++++++++++++++++++++++++++++++++++++++++++++++++
src/handshake.h | 11 ++++++-
3 files changed, 91 insertions(+), 1 deletion(-)
diff --git a/Makefile.am b/Makefile.am
index 598b8f90..89198289 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -458,6 +458,7 @@ unit_test_eap_sim_SOURCES = unit/test-eap-sim.c \
src/eapol.h src/eapol.c \
src/eapolutil.h src/eapolutil.c \
src/handshake.h src/handshake.c \
+ src/pmksa.h src/pmksa.c \
src/eap.h src/eap.c src/eap-private.h \
src/util.h src/util.c \
src/simauth.h src/simauth.c \
@@ -517,6 +518,7 @@ unit_test_eapol_SOURCES = unit/test-eapol.c \
src/eapol.h src/eapol.c \
src/eapolutil.h src/eapolutil.c \
src/handshake.h src/handshake.c \
+ src/pmksa.h src/pmksa.c \
src/eap.h src/eap.c src/eap-private.h \
src/eap-tls.c src/eap-ttls.c \
src/eap-md5.c src/util.c \
@@ -547,6 +549,7 @@ unit_test_wsc_SOURCES = unit/test-wsc.c src/wscutil.h src/wscutil.c \
src/eapol.h src/eapol.c \
src/eapolutil.h src/eapolutil.c \
src/handshake.h src/handshake.c \
+ src/pmksa.h src/pmksa.c \
src/eap.h src/eap.c src/eap-private.h \
src/util.h src/util.c \
src/erp.h src/erp.c \
@@ -565,6 +568,7 @@ unit_test_sae_SOURCES = unit/test-sae.c \
src/crypto.h src/crypto.c \
src/ie.h src/ie.c \
src/handshake.h src/handshake.c \
+ src/pmksa.h src/pmksa.c \
src/erp.h src/erp.c \
src/band.h src/band.c \
src/util.h src/util.c \
diff --git a/src/handshake.c b/src/handshake.c
index 7fb75dc4..a93143d1 100644
--- a/src/handshake.c
+++ b/src/handshake.c
@@ -43,6 +43,7 @@
#include "src/handshake.h"
#include "src/erp.h"
#include "src/band.h"
+#include "src/pmksa.h"
static inline unsigned int n_ecc_groups(void)
{
@@ -138,6 +139,9 @@ void handshake_state_unref(struct handshake_state *s)
l_free(s->fils_ip_resp_ie);
l_free(s->vendor_ies);
+ if (s->have_pmksa)
+ l_free(s->pmksa);
+
if (s->erp_cache)
erp_cache_put(s->erp_cache);
@@ -701,6 +705,11 @@ void handshake_state_install_ptk(struct handshake_state *s)
{
s->ptk_complete = true;
+ if (!s->have_pmksa && IE_AKM_IS_SAE(s->akm_suite)) {
+ l_debug("Adding PMKSA expiration");
+ s->expiration = l_time_now() + pmksa_lifetime();
+ }
+
if (install_tk) {
uint32_t cipher = ie_rsn_cipher_suite_to_cipher(
s->pairwise_cipher);
@@ -1203,3 +1212,71 @@ done:
return r;
}
+
+bool handshake_state_set_pmksa(struct handshake_state *s,
+ struct pmksa *pmksa)
+{
+ /* checks for both expiration || pmksa being set */
+ if (s->expiration)
+ return false;
+
+ s->pmksa = pmksa;
+ s->have_pmksa = true;
+
+ handshake_state_set_pmkid(s, pmksa->pmkid);
+ handshake_state_set_pmk(s, pmksa->pmk, pmksa->pmk_len);
+
+ return true;
+}
+
+static struct pmksa *handshake_state_steal_pmksa(struct handshake_state *s)
+{
+ struct pmksa *pmksa;
+ uint64_t now = l_time_now();
+
+ if (s->have_pmksa) {
+ pmksa = l_steal_ptr(s->pmksa);
+ s->have_pmksa = false;
+
+ if (l_time_after(now, pmksa->expiration)) {
+ l_free(pmksa);
+ pmksa = NULL;
+ }
+
+ return pmksa;
+ }
+
+ if (s->expiration && l_time_after(now, s->expiration)) {
+ s->expiration = 0;
+ return NULL;
+ }
+
+ if (!s->have_pmkid)
+ return NULL;
+
+ pmksa = l_new(struct pmksa, 1);
+ pmksa->expiration = s->expiration;
+ memcpy(pmksa->spa, s->spa, sizeof(s->spa));
+ memcpy(pmksa->aa, s->aa, sizeof(s->aa));
+ memcpy(pmksa->ssid, s->ssid, s->ssid_len);
+ pmksa->ssid_len = s->ssid_len;
+ pmksa->akm = s->akm_suite;
+ memcpy(pmksa->pmkid, s->pmkid, sizeof(s->pmkid));
+ pmksa->pmk_len = s->pmk_len;
+ memcpy(pmksa->pmk, s->pmk, s->pmk_len);
+
+ return pmksa;
+}
+
+void handshake_state_cache_pmksa(struct handshake_state *s)
+{
+ struct pmksa *pmksa = handshake_state_steal_pmksa(s);
+
+ l_debug("%p", pmksa);
+
+ if (!pmksa)
+ return;
+
+ if (L_WARN_ON(pmksa_cache_put(pmksa) < 0))
+ l_free(pmksa);
+}
diff --git a/src/handshake.h b/src/handshake.h
index 6c0946d4..cf7dc48c 100644
--- a/src/handshake.h
+++ b/src/handshake.h
@@ -29,6 +29,7 @@
struct handshake_state;
enum crypto_cipher;
struct eapol_frame;
+struct pmksa;
enum handshake_kde {
/* 802.11-2020 Table 12-9 in section 12.7.2 */
@@ -141,7 +142,12 @@ struct handshake_state {
bool supplicant_ocvc : 1;
bool ext_key_id_capable : 1;
bool force_default_ecc_group : 1;
- uint8_t ssid[SSID_MAX_SIZE];
+ bool have_pmksa : 1;
+ union {
+ struct pmksa *pmksa;
+ uint64_t expiration;
+ };
+ uint8_t ssid[32];
size_t ssid_len;
char *passphrase;
char *password_identifier;
@@ -302,6 +308,9 @@ void handshake_state_set_chandef(struct handshake_state *s,
int handshake_state_verify_oci(struct handshake_state *s, const uint8_t *oci,
size_t oci_len);
+bool handshake_state_set_pmksa(struct handshake_state *s, struct pmksa *pmksa);
+void handshake_state_cache_pmksa(struct handshake_state *s);
+
bool handshake_util_ap_ie_matches(const struct ie_rsn_info *msg_info,
const uint8_t *scan_ie, bool is_wpa);
--
2.34.1
next prev parent reply other threads:[~2024-11-22 15:16 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-11-22 15:15 [PATCH 00/15] PMKSA support (SAE only) James Prestwood
2024-11-22 15:15 ` [PATCH 01/15] handshake: add ref counting to handshake_state James Prestwood
2024-11-22 15:15 ` [PATCH 02/15] unit: update use of handshake_state with ref/unref James Prestwood
2024-11-22 15:15 ` [PATCH 03/15] auto-t: always initialize StationDebug in Device class James Prestwood
2024-11-22 15:15 ` [PATCH 04/15] auto-t: add pmksa_flush() to hostapd module James Prestwood
2024-11-22 15:15 ` [PATCH 05/15] auto-t: update testSAE to disable PMKSA James Prestwood
2024-11-22 15:15 ` [PATCH 06/15] pmksa: Add skeleton James Prestwood
2024-11-22 15:15 ` [PATCH 07/15] unit: Add basic pmksa test James Prestwood
2024-11-22 15:15 ` [PATCH 08/15] pmksa: Add debugging James Prestwood
2024-11-22 15:15 ` James Prestwood [this message]
2024-11-25 14:56 ` [PATCH 09/15] handshake: Add pmksa setter & stealer Denis Kenzior
2024-11-25 15:01 ` James Prestwood
2024-11-25 19:25 ` Bryce Johnson
2024-11-25 19:49 ` James Prestwood
2024-11-25 20:18 ` Bryce Johnson
2024-11-22 15:15 ` [PATCH 10/15] handshake: add handshake_state_remove_pmksa James Prestwood
2024-11-22 15:15 ` [PATCH 11/15] netdev: add support to use PMKSA over SAE if available James Prestwood
2024-11-22 15:15 ` [PATCH 12/15] station: hold reference to handshake object James Prestwood
2024-11-22 15:15 ` [PATCH 13/15] station: support PMKSA connections James Prestwood
2024-11-22 15:15 ` [PATCH 14/15] auto-t: add PMKSA tests James Prestwood
2024-11-22 15:15 ` [PATCH 15/15] doc: document DisablePMKSA option James Prestwood
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20241122151551.286355-10-prestwoj@gmail.com \
--to=prestwoj@gmail.com \
--cc=denkenz@gmail.com \
--cc=iwd@lists.linux.dev \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox