b43: Don't use memcpy for overlapping areas
authorMichael Buesch <mb@bu3sch.de>
Thu, 28 Aug 2008 17:13:51 +0000 (19:13 +0200)
committerJohn W. Linville <linville@tuxdriver.com>
Fri, 5 Sep 2008 20:15:24 +0000 (16:15 -0400)
It is valid to pass &gphy->rfatt and &gphy->bbatt as rfatt and bbatt
pointer arguments to the function. So we have to use memmove for the
possibly overlapping memory areas.

Signed-off-by: Michael Buesch <mb@bu3sch.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/b43/phy_g.c

index fce84896d34cdb1578e3ef991aeac8e0edd657e3..063db5c00ce7f81f678547b0152a130f4ed6261d 100644 (file)
@@ -232,11 +232,12 @@ void b43_set_txpower_g(struct b43_wldev *dev,
        if (unlikely(tx_bias == 0xFF))
                tx_bias = 0;
 
-       /* Save the values for later */
+       /* Save the values for later. Use memmove, because it's valid
+        * to pass &gphy->rfatt as rfatt pointer argument. Same for bbatt. */
        gphy->tx_control = tx_control;
-       memcpy(&gphy->rfatt, rfatt, sizeof(*rfatt));
+       memmove(&gphy->rfatt, rfatt, sizeof(*rfatt));
        gphy->rfatt.with_padmix = !!(tx_control & B43_TXCTL_TXMIX);
-       memcpy(&gphy->bbatt, bbatt, sizeof(*bbatt));
+       memmove(&gphy->bbatt, bbatt, sizeof(*bbatt));
 
        if (b43_debug(dev, B43_DBG_XMITPOWER)) {
                b43dbg(dev->wl, "Tuning TX-power to bbatt(%u), "