Message ID | 1398098324-5193-1-git-send-email-colin.king@canonical.com (mailing list archive) |
---|---|
State | Not Applicable, archived |
Headers | show |
On 04/21/2014 11:38 AM, Colin King wrote: > From: Colin Ian King <colin.king@canonical.com> > > static code analysis from cppcheck reports: > > [drivers/net/wireless/rtlwifi/rtl8188ee/trx.c:322]: > (error) Uninitialized variable: packet_beacon > > packet_beacon is not initialized and hence packet_beacon > contains garbage from the stack, so set it to false. > > Signed-off-by: Colin Ian King <colin.king@canonical.com> > --- > drivers/net/wireless/rtlwifi/rtl8188ee/trx.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) This change should have a "Cc: Stable <stable@vger.kernel.org> [3.10+] notation. Other than that, Acked-by: Larry Finger <Larry.Finger@lwfinger.net> John: Is it possible for you to add stable, or do you want Colin to resubmit. Larry > > diff --git a/drivers/net/wireless/rtlwifi/rtl8188ee/trx.c b/drivers/net/wireless/rtlwifi/rtl8188ee/trx.c > index 06ef47c..5b4c225 100644 > --- a/drivers/net/wireless/rtlwifi/rtl8188ee/trx.c > +++ b/drivers/net/wireless/rtlwifi/rtl8188ee/trx.c > @@ -293,7 +293,7 @@ static void _rtl88ee_translate_rx_signal_stuff(struct ieee80211_hw *hw, > u8 *psaddr; > __le16 fc; > u16 type, ufc; > - bool match_bssid, packet_toself, packet_beacon, addr; > + bool match_bssid, packet_toself, packet_beacon = false, addr; > > tmp_buf = skb->data + pstatus->rx_drvinfo_size + pstatus->rx_bufshift; > > -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Mon, Apr 21, 2014 at 12:15:44PM -0500, Larry Finger wrote: > On 04/21/2014 11:38 AM, Colin King wrote: > >From: Colin Ian King <colin.king@canonical.com> > > > >static code analysis from cppcheck reports: > > > >[drivers/net/wireless/rtlwifi/rtl8188ee/trx.c:322]: > > (error) Uninitialized variable: packet_beacon > > > >packet_beacon is not initialized and hence packet_beacon > >contains garbage from the stack, so set it to false. > > > >Signed-off-by: Colin Ian King <colin.king@canonical.com> > >--- > > drivers/net/wireless/rtlwifi/rtl8188ee/trx.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > This change should have a "Cc: Stable <stable@vger.kernel.org> [3.10+] notation. > > Other than that, Acked-by: Larry Finger <Larry.Finger@lwfinger.net> > > John: Is it possible for you to add stable, or do you want Colin to resubmit. Got it, thanks...
diff --git a/drivers/net/wireless/rtlwifi/rtl8188ee/trx.c b/drivers/net/wireless/rtlwifi/rtl8188ee/trx.c index 06ef47c..5b4c225 100644 --- a/drivers/net/wireless/rtlwifi/rtl8188ee/trx.c +++ b/drivers/net/wireless/rtlwifi/rtl8188ee/trx.c @@ -293,7 +293,7 @@ static void _rtl88ee_translate_rx_signal_stuff(struct ieee80211_hw *hw, u8 *psaddr; __le16 fc; u16 type, ufc; - bool match_bssid, packet_toself, packet_beacon, addr; + bool match_bssid, packet_toself, packet_beacon = false, addr; tmp_buf = skb->data + pstatus->rx_drvinfo_size + pstatus->rx_bufshift;