|
 |
92bdd1 |
From 7612e65b9bdfe03e5a018e3c897f4a3292c42ee4 Mon Sep 17 00:00:00 2001
|
|
|
92bdd1 |
Message-Id: <7612e65b9bdfe03e5a018e3c897f4a3292c42ee4.1506941240.git.davide.caratti@gmail.com>
|
|
|
92bdd1 |
From: Sabrina Dubroca <sd@queasysnail.net>
|
|
|
92bdd1 |
Date: Tue, 22 Aug 2017 10:34:19 +0200
|
|
|
92bdd1 |
Subject: [PATCH] mka: Add error handling for secy_init_macsec() calls
|
|
|
92bdd1 |
|
|
|
92bdd1 |
secy_init_macsec() can fail (if ->macsec_init fails), and
|
|
|
92bdd1 |
ieee802_1x_kay_init() should handle this and not let MKA run any
|
|
|
92bdd1 |
further, because nothing is going to work anyway.
|
|
|
92bdd1 |
|
|
|
92bdd1 |
On failure, ieee802_1x_kay_init() must deinit its kay, which will free
|
|
|
92bdd1 |
kay->ctx, so ieee802_1x_kay_init callers (only ieee802_1x_alloc_kay_sm)
|
|
|
92bdd1 |
must not do it. Before this patch there is a double-free of the ctx
|
|
|
92bdd1 |
argument when ieee802_1x_kay_deinit() was called.
|
|
|
92bdd1 |
|
|
|
92bdd1 |
Signed-off-by: Sabrina Dubroca <sd@queasysnail.net>
|
|
|
92bdd1 |
---
|
|
|
92bdd1 |
src/pae/ieee802_1x_kay.c | 25 ++++++++++++++-----------
|
|
|
92bdd1 |
wpa_supplicant/wpas_kay.c | 5 ++---
|
|
|
92bdd1 |
2 files changed, 16 insertions(+), 14 deletions(-)
|
|
|
92bdd1 |
|
|
|
92bdd1 |
diff --git a/src/pae/ieee802_1x_kay.c b/src/pae/ieee802_1x_kay.c
|
|
|
92bdd1 |
index ff55f88b8..c4bfcbc63 100644
|
|
|
92bdd1 |
--- a/src/pae/ieee802_1x_kay.c
|
|
|
92bdd1 |
+++ b/src/pae/ieee802_1x_kay.c
|
|
|
92bdd1 |
@@ -3100,6 +3100,7 @@ ieee802_1x_kay_init(struct ieee802_1x_kay_ctx *ctx, enum macsec_policy policy,
|
|
|
92bdd1 |
kay = os_zalloc(sizeof(*kay));
|
|
|
92bdd1 |
if (!kay) {
|
|
|
92bdd1 |
wpa_printf(MSG_ERROR, "KaY-%s: out of memory", __func__);
|
|
|
92bdd1 |
+ os_free(ctx);
|
|
|
92bdd1 |
return NULL;
|
|
|
92bdd1 |
}
|
|
|
92bdd1 |
|
|
|
92bdd1 |
@@ -3134,10 +3135,8 @@ ieee802_1x_kay_init(struct ieee802_1x_kay_ctx *ctx, enum macsec_policy policy,
|
|
|
92bdd1 |
dl_list_init(&kay->participant_list);
|
|
|
92bdd1 |
|
|
|
92bdd1 |
if (policy != DO_NOT_SECURE &&
|
|
|
92bdd1 |
- secy_get_capability(kay, &kay->macsec_capable) < 0) {
|
|
|
92bdd1 |
- os_free(kay);
|
|
|
92bdd1 |
- return NULL;
|
|
|
92bdd1 |
- }
|
|
|
92bdd1 |
+ secy_get_capability(kay, &kay->macsec_capable) < 0)
|
|
|
92bdd1 |
+ goto error;
|
|
|
92bdd1 |
|
|
|
92bdd1 |
if (policy == DO_NOT_SECURE ||
|
|
|
92bdd1 |
kay->macsec_capable == MACSEC_CAP_NOT_IMPLEMENTED) {
|
|
|
92bdd1 |
@@ -3164,16 +3163,17 @@ ieee802_1x_kay_init(struct ieee802_1x_kay_ctx *ctx, enum macsec_policy policy,
|
|
|
92bdd1 |
wpa_printf(MSG_DEBUG, "KaY: state machine created");
|
|
|
92bdd1 |
|
|
|
92bdd1 |
/* Initialize the SecY must be prio to CP, as CP will control SecY */
|
|
|
92bdd1 |
- secy_init_macsec(kay);
|
|
|
92bdd1 |
+ if (secy_init_macsec(kay) < 0) {
|
|
|
92bdd1 |
+ wpa_printf(MSG_DEBUG, "KaY: Could not initialize MACsec");
|
|
|
92bdd1 |
+ goto error;
|
|
|
92bdd1 |
+ }
|
|
|
92bdd1 |
|
|
|
92bdd1 |
wpa_printf(MSG_DEBUG, "KaY: secy init macsec done");
|
|
|
92bdd1 |
|
|
|
92bdd1 |
/* init CP */
|
|
|
92bdd1 |
kay->cp = ieee802_1x_cp_sm_init(kay);
|
|
|
92bdd1 |
- if (kay->cp == NULL) {
|
|
|
92bdd1 |
- ieee802_1x_kay_deinit(kay);
|
|
|
92bdd1 |
- return NULL;
|
|
|
92bdd1 |
- }
|
|
|
92bdd1 |
+ if (kay->cp == NULL)
|
|
|
92bdd1 |
+ goto error;
|
|
|
92bdd1 |
|
|
|
92bdd1 |
if (policy == DO_NOT_SECURE) {
|
|
|
92bdd1 |
ieee802_1x_cp_connect_authenticated(kay->cp);
|
|
|
92bdd1 |
@@ -3184,12 +3184,15 @@ ieee802_1x_kay_init(struct ieee802_1x_kay_ctx *ctx, enum macsec_policy policy,
|
|
|
92bdd1 |
if (kay->l2_mka == NULL) {
|
|
|
92bdd1 |
wpa_printf(MSG_WARNING,
|
|
|
92bdd1 |
"KaY: Failed to initialize L2 packet processing for MKA packet");
|
|
|
92bdd1 |
- ieee802_1x_kay_deinit(kay);
|
|
|
92bdd1 |
- return NULL;
|
|
|
92bdd1 |
+ goto error;
|
|
|
92bdd1 |
}
|
|
|
92bdd1 |
}
|
|
|
92bdd1 |
|
|
|
92bdd1 |
return kay;
|
|
|
92bdd1 |
+
|
|
|
92bdd1 |
+error:
|
|
|
92bdd1 |
+ ieee802_1x_kay_deinit(kay);
|
|
|
92bdd1 |
+ return NULL;
|
|
|
92bdd1 |
}
|
|
|
92bdd1 |
|
|
|
92bdd1 |
|
|
|
92bdd1 |
diff --git a/wpa_supplicant/wpas_kay.c b/wpa_supplicant/wpas_kay.c
|
|
|
92bdd1 |
index d087e00ad..587e5c3dd 100644
|
|
|
92bdd1 |
--- a/wpa_supplicant/wpas_kay.c
|
|
|
92bdd1 |
+++ b/wpa_supplicant/wpas_kay.c
|
|
|
92bdd1 |
@@ -235,10 +235,9 @@ int ieee802_1x_alloc_kay_sm(struct wpa_supplicant *wpa_s, struct wpa_ssid *ssid)
|
|
|
92bdd1 |
res = ieee802_1x_kay_init(kay_ctx, policy, ssid->macsec_port,
|
|
|
92bdd1 |
ssid->mka_priority, wpa_s->ifname,
|
|
|
92bdd1 |
wpa_s->own_addr);
|
|
|
92bdd1 |
- if (res == NULL) {
|
|
|
92bdd1 |
- os_free(kay_ctx);
|
|
|
92bdd1 |
+ /* ieee802_1x_kay_init() frees kay_ctx on failure */
|
|
|
92bdd1 |
+ if (res == NULL)
|
|
|
92bdd1 |
return -1;
|
|
|
92bdd1 |
- }
|
|
|
92bdd1 |
|
|
|
92bdd1 |
wpa_s->kay = res;
|
|
|
92bdd1 |
|
|
|
92bdd1 |
--
|
|
|
92bdd1 |
2.13.5
|
|
|
92bdd1 |
|