summaryrefslogtreecommitdiffstats
path: root/net/wireless/core.h
diff options
context:
space:
mode:
authorLuciano Coelho <coelho@ti.com>2012-06-18 15:52:51 +0300
committerJohannes Berg <johannes.berg@intel.com>2012-06-20 10:54:23 +0200
commit6df653c71e8168e1df01118cc85cd84d0deeb583 (patch)
treef1a05e10a875eeff275bb7256fa06ced3ced0842 /net/wireless/core.h
parent559cef996d2e4c9b652a53bb3a53e5787e247f57 (diff)
downloadlinux-6df653c71e8168e1df01118cc85cd84d0deeb583.tar.bz2
mac80211: initialize sta pointer to avoid false-positive warning
Some compilers (eg. gcc 4.4.1 for ARM) report a false positive warning in mlme.c: net/mac80211/mlme.c: In function 'ieee80211_prep_connection': net/mac80211/mlme.c:3035: warning: 'sta' may be used uninitialized in this function This is a false positive because the place where 'sta' is used is inside an if with the same condition of where it is set: [...] if (!have_sta) { sta = sta_info_alloc(sdata, cbss->bssid, GFP_KERNEL); if (!sta) return -ENOMEM; } [...] if (!have_sta) { [...] sta->sta.supp_rates[cbss->channel->band] = rates; [...] For some reason the compiler doesn't understand this and warns. While this is not a problem in the code itself, we can avoid polluting the build logs with false positives by setting sta to NULL on declaration and checking for sta instead of !have_sta in the second if. Reported-by: Tony Lindgren <tony@atomide.com> Signed-off-by: Luciano Coelho <coelho@ti.com> Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Diffstat (limited to 'net/wireless/core.h')
0 files changed, 0 insertions, 0 deletions