[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <1491751717.10783.21.camel@hadess.net>
Date: Sun, 09 Apr 2017 17:28:37 +0200
From: Bastien Nocera <hadess@...ess.net>
To: Larry Finger <Larry.Finger@...inger.net>,
gregkh@...uxfoundation.org
Cc: netdev@...r.kernel.org, devel@...verdev.osuosl.org,
hdegoede@...hat.com, jes.sorensen@...il.com
Subject: Re: [PATCH 20/22] staging rtl8723bs: Fix indenting errors and an
off-by-one mistake in core/rtw_mlme_ext.c
On Sat, 2017-04-08 at 11:07 -0500, Larry Finger wrote:
> } else {
> - for (pstat->aid = 1; pstat->aid <= NUM_STA; pstat->aid++)
> + for (pstat->aid = 1; pstat->aid < NUM_STA; pstat->aid++)
> if (pstapriv->sta_aid[pstat->aid - 1] == NULL)
> break;
why not start at 0 and increment pstat->aid afterwards? Meh.
Powered by blists - more mailing lists