[PATCH 03/11] staging: rtl8188eu: traversal order of the 'remain' variable values inverted
Dan Carpenter
dan.carpenter at oracle.com
Wed Nov 4 11:24:27 UTC 2015
On Wed, Nov 04, 2015 at 10:12:51AM +0700, Ivan Safonov wrote:
> The number of the loop passes has not changed.
>
> Signed-off-by: Ivan Safonov <insafonov at gmail.com>
> ---
> drivers/staging/rtl8188eu/hal/fw.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/rtl8188eu/hal/fw.c b/drivers/staging/rtl8188eu/hal/fw.c
> index 6d6dd15..9cd898c 100644
> --- a/drivers/staging/rtl8188eu/hal/fw.c
> +++ b/drivers/staging/rtl8188eu/hal/fw.c
> @@ -89,7 +89,7 @@ static void _rtl88e_fill_dummy(u8 *pfwbuf, u32 *pfwlen)
> remain = (remain == 0) ? 0 : (4 - remain);
> n = remain;
>
> - for (; remain > 0; fwlen++, remain--) {
> + for (remain = 0; remain < n; fwlen++, remain++) {
We are using the "remain" variable for a couple different things. It
should only hold the remaining number of array elements. It's shouldn't
be the iterator as well.
regards,
dan carpenter
More information about the devel
mailing list