wifi: fix memory leak

each kmalloc should have its own kfree

This fixes coverity issue 33605

Bug 2028892

Change-Id: I5903ec44cca0ad274e434df5517972b357991c0b
Signed-off-by: dmitry pervushin <dpervushin@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2311738
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>
Reviewed-by: automaticguardword <automaticguardword@nvidia.com>
Reviewed-by: Automatic_Commit_Validation_User
Reviewed-by: Phoenix Jung <pjung@nvidia.com>
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
GVS: Gerrit_Virtual_Submit

1 files changed, 7 insertions, 2 deletions

diff --git a/drivers/net/wireless/bcmdhd_pcie/wldev_common.c b/drivers/net/wireless/bcmdhd_pcie/wldev_common.c
index 61280157e..0eec088ea 100644
--- a/drivers/net/wireless/bcmdhd_pcie/wldev_common.c
+++ b/drivers/net/wireless/bcmdhd_pcie/wldev_common.c
@@ -4,6 +4,7 @@
  * Portions of this code are copyright (c) 2017 Cypress Semiconductor Corporation
  * 
  * Copyright (C) 1999-2017, Broadcom Corporation
+ * Copyright (c) 2020, NVIDIA CORPORATION.  All rights reserved.
  * 
  *      Unless you and Broadcom execute a separate written software license
  * agreement governing use of this software, this software is licensed to you
@@ -374,7 +375,8 @@ int wldev_get_mode(
         error = wldev_ioctl(dev, WLC_GET_BSS_INFO, (void*)buf, WL_EXTRA_BUF_MAX, false);
         if (error) {
                 WLDEV_ERROR(("%s:failed:%d\n", __FUNCTION__, error));
-                return -1;
+                error = -1;
+                goto out;
         }
         bss = (struct  wl_bss_info *)(buf + 4);
         chanspec = wl_chspec_driver_to_host(bss->chanspec);
@@ -401,10 +403,13 @@ int wldev_get_mode(
                                 strcpy(cap, "a");
                 } else {
                         WLDEV_ERROR(("%s:Mode get failed\n", __FUNCTION__));
-                        return -1;
+                        error = -1;
+                        goto out;
                 }
 
         }
+out:
+        kfree(buf);
         return error;
 }
 int wldev_set_country(