staging: rtl8188eu: while loop replaced by for loop in rtw_restruct_wmm_ie
authorIvan Safonov <insafonov@gmail.com>
Tue, 27 Oct 2015 15:23:57 +0000 (22:23 +0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 29 Oct 2015 00:09:08 +0000 (09:09 +0900)
This patch replaces while loop with for loop, because the initial condition and the increment clearly and briefly defined for this loop.

Signed-off-by: Ivan Safonov <insafonov@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8188eu/core/rtw_mlme.c

index 2b917a1..c1b82f7 100644 (file)
@@ -1709,8 +1709,8 @@ int rtw_restruct_wmm_ie(struct adapter *adapter, u8 *in_ie, u8 *out_ie, uint in_
        unsigned        int ielength = 0;
        unsigned int i, j;
 
-       i = 12; /* after the fixed IE */
-       while (i < in_len) {
+       /* i = 12; after the fixed IE */
+       for (i = 12; i < in_len; i += (in_ie[i + 1] + 2) /* to the next IE element */) {
                ielength = initial_out_len;
 
                if (in_ie[i] == 0xDD && in_ie[i+2] == 0x00 && in_ie[i+3] == 0x50  && in_ie[i+4] == 0xF2 && in_ie[i+5] == 0x02 && i+5 < in_len) {
@@ -1726,7 +1726,6 @@ int rtw_restruct_wmm_ie(struct adapter *adapter, u8 *in_ie, u8 *out_ie, uint in_
                        out_ie[initial_out_len + 8] = 0x00;
                        break;
                }
-               i += (in_ie[i+1]+2); /*  to the next IE element */
        }
        return ielength;
 }