lkml.org 
[lkml]   [2022]   [Oct]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 6.0 266/862] wifi: mac80211: mlme: assign link address correctly
    Date
    From: Johannes Berg <johannes.berg@intel.com>

    [ Upstream commit acdc3e47881d86dc1cb89d4603e3fed90ab150db ]

    Right now, we assign the link address only after we add
    the link to the driver, which is quite obviously wrong.
    It happens to work in many cases because it gets updated
    immediately, and then link_conf updates may update it,
    but it's clearly not really right.

    Set the link address during ieee80211_mgd_setup_link()
    so it's set before telling the driver about the link.

    Fixes: 81151ce462e5 ("wifi: mac80211: support MLO authentication/association with one link")
    Signed-off-by: Johannes Berg <johannes.berg@intel.com>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    net/mac80211/mlme.c | 5 ++---
    1 file changed, 2 insertions(+), 3 deletions(-)

    diff --git a/net/mac80211/mlme.c b/net/mac80211/mlme.c
    index 76ae6f03d77e..654414caeb71 100644
    --- a/net/mac80211/mlme.c
    +++ b/net/mac80211/mlme.c
    @@ -6291,6 +6291,8 @@ void ieee80211_mgd_setup_link(struct ieee80211_link_data *link)
    if (sdata->u.mgd.assoc_data)
    ether_addr_copy(link->conf->addr,
    sdata->u.mgd.assoc_data->link[link_id].addr);
    + else if (!is_valid_ether_addr(link->conf->addr))
    + eth_random_addr(link->conf->addr);
    }

    /* scan finished notification */
    @@ -6378,9 +6380,6 @@ static int ieee80211_prep_connection(struct ieee80211_sub_if_data *sdata,
    goto out_err;
    }

    - if (mlo && !is_valid_ether_addr(link->conf->addr))
    - eth_random_addr(link->conf->addr);
    -
    if (WARN_ON(!ifmgd->auth_data && !ifmgd->assoc_data)) {
    err = -EINVAL;
    goto out_err;
    --
    2.35.1


    \
     
     \ /
      Last update: 2022-10-19 12:43    [W:2.272 / U:0.196 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site