Skip to content

Commit eae127b

Browse files
Phil Elwellpopcornmix
Phil Elwell
authored andcommitted
brcmfmac: Plug memory leak in brcmf_fill_bss_param
See: #1471 Signed-off-by: Phil Elwell <[email protected]>
1 parent b46f534 commit eae127b

File tree

1 file changed

+4
-1
lines changed
  • drivers/net/wireless/broadcom/brcm80211/brcmfmac

1 file changed

+4
-1
lines changed

drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2473,7 +2473,7 @@ static void brcmf_fill_bss_param(struct brcmf_if *ifp, struct station_info *si)
24732473
WL_BSS_INFO_MAX);
24742474
if (err) {
24752475
brcmf_err("Failed to get bss info (%d)\n", err);
2476-
return;
2476+
goto out_err;
24772477
}
24782478
si->filled |= BIT(NL80211_STA_INFO_BSS_PARAM);
24792479
si->bss_param.beacon_interval = le16_to_cpu(buf->bss_le.beacon_period);
@@ -2485,6 +2485,9 @@ static void brcmf_fill_bss_param(struct brcmf_if *ifp, struct station_info *si)
24852485
si->bss_param.flags |= BSS_PARAM_FLAGS_SHORT_PREAMBLE;
24862486
if (capability & WLAN_CAPABILITY_SHORT_SLOT_TIME)
24872487
si->bss_param.flags |= BSS_PARAM_FLAGS_SHORT_SLOT_TIME;
2488+
2489+
out_err:
2490+
kfree(buf);
24882491
}
24892492

24902493
static s32

0 commit comments

Comments
 (0)