mwifiex: replace kmalloc & memcpy sequence with kmemdup
authorYogesh Ashok Powar <yogeshp@marvell.com>
Mon, 29 Aug 2011 20:21:10 +0000 (13:21 -0700)
committerJohn W. Linville <linville@tuxdriver.com>
Tue, 13 Sep 2011 19:42:32 +0000 (15:42 -0400)
Sequence of kmalloc/kzalloc and memcpy is replaced with
kmemdup.

Cc: Walter Harms <wharms@bfs.de>
Signed-off-by: Yogesh Ashok Powar <yogeshp@marvell.com>
Signed-off-by: Kiran Divekar <dkiran@marvell.com>
Signed-off-by: Bing Zhao <bzhao@marvell.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/mwifiex/join.c
drivers/net/wireless/mwifiex/scan.c
drivers/net/wireless/mwifiex/sta_ioctl.c

index 5cdad92277fab0252d55d1bf873d4e30d3b1e4b3..62b4c2938608d67873d66afb8be9e288247820b9 100644 (file)
@@ -147,13 +147,12 @@ static int mwifiex_get_common_rates(struct mwifiex_private *priv, u8 *rate1,
        u8 *ptr = rate1, *tmp;
        u32 i, j;
 
-       tmp = kmalloc(rate1_size, GFP_KERNEL);
+       tmp = kmemdup(rate1, rate1_size, GFP_KERNEL);
        if (!tmp) {
                dev_err(priv->adapter->dev, "failed to alloc tmp buf\n");
                return -ENOMEM;
        }
 
-       memcpy(tmp, rate1, rate1_size);
        memset(rate1, 0, rate1_size);
 
        for (i = 0; rate2[i] && i < rate2_size; i++) {
index 37ca2f90ad63360486b8e2e7cd441e13931bed12..8d8588db1cd960d71fc34c4e564768ecf40bd30b 100644 (file)
@@ -1479,12 +1479,12 @@ mwifiex_update_curr_bss_params(struct mwifiex_private *priv,
                dev_err(priv->adapter->dev, " failed to alloc bss_desc\n");
                return -ENOMEM;
        }
-       beacon_ie = kzalloc(ie_len, GFP_KERNEL);
+
+       beacon_ie = kmemdup(ie_buf, ie_len, GFP_KERNEL);
        if (!beacon_ie) {
                dev_err(priv->adapter->dev, " failed to alloc beacon_ie\n");
                return -ENOMEM;
        }
-       memcpy(beacon_ie, ie_buf, ie_len);
 
        ret = mwifiex_fill_new_bss_desc(priv, bssid, rssi, beacon_ie,
                                        ie_len, beacon_period,
@@ -1986,7 +1986,7 @@ mwifiex_save_curr_bcn(struct mwifiex_private *priv)
                priv->curr_bcn_size = curr_bss->beacon_buf_size;
 
                kfree(priv->curr_bcn_buf);
-               priv->curr_bcn_buf = kzalloc(curr_bss->beacon_buf_size,
+               priv->curr_bcn_buf = kmalloc(curr_bss->beacon_buf_size,
                                                GFP_KERNEL);
                if (!priv->curr_bcn_buf) {
                        dev_err(priv->adapter->dev,
index 3fca219bcfb6deea19dcef14e324c2f5e9d8b1ea..eb569fa9adbaa5652da67ae297e9e6b8693c0684 100644 (file)
@@ -199,13 +199,14 @@ int mwifiex_bss_start(struct mwifiex_private *priv, struct cfg80211_bss *bss,
                        dev_err(priv->adapter->dev, " failed to alloc bss_desc\n");
                        return -ENOMEM;
                }
-               beacon_ie = kzalloc(bss->len_beacon_ies, GFP_KERNEL);
+
+               beacon_ie = kmemdup(bss->information_elements,
+                                       bss->len_beacon_ies, GFP_KERNEL);
                if (!beacon_ie) {
-                       dev_err(priv->adapter->dev, " failed to alloc bss_desc\n");
+                       dev_err(priv->adapter->dev, " failed to alloc beacon_ie\n");
                        return -ENOMEM;
                }
-               memcpy(beacon_ie, bss->information_elements,
-                      bss->len_beacon_ies);
+
                ret = mwifiex_fill_new_bss_desc(priv, bss->bssid, bss->signal,
                                                beacon_ie, bss->len_beacon_ies,
                                                bss->beacon_interval,