batman-adv: get rid of pointless cast in memcpy()
authorAl Viro <viro@ZenIV.linux.org.uk>
Sun, 22 Apr 2012 06:50:29 +0000 (07:50 +0100)
committerAntonio Quartulli <ordex@autistici.org>
Mon, 18 Jun 2012 16:01:01 +0000 (18:01 +0200)
memcpy() arguments are void *, precisely to avoid that kind of pointless
casts.

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Sven Eckelmann <sven@narfation.org>
net/batman-adv/bridge_loop_avoidance.c

index 5c1ac559edbb8ba749892224c28ebe486f4251d2..89e2b1cda1827498712e2898dc600654ae6ddeb1 100644 (file)
@@ -510,7 +510,7 @@ static void bla_send_announce(struct bat_priv *bat_priv,
 
        memcpy(mac, announce_mac, 4);
        crc = htons(backbone_gw->crc);
-       memcpy(&mac[4], (uint8_t *)&crc, 2);
+       memcpy(&mac[4], &crc, 2);
 
        bla_send_claim(bat_priv, mac, backbone_gw->vid, CLAIM_TYPE_ANNOUNCE);