brcm80211: Remove bogus memcpy in ai_detach
authorAndi Kleen <andi@firstfloor.org>
Mon, 3 Sep 2012 19:13:35 +0000 (21:13 +0200)
committerBen Hutchings <ben@decadent.org.uk>
Mon, 19 Mar 2018 18:58:21 +0000 (18:58 +0000)
commit6fee38de5df30ffb0a5e707c5b1ee16bb35c669b
tree12f5bda1e5f183eb51fa2d52b8e889e0ff8bfd6e
parenta25b5adbc6b283c17a025a706b926faf51e06478
brcm80211: Remove bogus memcpy in ai_detach

commit af2c8ffe56133928355d1d51978b35115ffbbc2a upstream.

gcc 4.8 warns for this memcpy. While the copy size is correct, the whole
copy seems to be a nop because the destination is never used, and
there's no need to use memcpy to copy pointers anyways. And the
type of the pointer was wrong, but at least those are always the same.

Just remove it.

/backup/lsrc/git/linux-lto-2.6/drivers/net/wireless/brcm80211/brcmsmac/aiutils.c: In function 'ai_detach':
/backup/lsrc/git/linux-lto-2.6/drivers/net/wireless/brcm80211/brcmsmac/aiutils.c:539:32: warning: argument to 'sizeof' in 'memcpy' call is the same pointer type 'struct si_pub **' as the destination; expected 'struct si_pub *' or an explicit length [-Wsizeof-pointer-memaccess]
  memcpy(&si_local, &sih, sizeof(struct si_pub **));
                                ^

Signed-off-by: Andi Kleen <ak@linux.intel.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
[bwh: Backported to 3.2: adjust context]
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
drivers/net/wireless/brcm80211/brcmsmac/aiutils.c