[PATCH] WPA: pairwise should be printed as a bool

Johannes Berg johannes
Thu May 2 02:29:47 PDT 2013


From: Johannes Berg <johannes.berg at intel.com>

Since "pairwise" is defined as an integer, the current
assignment leads to it having the value 0 or 8, which
is a bit strange in debug output:

WPA: Send EAPOL(version=2 secure=1 mic=1 ack=1 install=1 pairwise=8 kde_len=46 keyidx=2 encr=1)

Use !!(...) to normalize it to 0 or 1.

Change-Id: I72735a4fd68c9bc768fe833ab8b175715e77d8c3
Signed-hostap: Johannes Berg <johannes.berg at intel.com>
---
 src/ap/wpa_auth.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/ap/wpa_auth.c b/src/ap/wpa_auth.c
index 4f1f6fb..4ecae2d 100644
--- a/src/ap/wpa_auth.c
+++ b/src/ap/wpa_auth.c
@@ -1232,7 +1232,7 @@ void __wpa_send_eapol(struct wpa_authenticator *wpa_auth,
 	else
 		version = WPA_KEY_INFO_TYPE_HMAC_MD5_RC4;
 
-	pairwise = key_info & WPA_KEY_INFO_KEY_TYPE;
+	pairwise = !!(key_info & WPA_KEY_INFO_KEY_TYPE);
 
 	wpa_printf(MSG_DEBUG, "WPA: Send EAPOL(version=%d secure=%d mic=%d "
 		   "ack=%d install=%d pairwise=%d kde_len=%lu keyidx=%d "
-- 
1.8.0




More information about the Hostap mailing list