lkml.org 
[lkml]   [2018]   [May]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 4.14 256/496] iwlwifi: mvm: fix error checking for multi/broadcast sta
    Date
    4.14-stable review patch.  If anyone has any objections, please let me know.

    ------------------

    From: Johannes Berg <johannes.berg@intel.com>

    [ Upstream commit 75fd4fec3e4c43b131c7c4958adb3ab9f1665513 ]

    The earlier patch called the station add functions but didn't
    assign their return value to the ret variable, so that the
    checks for it were meaningless. Fix that.

    Found by smatch:

    .../mac80211.c:2560 iwl_mvm_start_ap_ibss() warn: we tested 'ret' before and it was 'false'
    .../mac80211.c:2563 iwl_mvm_start_ap_ibss() warn: we tested 'ret' before and it was 'false'

    Fixes: 3a89411cd31c ("iwlwifi: mvm: fix assert 0x2B00 on older FWs")
    Signed-off-by: Johannes Berg <johannes.berg@intel.com>
    Signed-off-by: Luca Coelho <luciano.coelho@intel.com>
    Signed-off-by: Sasha Levin <alexander.levin@microsoft.com>
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
    ---
    drivers/net/wireless/intel/iwlwifi/mvm/mac80211.c | 4 ++--
    1 file changed, 2 insertions(+), 2 deletions(-)

    --- a/drivers/net/wireless/intel/iwlwifi/mvm/mac80211.c
    +++ b/drivers/net/wireless/intel/iwlwifi/mvm/mac80211.c
    @@ -2153,10 +2153,10 @@ static int iwl_mvm_start_ap_ibss(struct
    * Send the bcast station. At this stage the TBTT and DTIM time
    * events are added and applied to the scheduler
    */
    - iwl_mvm_send_add_bcast_sta(mvm, vif);
    + ret = iwl_mvm_send_add_bcast_sta(mvm, vif);
    if (ret)
    goto out_unbind;
    - iwl_mvm_add_mcast_sta(mvm, vif);
    + ret = iwl_mvm_add_mcast_sta(mvm, vif);
    if (ret) {
    iwl_mvm_send_rm_bcast_sta(mvm, vif);
    goto out_unbind;

    \
     
     \ /
      Last update: 2018-05-28 15:05    [W:7.167 / U:0.052 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site