From fb2b7858a728eb1ad32b49e9e129f2fc1d3ea880 Mon Sep 17 00:00:00 2001 From: Jouni Malinen Date: Tue, 18 Jul 2023 16:11:08 +0300 Subject: [PATCH] FILS: Fix HE MCS field initialization The second argument to memset() is only eight bits, so there is no point in trying to set 0xffff values for an array of 16-bit fields. 0xff will do the exact same thing without causing static analyzes warnings about truncated value. Fixes: 903e3a1e6259 ("FILS: Fix maximum NSS calculation for FD frame") Signed-off-by: Jouni Malinen --- src/ap/beacon.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/ap/beacon.c b/src/ap/beacon.c index 32c143a2b..772825d88 100644 --- a/src/ap/beacon.c +++ b/src/ap/beacon.c @@ -1450,7 +1450,7 @@ static u16 hostapd_fils_discovery_cap(struct hostapd_data *hapd) int i; u16 nss = 0, mcs[6]; - os_memset(mcs, 0xffff, 6 * sizeof(u16)); + os_memset(mcs, 0xff, 6 * sizeof(u16)); if (mcs_nss_size == 4) { mcs[0] = WPA_GET_LE16(&he_mcs[0]);