]> git.ipfire.org Git - thirdparty/hostap.git/commitdiff
WNM: Clean up le16 variable use to avoid sparse warnings
authorJouni Malinen <j@w1.fi>
Tue, 31 Dec 2013 17:41:47 +0000 (19:41 +0200)
committerJouni Malinen <j@w1.fi>
Tue, 31 Dec 2013 17:41:47 +0000 (19:41 +0200)
intval is marked le16 and should be used through proper byte order
conversion functions even if it ended up getting set correctly due to
the two operations cancelling each other.

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

src/ap/wnm_ap.c

index 825520464eac6c096a24f46cee011b275e60296f..8e5bdcb068ac614928de056ae8bc07b4156d42e1 100644 (file)
@@ -73,7 +73,7 @@ static int ieee802_11_send_wnmsleep_resp(struct hostapd_data *hapd,
        wnmsleep_ie.len = wnmsleep_ie_len - 2;
        wnmsleep_ie.action_type = action_type;
        wnmsleep_ie.status = WNM_STATUS_SLEEP_ACCEPT;
-       wnmsleep_ie.intval = intval;
+       wnmsleep_ie.intval = host_to_le16(intval);
 
        /* TFS IE(s) */
        wnmtfs_ie = os_zalloc(MAX_TFS_IE_LEN);
@@ -236,7 +236,7 @@ static void ieee802_11_rx_wnmsleep_req(struct hostapd_data *hapd,
 
        ieee802_11_send_wnmsleep_resp(hapd, addr, dialog_token,
                                      wnmsleep_ie->action_type,
-                                     wnmsleep_ie->intval);
+                                     le_to_host16(wnmsleep_ie->intval));
 
        if (wnmsleep_ie->action_type == WNM_SLEEP_MODE_EXIT) {
                /* clear the tfs after sending the resp frame */