On Thu, Mar 7, 2019 at 4:03 PM Nathan Chancellor <natechancellor@xxxxxxxxx> wrote: > > When building with -Wsometimes-uninitialized, Clang warns: > > drivers/net/wireless/intel/iwlwifi/mvm/sta.c:2114:12: warning: variable > 'queue' is used uninitialized whenever 'if' condition is false > [-Wsometimes-uninitialized] > > Clang can't evaluate at this point that WARN(1, ...) always returns true > because __ret_warn_on is defined as !!(condition), which isn't > immediately evaluated as 1. Change this branch to else so that it's > clear to Clang that we intend to bail out here. > > Link: https://github.com/ClangBuiltLinux/linux/issues/399 > Signed-off-by: Nathan Chancellor <natechancellor@xxxxxxxxx> Thanks for the simple fix. Reviewed-by: Nick Desaulniers <ndesaulniers@xxxxxxxxxx> > --- > drivers/net/wireless/intel/iwlwifi/mvm/sta.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/drivers/net/wireless/intel/iwlwifi/mvm/sta.c b/drivers/net/wireless/intel/iwlwifi/mvm/sta.c > index 498c315291cf..360724ec41a6 100644 > --- a/drivers/net/wireless/intel/iwlwifi/mvm/sta.c > +++ b/drivers/net/wireless/intel/iwlwifi/mvm/sta.c > @@ -2111,8 +2111,10 @@ int iwl_mvm_send_add_bcast_sta(struct iwl_mvm *mvm, struct ieee80211_vif *vif) > queue = mvm->probe_queue; > else if (vif->type == NL80211_IFTYPE_P2P_DEVICE) > queue = mvm->p2p_dev_queue; > - else if (WARN(1, "Missing required TXQ for adding bcast STA\n")) > + else { > + WARN(1, "Missing required TXQ for adding bcast STA\n"); > return -EINVAL; > + } > > bsta->tfd_queue_msk |= BIT(queue); > > -- > 2.21.0 > -- Thanks, ~Nick Desaulniers
