On 11/11/2011 12:32 PM, John W. Linville wrote:
CC [M] net/mac80211/cfg.o net/mac80211/cfg.c: In function ‘ieee80211_probe_client’: net/mac80211/cfg.c:2567:7: warning: ‘qos’ may be used uninitialized in this function Signed-off-by: John W. Linville<linville@xxxxxxxxxxxxx> --- net/mac80211/cfg.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/net/mac80211/cfg.c b/net/mac80211/cfg.c index c2416fb..1d82d2b 100644 --- a/net/mac80211/cfg.c +++ b/net/mac80211/cfg.c @@ -2564,7 +2564,7 @@ static int ieee80211_probe_client(struct wiphy *wiphy, struct net_device *dev, struct sk_buff *skb; int size = sizeof(*nullfunc); __le16 fc; - bool qos; + bool qos = NULL; struct ieee80211_tx_info *info; struct sta_info *sta;
Shouldn't the initialization for a bool be "false"? Doesn't checkpatch.pl complain about that? I know it does if you try to initialize a pointer to 0 rather than NULL.
Larry -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
