lkml.org 
[lkml]   [2021]   [Nov]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.15 347/917] wilc1000: fix possible memory leak in cfg_scan_result()
    Date
    From: Ajay Singh <ajay.kathat@microchip.com>

    [ Upstream commit 3c719fed0f3a5e95b1d164609ecc81c4191ade70 ]

    When the BSS reference holds a valid reference, it is not freed. The 'if'
    condition is wrong. Instead of the 'if (bss)' check, the 'if (!bss)' check
    is used.
    The issue is solved by removing the unnecessary 'if' check because
    cfg80211_put_bss() already performs the NULL validation.

    Fixes: 6cd4fa5ab691 ("staging: wilc1000: make use of cfg80211_inform_bss_frame()")
    Signed-off-by: Ajay Singh <ajay.kathat@microchip.com>
    Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
    Link: https://lore.kernel.org/r/20210916164902.74629-3-ajay.kathat@microchip.com
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/net/wireless/microchip/wilc1000/cfg80211.c | 3 +--
    1 file changed, 1 insertion(+), 2 deletions(-)

    diff --git a/drivers/net/wireless/microchip/wilc1000/cfg80211.c b/drivers/net/wireless/microchip/wilc1000/cfg80211.c
    index 96973ec7bd9ac..87c14969c75fa 100644
    --- a/drivers/net/wireless/microchip/wilc1000/cfg80211.c
    +++ b/drivers/net/wireless/microchip/wilc1000/cfg80211.c
    @@ -129,8 +129,7 @@ static void cfg_scan_result(enum scan_event scan_event,
    info->frame_len,
    (s32)info->rssi * 100,
    GFP_KERNEL);
    - if (!bss)
    - cfg80211_put_bss(wiphy, bss);
    + cfg80211_put_bss(wiphy, bss);
    } else if (scan_event == SCAN_EVENT_DONE) {
    mutex_lock(&priv->scan_req_lock);

    --
    2.33.0


    \
     
     \ /
      Last update: 2021-11-16 00:38    [W:2.771 / U:0.064 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site