diff options
| author | Mohan T <mohant@nvidia.com> | 2014-02-11 15:48:40 +0530 |
|---|---|---|
| committer | Mandar Padmawar <mpadmawar@nvidia.com> | 2014-06-03 05:18:59 -0700 |
| commit | 7c7de221c379f99e2bbae3af51628e8b8610335f (patch) | |
| tree | 4bc1d81c6ff4c67d58fc515505db6d500151557a /drivers/net | |
| parent | edb3d07a348a9f2e88dcfb903c2bc65c092b92d8 (diff) | |
bcmdhd: NULL checks for wl_cfgp2p_generate_bss_mac
On concurrent P2P and STA connection stress tests observe
NULL pointer issue in wl_cfgp2p_generate_bss_mac function.
So add NULL checks for input parameters.
Bug 1455328
Change-Id: I884d1612b1df16c5acd038b8cd0f51129e5d9130
Signed-off-by: Mohan T <mohant@nvidia.com>
Reviewed-on: http://git-master/r/385779
(cherry picked from commit 21a162bc3f9eb7ec0153ff87cdbf28433fe34a6f)
Reviewed-on: http://git-master/r/417394
GVS: Gerrit_Virtual_Submit
Reviewed-by: Narayan Reddy <narayanr@nvidia.com>
Tested-by: Narayan Reddy <narayanr@nvidia.com>
Reviewed-by: Laxman Dewangan <ldewangan@nvidia.com>
Diffstat (limited to 'drivers/net')
| -rwxr-xr-x | drivers/net/wireless/bcmdhd/wl_cfgp2p.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/drivers/net/wireless/bcmdhd/wl_cfgp2p.c b/drivers/net/wireless/bcmdhd/wl_cfgp2p.c index bc5caf1dfbbb..2a8054452e47 100755 --- a/drivers/net/wireless/bcmdhd/wl_cfgp2p.c +++ b/drivers/net/wireless/bcmdhd/wl_cfgp2p.c @@ -1857,6 +1857,11 @@ void wl_cfgp2p_generate_bss_mac(struct ether_addr *primary_addr, struct ether_addr *out_dev_addr, struct ether_addr *out_int_addr) { + if ((out_dev_addr == NULL) || (out_int_addr == NULL)) { + WL_ERR(("Invalid input data\n")); + return; + } + memset(out_dev_addr, 0, sizeof(*out_dev_addr)); memset(out_int_addr, 0, sizeof(*out_int_addr)); |