Re: [PATCH 09/15] handshake: Add pmksa setter & stealer

Denis Kenzior
2024-11-25T14:56:37Z

Hi James,

On 11/22/24 9:15 AM, James Prestwood wrote:
> 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(-)
>

<snip>

> @@ -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];

I changed this back to using SSID_MAX_SIZE instead of the magic number.

> size_t ssid_len;
> char *passphrase;
> char *password_identifier;

All 15 patches applied, thanks.

Regards,
-Denis