staging: vt6656: Use proper target pointer in memcpy()
authorTobias Klauser <tklauser@distanz.ch>
Fri, 25 Apr 2014 09:53:58 +0000 (11:53 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 15 May 2014 20:23:03 +0000 (13:23 -0700)
commit838c2d6cfc0c2dc8ce136e01e681fc6cf4124c39
treee39d5b54774340fe67c3a1d951b7722af486a1f7
parent34bdb981f7cb446259b0999d40f090c16f732eca
staging: vt6656: Use proper target pointer in memcpy()

The coverity scanner marked these two memcpy()'s as causing a buffer
overflow in CIDs 142743 and 142744. This is due the h_dest member of
struct ethhdr being used as a target (size ETH_ALEN) in memcpy, but the
copy is of size ETH_HLEN. However, the intention here seems to be to
copy the entire ethernet header. Make that clear by specifying the
proper destination buffer.

Also remove the unnecessary casts of the source argument.

Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/vt6656/rxtx.c