Parcourir la source

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>
Jouni Malinen il y a 10 ans
Parent
commit
56b352ec58
2 fichiers modifiés avec 2 ajouts et 2 suppressions
  1. 1 1
      src/eap_peer/eap.c
  2. 1 1
      src/eap_server/eap_server.c

+ 1 - 1
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;

+ 1 - 1
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;