ERP: Avoid a static analyzer warning on uninitialized emsk_len

This was not really a real issue since bin_clear_free() would not use
the emsk_len argument when emsk is NULL as it would be on the path where
emsk_len has not been initilized. Anyway, it is better to get rid of the
warning.

Signed-off-by: Jouni Malinen <j@w1.fi>

diff --git a/src/eap_peer/eap.c b/src/eap_peer/eap.c
index 52e9e312dcf1405f730661f28e963e056f6a3700..6e356c0df751044f6fb03b3ca77db1fd85744299 100644 (file)
--- a/src/eap_peer/eap.c
+++ b/src/eap_peer/eap.c
@@ -474,7 +474,7 @@ static void eap_peer_erp_init(struct eap_sm *sm)
 {
 #ifdef CONFIG_ERP
        u8 *emsk = NULL;
-       size_t emsk_len;
+       size_t emsk_len = 0;
        u8 EMSKname[EAP_EMSK_NAME_LEN];
        u8 len[2];
        char *realm;

diff --git a/src/eap_server/eap_server.c b/src/eap_server/eap_server.c
index 53e32c3817b1c19f2c47c3e237051992c35e0a58..bd919e570c825549d7732b240b5340bd4cb5e879 100644 (file)
--- a/src/eap_server/eap_server.c
+++ b/src/eap_server/eap_server.c
@@ -412,7 +412,7 @@ static void eap_server_erp_init(struct eap_sm *sm)
 {
 #ifdef CONFIG_ERP
        u8 *emsk = NULL;
-       size_t emsk_len;
+       size_t emsk_len = 0;
        u8 EMSKname[EAP_EMSK_NAME_LEN];
        u8 len[2];
        const char *domain;